<div dir="ltr">I think they need fixing as well. I'll do that.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 8, 2017 at 2:39 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif"><span class="">
<p>​> ​<span style="color:rgb(33,33,33);font-size:12pt">Please follow the convention -- if you have two options, -foo= and -foo, one is named as foo_eq in this file. </span></p>
<p><span style="color:rgb(33,33,33);font-size:12pt"><br>
</span></p>
</span><p>I afraid we are not consistent and probably don't have any convention here<br>
</p>
<p>When I did that patch I checked naming carefully I think my naming is OK.<br>
</p>
<p><br>
</p>
<p>See (just a quick look):<br>
</p>
<div>1) <span style="font-size:12pt">def entry: S<"entry">, MetaVarName<"<entry>">,</span></div>
<div>  HelpText<"Name of entry point symbol">;<br>
</div>
<div>def alias_entry_entry: J<"entry=">, Alias<entry>;<br>
</div>
<p><br>
</p>
<div>def fini: S<"fini">, MetaVarName<"<symbol>">,</div>
<div>  HelpText<"Specify a finalizer function">;<br>
</div>
<div>def alias_fini_fini: J<"fini=">, Alias<fini>;<br>
</div>
<div><br>
</div>
<div>
<div>2) def error_limit: S<"error-limit">,</div>
<div>  HelpText<"Maximum number of errors to emit before stopping (0 = no limit)">;</div>
</div>
<div>def alias_error_limit: J<"error-limit=">, Alias<error_limit>;<br>
</div>
<div><br>
</div>
<div>
<div>def Tbss: S<"Tbss">, HelpText<"Same as --section-start with .bss as the sectionname">;</div>
<div><span style="font-family:Calibri,Arial,Helvetica,sans-serif;font-size:16px;background-color:rgb(255,255,255)">def alias_Tbss: J<"Tbss=">, Alias<Tbss>;</span><br>
</div>
<div><br>
</div>
<div>def Tdata: S<"Tdata">, HelpText<"Same as --section-start with .data as the sectionname">;<br>
</div>
</div>
<div>def alias_Tdata: J<"Tdata=">, Alias<Tdata>;<br>
</div>
<p><br>
</p>
<div>3) def build_id: F<"build-id">, HelpText<"Generate build ID note">;<br>
</div>
<div>def build_id_eq: J<"build-id=">, HelpText<"Generate build ID note">;<br>
</div>
<div><br>
</div>
<div>It seems we at least have 3 types of naming in that case, my patch used one of them.<br>
</div>
<p>I can check all options and prepare a patch to fix all options at once to be consistent.<br>
</p><span class="">
<p><br>
</p>
<p><span style="color:rgb(33,33,33);font-size:12pt">> Please send me a patch review. Thanks.</span><span style="color:rgb(33,33,33);font-size:12pt">​</span></p>
</span><div style="color:rgb(33,33,33)">
<div>
<div dir="ltr">
<div class="gmail_extra">I will.<span class="HOEnZb"><font color="#888888"><br>
</font></span></div><span class="HOEnZb"><font color="#888888">
<div class="gmail_extra"><br>
<div class="gmail_quote">George.<br>
</div>
</div>
</font></span></div>
</div>
</div>
</div>

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