[MPlayer-dev-eng] [PATCH]Support 64bit tiff via opengl

[Reimar Döffinger]

On Sat, Jan 14, 2012 at 02:14:01PM +0000, Carl Eugen Hoyos wrote:
> Reimar Döffinger <Reimar.Doeffinger <at> gmx.de> writes:
> 
> > On Sat, Jan 14, 2012 at 01:47:30PM +0100, Carl Eugen Hoyos wrote:
> > > Index: libmpcodecs/mp_image.c
> > > ===================================================================
> > > --- libmpcodecs/mp_image.c	(revision 34561)
> > > +++ libmpcodecs/mp_image.c	(working copy)
> > > @@ -107,7 +107,9 @@
> > >      }
> > >      mpi->num_planes=1;
> > >      if (IMGFMT_IS_RGB(out_fmt)) {
> > > +        if (out_fmt == IMGFMT_RGBA64LE || out_fmt == IMGFMT_RGBA64BE) {
> > > +            mpi->bpp = 64;
> > > -        if (IMGFMT_RGB_DEPTH(out_fmt) < 8 && !(out_fmt&128))
> > > +        } else if (IMGFMT_RGB_DEPTH(out_fmt) < 8 && !(out_fmt&128))
> > 
> > If you want that, move the BE signalling to | 128 for all formats and
> > change IMGFMT_RGB_DEPTH to use & 0x7F
> 
> I think I do not understand: "|64" is used to signal "non-native-endian" iiuc,

For RGB yes.

> BE is already signalled with "|128" for all formats, or am I wrong?

No, only for RGB48, all others use either | 64 (though to indicate
swapped endianness, not big-endian to be specific) or a reordered tag.

> 
> Since I don't know where "non-native-endian" is used, I cannot change it...

Hm, at least in vo_x11.c, but it's used wrongly anyway, only works
for RGB32 that way.