
<!DOCTYPE html>

<html>

<head>

<title></title>

<style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style>

</head>

<body><div><br></div>

<div><br></div>

<div><br></div>

<div>On Mon, Dec 3, 2018, at 14:31, Shaleen Jain wrote:<br></div>

<blockquote type="cite"><div>On Mon, 2018-12-03 at 12:10 +0100, Thomas Guillem wrote:<br></div>

<blockquote type="cite" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:2px;padding-left:1ex;"><pre>Hello,<br></pre><pre><br></pre><pre><br></pre><pre>Could you also refactor renderer module options, like "show-perf-warning", "audio-passthrough", "conversion-quality", and maybe some others.<br></pre><pre>You can check how it's done in modules/video_output/opengl/display.c, with add_glopts().<br></pre><pre><br></pre><pre>On Wed, Nov 28, 2018, at 16:44, Shaleen Jain wrote:<br></pre></blockquote><div><br></div>

<div>Do you mean to use a define for the common options? besides the three that you have already mentioned "ip" and "port" are the common<br></div>

<div>options as they are required by vlc_renderer_item_new but they have different defaults for chromecast and dlna each (for e.g chromecast has port 8009 and dlna has port 7070).<br></div>

<div>So I'm not sure if it'll be wise to make them (ip and port) common for each renderer.<br></div>

</blockquote><div><br></div>

<div>You're right, it won't.<br></div>

<div>Each modules should have their own options name:<br></div>

<div>"chromecast-something", "upnp-renderer-something" and not a common "renderer-something". This will allow users to setup different options for chromecast and upnp.<br></div>

<div><br></div>

<div>That said, modules can have different options name but using common options via some includes.<br></div>

<div>For ip and port, you could specifiy the default as a MACRO parameter.<br></div>

<div><br></div>

<blockquote type="cite"><div><br></div>

<div><div><span></span><br></div>

<pre><span>-- </span><br></pre><div style="width:71ch;"><span>Regards,</span><br></div>

<div style="width:71ch;"><span>Shaleen Jain</span><br></div>

<div><span></span><br></div>

</div>

<div><u>_______________________________________________</u><br></div>

<div>vlc-devel mailing list<br></div>

<div>To unsubscribe or modify your subscription options:<br></div>

<div><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div>

</blockquote><div><br></div>

</body>

</html>