<div dir="ltr">Thanks, Sean. Looks like we should rename all the getXXX{Begin,End} functions in this file at once. Is there any concern on doing that? If none, I'll do that in a different patch.</div><div class="gmail_extra">

<br><br><div class="gmail_quote">On Thu, Sep 26, 2013 at 4:38 PM, Sean Silva <span dir="ltr"><<a href="mailto:silvas@purdue.edu" target="_blank">silvas@purdue.edu</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"><div>We actually have this in the coding standard:</div><a href="http://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly" target="_blank">http://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly</a> :<br>


<div><br></div><div><span style="line-height:21px;font-size:14px;font-family:'Lucida Grande','Lucida Sans Unicode',Geneva,Verdana,sans-serif">As an exception, classes that mimic STL classes can have member names in STL’s style of lower-case words separated by underscores (e.g. </span><tt style="line-height:21px;font-size:0.95em;font-family:Consolas,'Deja Vu Sans Mono','Bitstream Vera Sans Mono',monospace"><span>begin()</span></tt><span style="line-height:21px;font-size:14px;font-family:'Lucida Grande','Lucida Sans Unicode',Geneva,Verdana,sans-serif">,</span><tt style="line-height:21px;font-size:0.95em;font-family:Consolas,'Deja Vu Sans Mono','Bitstream Vera Sans Mono',monospace"><span>push_back()</span></tt><span style="line-height:21px;font-size:14px;font-family:'Lucida Grande','Lucida Sans Unicode',Geneva,Verdana,sans-serif">, and </span><tt style="line-height:21px;font-size:0.95em;font-family:Consolas,'Deja Vu Sans Mono','Bitstream Vera Sans Mono',monospace"><span>empty()</span></tt><span style="line-height:21px;font-size:14px;font-family:'Lucida Grande','Lucida Sans Unicode',Geneva,Verdana,sans-serif">). Classes that provide multiple iterators should add a singular prefix to </span><tt style="line-height:21px;font-size:0.95em;font-family:Consolas,'Deja Vu Sans Mono','Bitstream Vera Sans Mono',monospace"><span>begin()</span></tt><span style="line-height:21px;font-size:14px;font-family:'Lucida Grande','Lucida Sans Unicode',Geneva,Verdana,sans-serif"> and </span><tt style="line-height:21px;font-size:0.95em;font-family:Consolas,'Deja Vu Sans Mono','Bitstream Vera Sans Mono',monospace"><span>end()</span></tt><span style="line-height:21px;font-size:14px;font-family:'Lucida Grande','Lucida Sans Unicode',Geneva,Verdana,sans-serif"> (e.g. </span><tt style="line-height:21px;font-size:0.95em;font-family:Consolas,'Deja Vu Sans Mono','Bitstream Vera Sans Mono',monospace"><span>global_begin()</span></tt><span style="line-height:21px;font-size:14px;font-family:'Lucida Grande','Lucida Sans Unicode',Geneva,Verdana,sans-serif"> and </span><tt style="line-height:21px;font-size:0.95em;font-family:Consolas,'Deja Vu Sans Mono','Bitstream Vera Sans Mono',monospace"><span>use_begin()</span></tt><span style="line-height:21px;font-size:14px;font-family:'Lucida Grande','Lucida Sans Unicode',Geneva,Verdana,sans-serif">).</span><br>


</div><div><span style="line-height:21px;font-size:14px;font-family:'Lucida Grande','Lucida Sans Unicode',Geneva,Verdana,sans-serif"><br></span></div>I.e. import_directory_begin (it returns an import_directory_iterator so I guess that import_directory_* is the prefix)<div>


<br></div><div>-- Sean Silva<br><div><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On Thu, Sep 26, 2013 at 7:21 PM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br>


</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div><br>
<br>
================<br>
Comment at: include/llvm/Object/COFF.h:298-299<br>
@@ -289,1 +297,4 @@<br>
<br>
+  import_directory_iterator getImportDirectoryBegin() const;<br>
+  import_directory_iterator getImportDirectoryEnd() const;<br>
+<br>
----------------<br>
</div><div>Michael Spencer wrote:<br>
> LLVM uses <type>_{begin,end} for these.<br>
</div>Looks like _{begin,end} is used only when the prefix is not in camel case.  We already have two different styles of naming (getSectionRelBegin and begin_symbols, for examples). I don't have a preference on that, so if you prefer one over the others, I'll rename them.<br>



</div></div><div><div><div><div class="h5"><br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1719" target="_blank">http://llvm-reviews.chandlerc.com/D1719</a><br>
<br>
BRANCH<br>
  readobj<br>
<br>
ARCANIST PROJECT<br>
  llvm<br></div></div>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>
</blockquote></div><br></div>