<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><span style="font-family:arial,sans-serif">On Thu, Apr 7, 2016 at 7:25 PM, Eric Sandeen </span><span dir="ltr" style="font-family:arial,sans-serif"><<a href="mailto:sandeen@sandeen.net" target="_blank">sandeen@sandeen.net</a>></span><span style="font-family:arial,sans-serif"> wrote:</span><br></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5"><br><br>
</div></div>"Many options allow for an optional argument of 0 or 1, ..."<br>
<span class=""><br>
> +disable or enable the functionality, in a forward-compatible syntax.<br>
<br>
</span>What does "forward-compatible syntax" mean?  I'm not sure that clarifies<br>
anything for the reader.<br>
<br></blockquote><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">​Yeah, I should reformulate it, I think. The meaning is that it won't matter what the defaults are now, or will be in the future. E.g., if you had a script creating a fs without crc before, when it was disabled by default, and we changed the default, you are now creating with the crc. But if you give it -m crc=0, then no matter what the default is, you have it always disabled.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">How about changing the line to "Boolean options allows for optional argument of value 0 or 1, to explicitly disable or enable the functionality," and dropping the forward-compatible part?</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Otherwise this looks ok to me; Dave explained that it is intentional to<br>
make every single option accept a value, whether it is now<br>
boolean or a numeric value, so there is no such thing as a bare "--flag"<br>
anymore; such flags are always "--flag [0|1]" now, right?<br></blockquote><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline"><br></div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">​For options inside of -m, -d and such, yes. Top-level flags, that is -f, -q, -N, -K and -V, are still only flags, but these don't change the FS attributes. They are something different from the other. Still, I wonder whether they should accept [0|1] too...</div><br></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​</div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<span class=""><br>
> @@ -312,10 +315,12 @@ and<br>
>  .B swidth<br>
>  values.<br>
>  .TP<br>
> -.BI noalign<br>
> -This option disables automatic geometry detection and creates the filesystem<br>
> +.BI noalign[= value ]<br>
> +If the<br>
> +.I value<br>
> +is 1, this option disables automatic geometry detection and creates the filesystem<br>
>  without stripe geometry alignment even if the underlying storage device provides<br>
> -this information.<br>
> +this information. 0 disables this option, that is, enables automatic detection.<br>
>  .RE<br>
>  .TP<br>
>  .B \-f<br>
<br>
</span>The realtime section of the manpage needs this same fixup now too, I guess.<br>
<br></blockquote><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Done​</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Thanks,<br>
-Eric<br>
<div class=""><div class="h5"><br>
_______________________________________________<br>
xfs mailing list<br>
<a href="mailto:xfs@oss.sgi.com">xfs@oss.sgi.com</a><br>
<a href="http://oss.sgi.com/mailman/listinfo/xfs" rel="noreferrer" target="_blank">http://oss.sgi.com/mailman/listinfo/xfs</a><br>
</div></div></blockquote></div><br><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">​Thanks,</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Jan​</div><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Jan Tulak<br></div><a href="mailto:jtulak@redhat.com" target="_blank">jtulak@redhat.com</a> / <a href="mailto:jan@tulak.me" target="_blank">jan@tulak.me</a></div></div></div></div>
</div></div>