<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
  <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
  <meta http-equiv="Content-Style-Type" content="text/css" />
  <meta name="generator" content="pandoc" />
  <title></title>
  <style type="text/css">code{white-space: pre;}</style>
</head>
<body>
<p>Hi Rémi,</p>
<p>On 2016-10-14 21:15, Rémi Denis-Courmont wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Another way of fixing the described issues is to only apply
 `VLC_VAR_SETMINMAX` if the associated module configuration for int and
 float variables, respectivally, differ from `[0,0]` and `[NaN,NaN]`.</code></pre>
</blockquote>
<pre><code> Sorry but this is just completely irrelevant to the problem. The matter is 
 what the default range should be, the historical range that is relied upon in 
 pre-existing code paths, or the whole range.

 There are no problems with how limits should be represented internally.</code></pre>
</blockquote>
<p>I agree with you on the fact that the question is what the default range should be, and from what Hugo wrote I concluded that one alternative would (perhaps) be to not have any range at all unless such has been explictly specified; hence the patch.</p>
<p>That way the old behavior for modules that do not specify a range for a certain variable is retained, while we still clamp values where an explicit range has been set.</p>
<p>Though I certainly trust that you guys know better given that you have far more experience with the codebase.. I just thought I’d drop my 2 cents.</p>
<p>Thanks for the reply and elaboration!</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> -- 
 Rémi Denis-Courmont
 Nonsponsored VLC developer
 http://www.remlab.net/CV.pdf</code></pre>
</blockquote>
</body>
</html>