[vlc-devel] [PATCH] Phosphor deinterlacer, reworked

Jean-Baptiste Kempf jb at videolan.org
Wed Feb 23 12:22:36 CET 2011

On Wed, Feb 23, 2011 at 12:39:35PM +0200, Juha Jeronen wrote :
> I'm not sure I understand? Obviously, it's currently a compile-time
> option... did you mean, why propose it as an option at all? Or why not
> propose it as a runtime option?

Why not proposing as a runtime option.

> For the second one - any pointers on how to do that? Currently none of
> the deinterlacers have any runtime options, so there was no example. (I
> think that's pretty neat - since they have nothing to configure, they
> have to be designed carefully, and also are easier to use. The "no
> options" approach is not always reasonable, but for deinterlacers I
> think it should work - at least for the most part, hence the current
> confusion.)

After the vlc_module_begin () declaration, use add_bool( ... ), like in
Then use it in the code with var_Inherit...

Yes, no option is better, sure. But here, it avoids recompilation to get
a new mode. And if it is an obscure option, it will not  bother users,
as long as the default is the sane one.

Best Regards,

Jean-Baptiste Kempf
+33 672 704 734

More information about the vlc-devel mailing list