[vlc-devel] [PATCH] Soft field repeat support for deinterlacer
jb at videolan.org
Fri Feb 11 18:39:02 CET 2011
On Thu, Feb 10, 2011 at 01:55:29PM +0200, Juha Jeronen wrote :
> Based on what I currently know about VLC's deinterlacer, I would
> tentatively suggest the following split:
> - Consider deinterlace.c as the main source file for the deinterlacer
> subsystem. Keep Deinterlace(), Open(), Flush(), Close(),
> SetFilterMethod(), GetOutputFormat() etc. general stuff in that one.
> - filter_sys_t: this definition needs to be seen by all parts of the
> deinterlacer. Move to header?
> - Simple rendering algorithms with just one function, or maybe two
> functions at most: make a new file that gathers these.
Yes. What the core deinterlace.c was.
> - More complex rendering algorithms with lots of helper functions (e.g.
> X) or private data (e.g. the upcoming IVTC): one file per algorithm.
Like yadif, yes.
> - Maybe one file for common helper functions? I have such a case coming
> up: both IVTC and Phosphor need ComposeFrame().
Maybe this is overkill.
However, I would propose to split AFTER the commits of the IVTC (and
And I would also suggest that you resend a clean batch for the IVTC and
+33 672 704 734
More information about the vlc-devel