On Tue, Aug 14, 2012 at 11:56 AM, Julien Chaffraix <span dir="ltr">&lt;<a href="mailto:julien.chaffraix@gmail.com" target="_blank">julien.chaffraix@gmail.com</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">&gt; I don&#39;t think it&#39;s appropriate to add settings for CSS features that are under development,<br>
&gt; for a number of reasons:<br>
&gt;<br>
&gt; * If we did this for every feature, we&#39;d end up with hundreds of Settings.<br>
&gt; * Traditionally, Settings don&#39;t tend to get removed, resulting in an ever-accumulating number of Settings.<br>
<br>
</div>ENABLE has a slightly better track of record but I don&#39;t think we<br>
should push back on runtime flags just because of that.</blockquote><div><br></div><div>Having a runtime flag incurs runtime cost.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Having tons of #ifdef&#39;s is a lot more worrying from my perspective,</blockquote><div><br></div><div>Having a runtime flag is significantly worse in that it affects end users. Having a compile flag is a painful for developers but has absolutely zero cost for end users.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
&gt; * If your feature is protected by an ENABLE flag, vendors that want to ship release software can turn it off.<br>
<br>
</div>That&#39;s true, except that the original thread didn&#39;t mention any form<br>
of feature flag [1]. Nobody objected at the time and thus patches that<br>
implemented part of the spec arrived on bugzilla: prefixed but not<br>
protected by any flag.<br></blockquote><div><br></div><div>I had certainly assumed that this was done under a new build flag. If that were not the case, I expected relevant reviewers to r- those patches. Maybe this was a bad assumption to make.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This update isn&#39;t about bringing a discussion about ENABLE vs runtime,<br>
more about mentioning that there will be a new flag contrary to what<br>
was said previously.</blockquote><div><br></div><div>I strongly object to implementing this feature behind a setting. It should be developed under a build flag.</div><div><br></div><div>- Ryosuke</div><div><br></div></div>