[FFmpeg-devel] [PATCH]Improve rgb -> gray16 conversion

Michael Niedermayer michaelni at gmx.at
Tue Mar 18 22:14:48 CET 2014


On Tue, Mar 18, 2014 at 03:19:18AM +0100, Carl Eugen Hoyos wrote:
> On Sunday 16 March 2014 05:47:59 am Michael Niedermayer wrote:
> > On Thu, Mar 13, 2014 at 04:48:47PM +0000, Carl Eugen Hoyos wrote:
> > > Carl Eugen Hoyos <cehoyos <at> ag.or.at> writes:
> > > > Gray16 currently assumes MPEG-range, this is unexpected
> > > > for most users. Attached patch improves the dynamics of
> > > > the gray16 output here, addresses ticket #3422.
> 
> > breaks fate (fate checksum update missing at least / output of
> > changes should be checked, for at least a subset of the changes)
> 
> The pam and png output files look visually similar, in both cases the 
> dynamics increase to 0x0 -> 0xfffb.
> 
> New patch attached, Carl Eugen

>  libswscale/utils.c                       |    2 ++
>  tests/ref/fate/filter-pixdesc            |    4 ++--
>  tests/ref/fate/filter-pixfmts-copy       |    4 ++--
>  tests/ref/fate/filter-pixfmts-crop       |    4 ++--
>  tests/ref/fate/filter-pixfmts-field      |    4 ++--
>  tests/ref/fate/filter-pixfmts-fieldorder |    4 ++--
>  tests/ref/fate/filter-pixfmts-hflip      |    4 ++--
>  tests/ref/fate/filter-pixfmts-il         |    4 ++--
>  tests/ref/fate/filter-pixfmts-null       |    4 ++--
>  tests/ref/fate/filter-pixfmts-scale      |    4 ++--
>  tests/ref/fate/filter-pixfmts-vflip      |    4 ++--
>  tests/ref/lavf/pam                       |    4 ++--
>  tests/ref/lavf/png                       |    6 +++---
>  13 files changed, 27 insertions(+), 25 deletions(-)
> 8d9582111f9ce190041bdb5f56d7e62d59fbf34b  patchgray16.diff
> diff --git a/libswscale/utils.c b/libswscale/utils.c
> index 4e73853..f2e4167 100644

applied

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140318/95b442bf/attachment.asc>


More information about the ffmpeg-devel mailing list