On Thu, Aug 22, 2019 at 00:28:28 +0530, Shivam wrote:
> >> + dicom->height = *(uint16_t *)bytes;
> > does this work on big endian ?
> > maybe you where looking for AV_RL16()
>
> Big endian DICOM files are retired and no longer supported by the standard.

What Michael means: What happens if you use this to decode a (little
endian) DICOM file on big endian hardware?

In other words: "dicom->height = *(uint16_t *)bytes" will map the bytes
into the big endian int in the wrong order, resulting in an incorrect
integer value. AV_RL16() (e.g.) handles this properly for you.

@Michael, do you have some short instructions on how to set up a big
endian system like your MIPS+qemu? Thanks.

Moritz
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to