Re: PATCH: msrel32 new compiler

[Dmitry Timoshkov]

"Marcus Meissner" <marcus(a)jet.franken.de> wrote:
> --- dlls/msrle32/msrle32.c 27 Feb 2004 21:51:13 -0000 1.8
> +++ dlls/msrle32/msrle32.c 16 Jul 2004 12:36:10 -0000
> @@ -572,7 +572,8 @@
>
> /* add EOL -- end of line */
>        lpbiOut->biSizeImage += 2;
> -      *((LPWORD)lpOut)++ = 0;
> +      *(LPWORD)lpOut = 0;
> +      lpOut += sizeof(DWORD);

Shouldn't this one and others be lpOut += sizeof(WORD) and not DWORD?

-- 
Dmitry.

PATCH: msrel32 new compiler

[Marcus Meissner]

On Fri, Jul 16, 2004 at 10:33:00PM +0900, Dmitry Timoshkov wrote:
> "Marcus Meissner" <marcus(a)jet.franken.de> wrote:
> > --- dlls/msrle32/msrle32.c 27 Feb 2004 21:51:13 -0000 1.8
> > +++ dlls/msrle32/msrle32.c 16 Jul 2004 12:36:10 -0000
> > @@ -572,7 +572,8 @@
> >
> > /* add EOL -- end of line */
> >        lpbiOut->biSizeImage += 2;
> > -      *((LPWORD)lpOut)++ = 0;
> > +      *(LPWORD)lpOut = 0;
> > +      lpOut += sizeof(DWORD);
>
> Shouldn't this one and others be lpOut += sizeof(WORD) and not DWORD?

Thanks for spotting it!

I see that Alexandre already fixed with his commit.

Ciao, Marcus