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

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines