[vlc-devel] [PATCH 00/18] Move decoder output format cleaning in the decoder (v1)

Steve Lhomme robux4 at ycbcr.xyz
Mon Oct 28 15:19:13 CET 2019


On 2019-10-28 13:41, Steve Lhomme wrote:
> As there are many video decoder "owners" it makes more sense to do the cleanup
> usually done for the video output in the decoder itself, especially now that
> it handles its own picture allocations.
> 
> We also benefit from all the work done on the display in these other owners.
> 
> It also allows removing some update_format callbacks that we just cleaning

s/we/were/

> the decoder output format.
> 
> These patches of this set can be found in this branch:
> https://code.videolan.org/robUx4/vlc/tree/push/merge/56
> 
> Steve Lhomme (18):
>    decoder: do the output chroma fixup in decoder_UpdateVideoOutput()
>    image: don't do the chroma fixup in update_format callback
>    thumbnailer: don't do the chroma fixup in update_format callback
>    transcode: don't do the chroma fixup in update_format callback
>    sdi: don't do the chroma fixup in update_format callback
>    mosaic_bridge: don't do the chroma fixup in update_format callback
>    decoder: remove redundant chroma fix in FixDisplayFormat
>    decoder: allow empty update_format callbacks in decoder owners
>    image: remove empty update_format callback
>    thumbnailer: remove empty video_update_format callback
>    decoder: factorize the visible size fixup during decoder format update
>    mosaic_bridge: don't do the visible size fixup in update_format
>      callback
>    decoder: factorize the SAR fixup during decoder format update
>    mosaic_bridge: don't do the SAR fixup in update_format callback
>    mosaic_bridge: remove internal video_format equivalent to the decoder
>      output
>    decoder: factorize the output colorimetry fixup
>    decoder: factorize the output YUV subsampling constraints fixup
>    decoder: no need for special cleaning of the decoder format to create
>      the display
> 
>   modules/stream_out/mosaic_bridge.c   | 34 +-----------
>   modules/stream_out/sdi/SDIStream.cpp |  3 -
>   modules/stream_out/transcode/video.c |  3 -
>   src/input/decoder.c                  | 82 ++--------------------------
>   src/input/decoder_helpers.c          | 64 +++++++++++++++++++++-
>   src/misc/image.c                     |  8 ---
>   6 files changed, 67 insertions(+), 127 deletions(-)
> 
> -- 
> 2.17.1
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
> 


More information about the vlc-devel mailing list