[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: winedump: Add initial support for handling EMF+ records
From:       Piotr Caban <piotr.caban () gmail ! com>
Date:       2017-06-30 15:50:59
Message-ID: 070fca6b-203e-9634-5b91-3c602ad820ed () gmail ! com
[Download RAW message or body]

On 06/30/17 17:50, Dmitry Timoshkov wrote:
> Piotr Caban <piotr@codeweavers.com> wrote:
> 
> > #define EMRCASE(x) case x: printf("%-20s %08x\n", #x, length); break
> > +#define EMRPLUSCASE(x) case x: printf("    %-20s %04x %08x\n", #x, \
> > header->Flags, header->Size); break;
> 
> ';' at the end is redundant. EMRCASE macro intentionally omits it.
> 
> > +        /* Hand EMF+ records */
> > +        if (length >= 16 && !memcmp((char*)PRD(offset + 12, sizeof(unsigned \
> > int)), "EMF+", 4))
> 
> A typo in the comment.
Thanks, I'll send a fixed version.


[prev in list] [next in list] [prev in thread] [next in thread]