On Wed, Nov 05, 2014 at 02:13:29AM +0100, wm4 wrote: > The header pixfmt.h contains the following comment in the pixel format > list doxygen: > > * @note > * Make sure that all newly added big-endian formats have (pix_fmt & 1) == 1 > * and that all newly added little-endian formats have (pix_fmt & 1) == 0. > * This allows simpler detection of big vs little-endian. >
> This is currently broken for the following formats: nv20le/be gbrap16le/be yes > > Obviously, this can't be fixed until the next ABI bump. But more can you post a patch which does make sure it gets fixed then? and or that documentation corrected [...] > > Also, we should discuss whether swapped-endian formats should even > exist. From what I can see, only the raw decoders/encoders make > effective use of them. Wouldn't it be simpler to let the > decoders/encoders do the byte-swapping? they are usefull to avoid doubble swaping if both in and out are non native endian. also they are usefull for the user API of libswscale, which mey be used by end user applications which have such non native formats [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Republics decline into democracies and democracies degenerate into despotisms. -- Aristotle
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
