Bug in monochrome bitmaps

[Oldrich Jedlicka]

Hi all,

I found a bug in monochrome bitmaps with inverted colors (color[0]=0xFFFFFF, 
color[1]=0x000000). It is problem in translation from XImage into DIB created 
by CreateDIBSection.

Description can be found in my mail from 08 Oct 2005 and 03 Nov 2005. Patch is 
also attached to this mail.

I've got no response and CVS is untouched. So what is wrong with my patch? 
People from cedega team at least said 'thank you', there is nothing :-(

Thanks for reply,

Oldrich Jedlicka.

[Willie Sippel]

Am Mittwoch, 23. November 2005 09:41 schrieb Oldrich Jedlicka:
> Hi all,
>
> I found a bug in monochrome bitmaps with inverted colors
> (color[0]=0xFFFFFF, color[1]=0x000000). It is problem in translation from
> XImage into DIB created by CreateDIBSection.
>
> Description can be found in my mail from 08 Oct 2005 and 03 Nov 2005. Patch
> is also attached to this mail.
>
> I've got no response and CVS is untouched. So what is wrong with my patch?
> People from cedega team at least said 'thank you', there is nothing :-(

Well, it seems that Wine doesn't even compile with your patch applied (just 
tried with gcc 4.0.2) - I guess that's at least one of the reasons why it 
wasn't applied...?

See:
dib.c: In function 'X11DRV_DIB_DoCopyDIBSection':
dib.c:4186: error: invalid operands to binary ==
make[2]: *** [dib.o] Error 1
> Thanks for reply,
>
> Oldrich Jedlicka.

Ciao,
Willie

[Oldrich Jedlicka]

Thanks Willie,

ok, that's it. I didn't try to compile it after I sent the patch first time 
(it was working for me). I will fix it and resend a patch to wine-patches.

Cheers,

Oldrich.

Dne Wednesday 23 of November 2005 10:35 Willie Sippel napsal(a):
> Am Mittwoch, 23. November 2005 09:41 schrieb Oldrich Jedlicka:
> > Hi all,
> >
> > I found a bug in monochrome bitmaps with inverted colors
> > (color[0]=0xFFFFFF, color[1]=0x000000). It is problem in translation from
> > XImage into DIB created by CreateDIBSection.
> >
> > Description can be found in my mail from 08 Oct 2005 and 03 Nov 2005.
> > Patch is also attached to this mail.
> >
> > I've got no response and CVS is untouched. So what is wrong with my
> > patch? People from cedega team at least said 'thank you', there is
> > nothing :-(
>
> Well, it seems that Wine doesn't even compile with your patch applied (just
> tried with gcc 4.0.2) - I guess that's at least one of the reasons why it
> wasn't applied...?
>
> See:
> dib.c: In function 'X11DRV_DIB_DoCopyDIBSection':
> dib.c:4186: error: invalid operands to binary ==
> make[2]: *** [dib.o] Error 1
> > Thanks for reply,
> >
> > Oldrich Jedlicka.
>
> Ciao,
> Willie

[Oldrich Jedlicka]

Hi,
> See:
> dib.c: In function 'X11DRV_DIB_DoCopyDIBSection':
> dib.c:4186: error: invalid operands to binary ==
> make[2]: *** [dib.o] Error 1

just to be sure it compiles also on gcc 4. Here is the corrected patch.

Cheers,

Oldrich.

[Willie Sippel]

Am Mittwoch, 23. November 2005 12:08 schrieb Oldrich Jedlicka:
> Hi,
> > See:
> > dib.c: In function 'X11DRV_DIB_DoCopyDIBSection':
> > dib.c:4186: error: invalid operands to binary ==
> > make[2]: *** [dib.o] Error 1
>
> just to be sure it compiles also on gcc 4. Here is the corrected patch.

Works... :-)

[Oldrich Jedlicka]

Good, submitting to wine-patches.

Thanks :-)

Dne Wednesday 23 of November 2005 13:52 Willie Sippel napsal(a):
> Am Mittwoch, 23. November 2005 12:08 schrieb Oldrich Jedlicka:
> > Hi,
> > > See:
> > > dib.c: In function 'X11DRV_DIB_DoCopyDIBSection':
> > > dib.c:4186: error: invalid operands to binary ==
> > > make[2]: *** [dib.o] Error 1
> >
> > just to be sure it compiles also on gcc 4. Here is the corrected patch.
>
> Works... :-)