[FFmpeg-devel] Patch to fix qtrle endian issue with rgb555 pixel format

[Mo DeJong]

Michael, It is not clear that what you suggest is the best thing to
do. What of the command line argument? Should the user be expected to
pass "-pix_fmt rgb555be" instead of "-pix_fmt rgb555" in a case where
the file format is always written in big endian? Why should the user
need to indicate the endian format as well as how the pixels are
packed?

Mo DeJong

On Tue, Mar 2, 2010 at 2:07 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Mon, Mar 01, 2010 at 10:57:11PM -0800, Mo DeJong wrote:
>> As far as I could tell, there is no support in libswscale for
>> converting a rgb555le to rgb555be, and it is not clear how to add that
>> support in a generic way. Correct me if I am wrong here, but it seems
>> to me the -pix_fmt passed on the command line should be "-pix_fmt
>> rgb555" and not "-pix_fmt rgb555be" which would be required if
>> PIX_FMT_RGB555BE was put into the pix_fmts for this codec. My patch
>> makes the minimal change needed to get the quicktime encoder working
>> the way it seems it should be, please advise if there is a clearly
>> better approach.
>
> fix swscale, if swscale is lacking support.
>
> [...]
> --
> Michael ? ? GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> There will always be a question for which you do not know the correct awnser.
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.9 (GNU/Linux)
>
> iD8DBQFLjONjYR7HhwQLD6sRAqlbAJ9EuW/2fsQP5MttmXbSHyGz8uE3pgCfVPZC
> jm1DJzMSn93C4RXq8o46asU=
> =76qJ
> -----END PGP SIGNATURE-----
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>