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.