On Sat, May 25, 2002 at 04:35:18PM +0100, Nick Glencross wrote: > here's a small fix to a printf which I sometimes trigger. Looks like a > ')' and '\n' went missing. Feel free to adjust it to meets coding standards.
> Index: packfile.c > =================================================================== > RCS file: /cvs/public/parrot/packfile.c,v > retrieving revision 1.48 > diff -c -r1.48 packfile.c > *** packfile.c 20 May 2002 01:28:32 -0000 1.48 > --- packfile.c 25 May 2002 15:11:53 -0000 > *************** > *** 321,327 **** > */ > if(self->need_wordsize) { > fprintf(stderr, "PackFile_unpack: Unimplemented wordsize transform.\n"); > ! fprintf(stderr, "File has wordsize: %d (native is %dn", header->wordsize, > sizeof(opcode_t)); > return 0; > } > --- 321,328 ---- > */ > if(self->need_wordsize) { > fprintf(stderr, "PackFile_unpack: Unimplemented wordsize transform.\n"); > ! fprintf(stderr, "File has wordsize: %d (native is %d)\n", > ! header->wordsize, > sizeof(opcode_t)); > return 0; > } I've just found this in my mailbox. Presumably that means that nobody applied it. However, I can't find the bit of packfile that used to have this, so I presume that the unimplemented wordsize transforms were implemented at some point in the past 9 months. :-) Thanks for the patch. Sorry we never got to use it. Nicholas Clark