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

Reply via email to