<br><br><div class="gmail_quote">2010/11/12 Jonas Gehring <span dir="ltr"><<a href="mailto:jonas.gehring@boolsoft.org">jonas.gehring@boolsoft.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On 11/12/2010 12:55 AM, Jean-Baptiste Kempf wrote:<br>
> On Fri, Nov 12, 2010 at 12:51:11AM +0100, Jonas Gehring wrote :<br>
>> Thanks, I missed the comments on change_safe(). I moved the option<br>
>> definition outside of the change_safe() scope. The revised patch is<br>
>> attached.<br>
><br>
> I believe change_safe is per option.<br>
<br>
</div>Hm, could you suggest a more appropriate place for the option?<br></blockquote><div><br></div><div>I think J-B means that it should look something like:</div><div>add_integer( "http-caching", 4 * DEFAULT_PTS_DELAY / 1000, CACHING_TEXT, CACHING_LONGTEXT, true )</div>
<div>change_safe()</div><div>add_string( "http-referrer", NULL, NULL, NULL, false )</div><div>change_safe()</div><div>add_string( "http-user-agent", NULL, NULL, NULL, false )</div><div>change_safe() </div>
<div> </div><div>So your current patch turns http-caching non-safe since the http-referrer is inserted between two connected elements.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div><div></div><div class="h5">_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>