[05/11] windowscodecs: Add support for more types of IFD fields. Resend.

Alexandre Julliard julliard at winehq.org
Thu Jun 14 12:34:47 CDT 2012


Dmitry Timoshkov <dmitry at baikal.ru> writes:

> @@ -650,30 +672,43 @@ static HRESULT load_IFD_entry(IStream *input, const struct IFD_entry *entry,
>      SWAP_ULONG(count);
>      type = entry->type;
>      SWAP_USHORT(type);
> +    item->value.vt = tag_to_vt(type);
>      value = entry->value;
>      SWAP_ULONG(value);
>  
>      switch (type)
>      {
> +     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?

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list