Re: Patches for Wine on 256 colour display and sparc

[Tony Lambregts]

Gregg Mattinson wrote:
> This patch is against the wine-20020509 snapshot.
>
> include/winnt.h:
> - Fixed the LARGE_INTEGER and ULARGE_INTEGER unions for big endian.
>
> server/trace.c:
> - Removed crash from dump_uints
>
> dlls/ntdll/time.c:
> - Removed alignment problem in RtlSecondsSince19?0ToTime
>
> graphics/x11drv/palette.c:
> - Icon colours on 256 colour display were wrong.
> - Caused by using 0 in X11DRV_PALETTE_XPixelToPalette to represent free colour
> - index 0 originally mapped to pixel 1
> - X11DRV_PALETTE_FillDefaultColors mapped index 50 to pixel 1, and hence pixel
>
> 1 was now mapped to 50 instead of 0.
> - Replaced 0's with -1's
>
> Gregg Mattinson
> Co-op Developer
> Sun Microsystems of Canada

[snip]
> *** server/,trace.c     Thu May 23 10:42:47 2002
> --- server/trace.c      Wed May 15 13:45:54 2002
>
> *** 47,57 ****
>  static void dump_uints( const int *ptr, int len )
>  {
>      fputc( '{', stderr );
> !     while (len > 0)
>      {
> !       fprintf( stderr, "%08x", *ptr++ );
> !       if (--len) fputc( ',', stderr );
>      }
>      fputc( '}', stderr );
>  }
>
> --- 47,64 ----
>  static void dump_uints( const int *ptr, int len )
>  {
>      fputc( '{', stderr );
> !     if (!ptr)
>      {
> !         fprintf( stderr, "dump_uints error: ptr is NULL" );

^
[snip]

This patch is not in diff -u format. Please resubmit using diff -u. I 
recomend that you break the patchs up, one patch for each file or 
related .c and .h. Please look at the following.

http://www.winehq.com/Docs/wine-devel/patches.shtml

Tony Lambregts