ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/Compress-LZF/LZF.xs
(Generate patch)

Comparing Compress-LZF/LZF.xs (file contents):
Revision 1.19 by pcg, Tue Aug 3 15:07:51 2004 UTC vs.
Revision 1.31 by root, Sun Aug 25 18:13:26 2013 UTC

1#include "EXTERN.h" 1#include "EXTERN.h"
2#include "perl.h" 2#include "perl.h"
3#include "XSUB.h" 3#include "XSUB.h"
4 4
5/* try to be compatible with older perls */ 5#define LZF_STANDALONE 1
6/* SvPV_nolen() macro first defined in 5.005_55 */ 6#define LZF_STATE_ARG 1
7/* this is slow, not threadsafe, but works */
8#include "patchlevel.h"
9#if (PATCHLEVEL == 4) || ((PATCHLEVEL == 5) && (SUBVERSION < 55))
10static STRLEN nolen_na;
11# define SvPV_nolen(sv) SvPV ((sv), nolen_na)
12#endif
13#if PATCHLEVEL < 6
14# define call_sv perl_call_sv
15#endif
16 7
17#include "lzf_c.c" 8#include "lzf_c.c"
18#include "lzf_d.c" 9#include "lzf_d.c"
10#include "lzf_c_best.c"
19 11
20/* we re-use the storable header for our purposes */ 12/* we re-use the storable header for our purposes */
21#define MAGIC_LO 0 13#define MAGIC_LO 0
22#define MAGIC_U 0 /* uncompressed data follows */ 14#define MAGIC_U 0 /* uncompressed data follows */
23#define MAGIC_C 1 /* compressed data follows */ 15#define MAGIC_C 1 /* compressed data follows */
… …
25#define MAGIC_CR 3 /* storable (reference, freeze), compressed */ 17#define MAGIC_CR 3 /* storable (reference, freeze), compressed */
26#define MAGIC_R 4 /* storable (reference, freeze) */ 18#define MAGIC_R 4 /* storable (reference, freeze) */
27#define MAGIC_CR_deref 5 /* storable (NO reference, freeze), compressed */ 19#define MAGIC_CR_deref 5 /* storable (NO reference, freeze), compressed */
28#define MAGIC_R_deref 6 /* storable (NO reference, freeze) */ 20#define MAGIC_R_deref 6 /* storable (NO reference, freeze) */
29#define MAGIC_HI 7 /* room for one higher storable major */ 21#define MAGIC_HI 7 /* room for one higher storable major */
22/* for historical reasons, MAGIC_undef + MAGIC_R and MAGIC_undef + MAGIC_R_deref are used, too */
30 23
31#define IN_RANGE(v,l,h) ((unsigned int)((unsigned)(v) - (unsigned)(l)) <= (unsigned)(h) - (unsigned)(l)) 24#define IN_RANGE(v,l,h) ((unsigned int)((unsigned)(v) - (unsigned)(l)) <= (unsigned)(h) - (unsigned)(l))
32 25
33static SV *serializer_package, *serializer_mstore, *serializer_mretrieve; 26static SV *serializer_package, *serializer_mstore, *serializer_mretrieve;
34static CV *storable_mstore, *storable_mretrieve; 27static CV *storable_mstore, *storable_mretrieve;
35 28
29#if Size_t_size > 4
30# define MAX_LENGTH ((Size_t)0x80000000L)
31#else
32# define MAX_LENGTH ((Size_t) 0x8000000L)
33#endif
34
36static SV * 35static SV *
37compress_sv (SV *data, char cprepend, int uprepend) 36compress_sv (SV *data, char cprepend, int uprepend, int best)
38{ 37{
38 LZF_STATE *state;
39 STRLEN usize, csize; 39 STRLEN usize, csize;
40 char *src = (char *)SvPV (data, usize); 40 char *src = (char *)SvPVbyte (data, usize);
41 41
42 if (usize) 42 if (usize)
43 { 43 {
44 SV *ret = NEWSV (0, usize + 1); 44 SV *ret = NEWSV (0, usize + 1);
45 unsigned char *dst; 45 unsigned char *dst;
… …
49 dst = (unsigned char *)SvPVX (ret); 49 dst = (unsigned char *)SvPVX (ret);
50 50
51 if (cprepend) 51 if (cprepend)
52 dst[skip++] = cprepend; 52 dst[skip++] = cprepend;
53 53
54 if (usize < 0x80) 54 if (usize <= 0x7f)
55 { 55 {
56 dst[skip++] = usize; 56 dst[skip++] = usize;
57 } 57 }
58 else if (usize < 0x800) 58 else if (usize <= 0x7ff)
59 { 59 {
60 dst[skip++] = (( usize >> 6) | 0xc0); 60 dst[skip++] = (( usize >> 6) | 0xc0);
61 dst[skip++] = (( usize & 0x3f) | 0x80); 61 dst[skip++] = (( usize & 0x3f) | 0x80);
62 } 62 }
63 else if (usize < 0x10000) 63 else if (usize <= 0xffff)
64 { 64 {
65 dst[skip++] = (( usize >> 12) | 0xe0); 65 dst[skip++] = (( usize >> 12) | 0xe0);
66 dst[skip++] = (((usize >> 6) & 0x3f) | 0x80); 66 dst[skip++] = (((usize >> 6) & 0x3f) | 0x80);
67 dst[skip++] = (( usize & 0x3f) | 0x80); 67 dst[skip++] = (( usize & 0x3f) | 0x80);
68 } 68 }
69 else if (usize < 0x200000) 69 else if (usize <= 0x1fffff)
70 { 70 {
71 dst[skip++] = (( usize >> 18) | 0xf0); 71 dst[skip++] = (( usize >> 18) | 0xf0);
72 dst[skip++] = (((usize >> 12) & 0x3f) | 0x80); 72 dst[skip++] = (((usize >> 12) & 0x3f) | 0x80);
73 dst[skip++] = (((usize >> 6) & 0x3f) | 0x80); 73 dst[skip++] = (((usize >> 6) & 0x3f) | 0x80);
74 dst[skip++] = (( usize & 0x3f) | 0x80); 74 dst[skip++] = (( usize & 0x3f) | 0x80);
75 } 75 }
76 else if (usize < 0x4000000) 76 else if (usize <= 0x3ffffff)
77 { 77 {
78 dst[skip++] = (( usize >> 24) | 0xf8); 78 dst[skip++] = (( usize >> 24) | 0xf8);
79 dst[skip++] = (((usize >> 18) & 0x3f) | 0x80); 79 dst[skip++] = (((usize >> 18) & 0x3f) | 0x80);
80 dst[skip++] = (((usize >> 12) & 0x3f) | 0x80); 80 dst[skip++] = (((usize >> 12) & 0x3f) | 0x80);
81 dst[skip++] = (((usize >> 6) & 0x3f) | 0x80); 81 dst[skip++] = (((usize >> 6) & 0x3f) | 0x80);
82 dst[skip++] = (( usize & 0x3f) | 0x80); 82 dst[skip++] = (( usize & 0x3f) | 0x80);
83 } 83 }
84 else if (usize <= 0x7fffffff)
85 {
86 dst[skip++] = (( usize >> 30) | 0xfc);
87 dst[skip++] = (((usize >> 24) & 0x3f) | 0x80);
88 dst[skip++] = (((usize >> 18) & 0x3f) | 0x80);
89 dst[skip++] = (((usize >> 12) & 0x3f) | 0x80);
90 dst[skip++] = (((usize >> 6) & 0x3f) | 0x80);
91 dst[skip++] = (( usize & 0x3f) | 0x80);
92 }
84 else 93 else
85 croak ("compress can only compress up to %ld bytes", 0x4000000-1); 94 croak ("compress can only compress up to %ld bytes", 0x7fffffffL);
95
96 New (0, state, 1, LZF_STATE);
97 if (!state)
98 croak ("Compress::LZF unable to allocate memory for compression state");
86 99
87 /* 11 bytes is the smallest compressible string */ 100 /* 11 bytes is the smallest compressible string */
88 csize = usize < 11 ? 0 : 101 csize = usize < 11 ? 0 :
89 lzf_compress (src, usize, 102 (best ? lzf_compress_best (src, usize, dst + skip, usize - skip)
90 dst + skip, 103 : lzf_compress (src, usize, dst + skip, usize - skip, *state));
91 usize - skip); 104
105 Safefree (state);
92 106
93 if (csize) 107 if (csize)
94 { 108 {
95 SvCUR_set (ret, csize + skip); 109 SvCUR_set (ret, csize + skip);
96 } 110 }
… …
116 130
117static SV * 131static SV *
118decompress_sv (SV *data, int skip) 132decompress_sv (SV *data, int skip)
119{ 133{
120 STRLEN usize, csize; 134 STRLEN usize, csize;
121 unsigned char *src = (unsigned char *)SvPV (data, csize) + skip; 135 unsigned char *src = (unsigned char *)SvPVbyte (data, csize) + skip;
122 136
123 if (csize) 137 if (csize)
124 { 138 {
125 void *dst; 139 void *dst;
126 SV *ret; 140 SV *ret;
127 141
128 csize -= skip; 142 csize -= skip;
129 143
130 if (src[0]) 144 if (src[0])
131 { 145 {
132 if (!(src[0] & 0x80)) 146 if (!(src[0] & 0x80) && csize >= 1)
133 { 147 {
134 csize -= 1; 148 csize -= 1;
135 usize = *src++ & 0xff; 149 usize = *src++ & 0xff;
136 } 150 }
137 else if (!(src[0] & 0x20)) 151 else if (!(src[0] & 0x20) && csize >= 2)
138 { 152 {
139 csize -= 2; 153 csize -= 2;
140 usize = *src++ & 0x1f; 154 usize = *src++ & 0x1f;
141 usize = (usize << 6) | (*src++ & 0x3f); 155 usize = (usize << 6) | (*src++ & 0x3f);
142 } 156 }
143 else if (!(src[0] & 0x10)) 157 else if (!(src[0] & 0x10) && csize >= 3)
144 { 158 {
145 csize -= 3; 159 csize -= 3;
146 usize = *src++ & 0x0f; 160 usize = *src++ & 0x0f;
147 usize = (usize << 6) | (*src++ & 0x3f); 161 usize = (usize << 6) | (*src++ & 0x3f);
148 usize = (usize << 6) | (*src++ & 0x3f); 162 usize = (usize << 6) | (*src++ & 0x3f);
149 } 163 }
150 else if (!(src[0] & 0x08)) 164 else if (!(src[0] & 0x08) && csize >= 4)
151 { 165 {
152 csize -= 4; 166 csize -= 4;
153 usize = *src++ & 0x07; 167 usize = *src++ & 0x07;
154 usize = (usize << 6) | (*src++ & 0x3f); 168 usize = (usize << 6) | (*src++ & 0x3f);
155 usize = (usize << 6) | (*src++ & 0x3f); 169 usize = (usize << 6) | (*src++ & 0x3f);
156 usize = (usize << 6) | (*src++ & 0x3f); 170 usize = (usize << 6) | (*src++ & 0x3f);
157 } 171 }
158 else if (!(src[0] & 0x04)) 172 else if (!(src[0] & 0x04) && csize >= 5)
159 { 173 {
160 csize -= 5; 174 csize -= 5;
161 usize = *src++ & 0x03; 175 usize = *src++ & 0x03;
162 usize = (usize << 6) | (*src++ & 0x3f); 176 usize = (usize << 6) | (*src++ & 0x3f);
163 usize = (usize << 6) | (*src++ & 0x3f); 177 usize = (usize << 6) | (*src++ & 0x3f);
164 usize = (usize << 6) | (*src++ & 0x3f); 178 usize = (usize << 6) | (*src++ & 0x3f);
165 usize = (usize << 6) | (*src++ & 0x3f); 179 usize = (usize << 6) | (*src++ & 0x3f);
166 } 180 }
181 else if (!(src[0] & 0x02) && csize >= 6)
182 {
183 csize -= 6;
184 usize = *src++ & 0x01;
185 usize = (usize << 6) | (*src++ & 0x3f);
186 usize = (usize << 6) | (*src++ & 0x3f);
187 usize = (usize << 6) | (*src++ & 0x3f);
188 usize = (usize << 6) | (*src++ & 0x3f);
189 usize = (usize << 6) | (*src++ & 0x3f);
190 }
167 else 191 else
192 croak ("compressed data corrupted (invalid length)");
193
194 if (!usize)
168 croak ("compressed data corrupted (invalid length)"); 195 croak ("compressed data corrupted (invalid length)");
169 196
170 ret = NEWSV (0, usize); 197 ret = NEWSV (0, usize);
171 SvPOK_only (ret); 198 SvPOK_only (ret);
172 dst = SvPVX (ret); 199 dst = SvPVX (ret);
173 200
174 if (lzf_decompress (src, csize, dst, usize) != usize) 201 if (lzf_decompress (src, csize, dst, usize) != usize)
202 {
203 SvREFCNT_dec (ret);
175 croak ("compressed data corrupted (size mismatch)", csize, skip, usize); 204 croak ("compressed data corrupted (size mismatch)", csize, skip, usize);
205 }
176 } 206 }
177 else 207 else
178 { 208 {
179 usize = csize - 1; 209 usize = csize - 1;
180 ret = NEWSV (0, usize); 210 ret = NEWSV (0, usize | 1);
181 SvPOK_only (ret); 211 SvPOK_only (ret);
182 212
183 Move ((void *)(src + 1), (void *)SvPVX (ret), usize, unsigned char); 213 Move ((void *)(src + 1), (void *)SvPVX (ret), usize, unsigned char);
184 } 214 }
185 215
… …
190 else 220 else
191 return newSVpvn ("", 0); 221 return newSVpvn ("", 0);
192} 222}
193 223
194static void 224static void
195need_storable(void) 225need_storable (void)
196{ 226{
197#if PATCHLEVEL < 6 227 eval_sv (sv_2mortal (newSVpvf ("require %s", SvPVbyte_nolen (serializer_package))), G_VOID | G_DISCARD);
198 char req[8192];
199 sprintf (req, "require %s;", SvPV_nolen (serializer_package));
200 perl_eval_pv (req, 1);
201#else
202 load_module (PERL_LOADMOD_NOIMPORT, serializer_package, Nullsv);
203#endif
204 228
205 storable_mstore = GvCV (gv_fetchpv (SvPV_nolen (serializer_mstore ), TRUE, SVt_PVCV)); 229 storable_mstore = (CV *)SvREFCNT_inc (GvCV (gv_fetchpv (SvPVbyte_nolen (serializer_mstore ), TRUE, SVt_PVCV)));
206 storable_mretrieve = GvCV (gv_fetchpv (SvPV_nolen (serializer_mretrieve), TRUE, SVt_PVCV)); 230 storable_mretrieve = (CV *)SvREFCNT_inc (GvCV (gv_fetchpv (SvPVbyte_nolen (serializer_mretrieve), TRUE, SVt_PVCV)));
207} 231}
208 232
209MODULE = Compress::LZF PACKAGE = Compress::LZF 233MODULE = Compress::LZF PACKAGE = Compress::LZF
210 234
211BOOT: 235BOOT:
212 serializer_package = newSVpv ("Storable", 0); 236 serializer_package = newSVpv ("Storable", 0);
213 serializer_mstore = newSVpv ("Storable::mstore", 0); 237 serializer_mstore = newSVpv ("Storable::net_mstore", 0);
214 serializer_mretrieve = newSVpv ("Storable::mretrieve", 0); 238 serializer_mretrieve = newSVpv ("Storable::mretrieve", 0);
215 239
216void 240void
217set_serializer(package, mstore, mretrieve) 241set_serializer(package, mstore, mretrieve)
218 SV * package 242 SV * package
… …
221 PROTOTYPE: $$$ 245 PROTOTYPE: $$$
222 PPCODE: 246 PPCODE:
223 SvSetSV (serializer_package , package ); 247 SvSetSV (serializer_package , package );
224 SvSetSV (serializer_mstore , mstore ); 248 SvSetSV (serializer_mstore , mstore );
225 SvSetSV (serializer_mretrieve, mretrieve); 249 SvSetSV (serializer_mretrieve, mretrieve);
226 storable_mstore = 250 SvREFCNT_dec (storable_mstore ); storable_mstore = 0;
227 storable_mretrieve = 0; 251 SvREFCNT_dec (storable_mretrieve); storable_mretrieve = 0;
228 252
229void 253void
230compress(data) 254compress(data)
231 SV * data 255 SV * data
256 ALIAS:
257 compress_best = 1
232 PROTOTYPE: $ 258 PROTOTYPE: $
233 PPCODE: 259 PPCODE:
234 XPUSHs (sv_2mortal (compress_sv (data, 0, MAGIC_U))); 260 XPUSHs (sv_2mortal (compress_sv (data, 0, MAGIC_U, ix)));
235 261
236void 262void
237decompress(data) 263decompress(data)
238 SV * data 264 SV * data
239 PROTOTYPE: $ 265 PROTOTYPE: $
… …
242 268
243void 269void
244sfreeze(sv) 270sfreeze(sv)
245 SV * sv 271 SV * sv
246 ALIAS: 272 ALIAS:
273 sfreeze = 0
247 sfreeze_cr = 1 274 sfreeze_cr = 1
248 sfreeze_c = 2 275 sfreeze_c = 2
276 sfreeze_best = 4
277 sfreeze_cr_best = 5
278 sfreeze_c_best = 6
249 PROTOTYPE: $ 279 PROTOTYPE: $
250 PPCODE: 280 PPCODE:
281{
282 int best = ix & 4;
283 ix &= 3;
251 284
252 SvGETMAGIC (sv); 285 SvGETMAGIC (sv);
253 286
254 if (!SvOK (sv)) 287 if (!SvOK (sv))
255 XPUSHs (sv_2mortal (newSVpvn ("\02", 1))); /* 02 == MAGIC_undef */ 288 XPUSHs (sv_2mortal (newSVpvn ("\02", 1))); /* 02 == MAGIC_undef */
256 else if (SvROK (sv) 289 else if (SvROK (sv)
290 || SvUTF8 (sv)
257 || (SvTYPE(sv) != SVt_IV 291 || (SvTYPE(sv) != SVt_IV
258 && SvTYPE(sv) != SVt_NV 292 && SvTYPE(sv) != SVt_NV
259 && SvTYPE(sv) != SVt_PV 293 && SvTYPE(sv) != SVt_PV
260 && SvTYPE(sv) != SVt_PVIV 294 && SvTYPE(sv) != SVt_PVIV
261 && SvTYPE(sv) != SVt_PVNV 295 && SvTYPE(sv) != SVt_PVNV
262 && SvTYPE(sv) != SVt_PVMG)) /* mstore */ 296 && SvTYPE(sv) != SVt_PVMG)) /* mstore */
263 { 297 {
264 int deref = !SvROK (sv); 298 int deref = !SvROK (sv);
299 char *pv;
265 300
266 if (!storable_mstore) 301 if (!storable_mstore)
302 {
303 PUTBACK;
267 need_storable (); 304 need_storable ();
305 SPAGAIN;
306 }
268 307
269 if (deref) 308 if (deref)
270 sv = newRV_noinc (sv); 309 sv = newRV_noinc (sv);
271 310
272 PUSHMARK (SP); 311 PUSHMARK (SP);
273 XPUSHs (sv); 312 XPUSHs (sv);
274 PUTBACK; 313 PUTBACK;
275 314
276 if (1 != call_sv ((SV *)storable_mstore, G_SCALAR)) 315 if (1 != call_sv ((SV *)storable_mstore, G_SCALAR))
277 croak ("Storable::mstore didn't return a single scalar"); 316 croak ("%s didn't return a single scalar", SvPVbyte_nolen (serializer_mstore));
278 317
279 SPAGAIN; 318 SPAGAIN;
280 319
281 sv = POPs; 320 sv = POPs;
321 pv = SvPV_nolen (sv);
282 322
283 if (SvPVX (sv)[0] != MAGIC_R) 323 if (*pv == MAGIC_R)
284 croak ("Storable format changed, need newer version of Compress::LZF"); 324 {
285
286 if (deref) 325 if (deref)
287 SvPVX (sv)[0] = MAGIC_R_deref; 326 *pv = MAGIC_R_deref;
327 }
328 else
329 {
330 char pfx[2];
331
332 pfx[0] = MAGIC_undef;
333 pfx[1] = deref ? MAGIC_R_deref : MAGIC_R;
334
335 sv_insert (sv, 0, 0, pfx, 2);
336 }
288 337
289 if (ix) /* compress */ 338 if (ix) /* compress */
290 sv = sv_2mortal (compress_sv (sv, deref ? MAGIC_CR_deref : MAGIC_CR, -1)); 339 sv = sv_2mortal (compress_sv (sv, deref ? MAGIC_CR_deref : MAGIC_CR, -1, best));
291 340
292 XPUSHs (sv); 341 XPUSHs (sv);
293 } 342 }
294 else if (SvPOKp (sv) && IN_RANGE (SvPVX (sv)[0], MAGIC_LO, MAGIC_HI)) 343 else if (SvPOKp (sv) && IN_RANGE (SvPVX (sv)[0], MAGIC_LO, MAGIC_HI))
295 XPUSHs (sv_2mortal (compress_sv (sv, MAGIC_C, MAGIC_U))); /* need to prefix only */ 344 XPUSHs (sv_2mortal (compress_sv (sv, MAGIC_C, MAGIC_U, best))); /* need to prefix only */
296 else if (ix == 2) /* compress always */ 345 else if (ix == 2) /* compress always */
297 XPUSHs (sv_2mortal (compress_sv (sv, MAGIC_C, -1))); 346 XPUSHs (sv_2mortal (compress_sv (sv, MAGIC_C, -1, best)));
298 else if (SvNIOK (sv)) /* don't compress */ 347 else if (SvNIOK (sv)) /* don't compress */
299 { 348 {
300 STRLEN len; 349 STRLEN len;
301 char *s = SvPV (sv, len); 350 char *s = SvPV (sv, len);
302 XPUSHs (sv_2mortal (newSVpvn (s, len))); 351 XPUSHs (sv_2mortal (newSVpvn (s, len)));
303 } 352 }
304 else /* don't compress */ 353 else /* don't compress */
305 XPUSHs (sv_2mortal (newSVsv (sv))); 354 XPUSHs (sv_2mortal (newSVsv (sv)));
355}
306 356
307void 357void
308sthaw(sv) 358sthaw(sv)
309 SV * sv 359 SV * sv
310 PROTOTYPE: $ 360 PROTOTYPE: $
311 PPCODE: 361 PPCODE:
312 362{
363 STRLEN svlen;
313 int deref = 0; 364 int deref = 0;
314 365
315 SvGETMAGIC (sv); 366 SvGETMAGIC (sv);
316 if (SvPOK (sv) && IN_RANGE (SvPV_nolen (sv)[0], MAGIC_LO, MAGIC_HI)) 367 if (SvPOK (sv) && IN_RANGE (SvPVbyte (sv, svlen)[0], MAGIC_LO, MAGIC_HI))
317 { 368 {
369 redo:
370
318 switch (SvPVX (sv)[0]) 371 switch (SvPVX (sv)[0])
319 { 372 {
320 case MAGIC_undef: 373 case MAGIC_undef:
374 if (svlen <= 1)
321 XPUSHs (sv_2mortal (NEWSV (0, 0))); 375 XPUSHs (sv_2mortal (NEWSV (0, 0)));
376 else
377 {
378 if (SvPVX (sv)[1] == MAGIC_R_deref)
379 deref = 1;
380 else if (SvPVX (sv)[1] != MAGIC_R)
381 croak ("Compress::LZF::sthaw(): invalid data, maybe you need a newer version of Compress::LZF?");
382
383 sv_chop (sv, SvPVX (sv) + 2);
384
385 if (!storable_mstore)
386 {
387 PUTBACK;
388 need_storable ();
389 SPAGAIN;
390 }
391
392 PUSHMARK (SP);
393 XPUSHs (sv);
394 PUTBACK;
395
396 if (1 != call_sv ((SV *)storable_mretrieve, G_SCALAR))
397 croak ("%s didn't return a single scalar", SvPVbyte_nolen (serializer_mretrieve));
398
399 SPAGAIN;
400
401 if (deref)
402 SETs (sv_2mortal (SvREFCNT_inc (SvRV (TOPs))));
403 else
404 SETs (sv_2mortal (newSVsv (TOPs)));
405 }
322 break; 406 break;
323 407
324 case MAGIC_U: 408 case MAGIC_U:
325 XPUSHs (sv_2mortal (decompress_sv (sv, 0))); 409 XPUSHs (sv_2mortal (decompress_sv (sv, 0)));
326 break; 410 break;
… …
339 case MAGIC_CR: 423 case MAGIC_CR:
340 sv = sv_2mortal (decompress_sv (sv, 1)); /* mortal could be optimized */ 424 sv = sv_2mortal (decompress_sv (sv, 1)); /* mortal could be optimized */
341 if (deref) 425 if (deref)
342 if (SvPVX (sv)[0] == MAGIC_R_deref) 426 if (SvPVX (sv)[0] == MAGIC_R_deref)
343 SvPVX (sv)[0] = MAGIC_R; 427 SvPVX (sv)[0] = MAGIC_R;
344 else 428
345 croak ("Compress::LZF::sthaw(): invalid data, maybe you need a newer version of Compress::LZF?"); 429 goto redo;
346 430
347 case MAGIC_R: 431 case MAGIC_R:
348 handle_MAGIC_R: 432 handle_MAGIC_R:
349 if (!storable_mstore) 433 if (!storable_mstore)
434 {
435 PUTBACK;
350 need_storable (); 436 need_storable ();
437 SPAGAIN;
438 }
351 439
352 PUSHMARK (SP); 440 PUSHMARK (SP);
353 XPUSHs (sv); 441 XPUSHs (sv);
354 PUTBACK; 442 PUTBACK;
355 443
356 if (1 != call_sv ((SV *)storable_mretrieve, G_SCALAR)) 444 if (1 != call_sv ((SV *)storable_mretrieve, G_SCALAR))
357 croak ("Storable::mstore didn't return a single scalar"); 445 croak ("%s didn't return a single scalar", SvPVbyte_nolen (serializer_mretrieve));
358 446
359 SPAGAIN; 447 SPAGAIN;
360 448
361 if (deref) 449 if (deref)
362 { 450 {
… …
374 croak ("Compress::LZF::sthaw(): invalid data, maybe you need a newer version of Compress::LZF?"); 462 croak ("Compress::LZF::sthaw(): invalid data, maybe you need a newer version of Compress::LZF?");
375 } 463 }
376 } 464 }
377 else 465 else
378 XPUSHs (sv_2mortal (newSVsv (sv))); 466 XPUSHs (sv_2mortal (newSVsv (sv)));
467}
379 468

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines