Alexandre Julliard <julli...@winehq.org> wrote:

> > +     case IFD_BYTE:
> > +     case IFD_SBYTE:
> > +        if (count == 1)
> > +        {
> > +            item->value.u.bVal = *(BYTE *)&value;
> > +            break;
> > +        }
> > +        FIXME("loading multiple byte fields is not implemented\n");
> > +        break;
> 
> This seems to contradict the endianness support. Are you really supposed
> to byte-swap arrays of bytes?

It doesn't matter at this point. I'm planning to add a test for endianness
handling in IFD reader, if this is proved to be wrong the fix will be trivial.
Please consider accepting the patch as is.

-- 
Dmitry.


Reply via email to