<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 31, 2013 at 2:35 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">> Probably none, but I don't see why we should truncate unless there's a good reason. This class is going to be 16 bytes on a 64 bit system anyway.<br>

<br>
</div>ok<br>
<div class="im"><br>
> ================<br>
> Comment at: include/llvm/Object/ELF.h:365<br>
> @@ -786,4 +364,3 @@<br>
><br>
> -  Elf_Rel_Iter endELFRel(const Elf_Shdr *sec) const {<br>
> -    return Elf_Rel_Iter(sec->sh_entsize, (const char *)<br>
> -                        (base() + sec->sh_offset + sec->sh_size));<br>
> +  Elf_Rel_Iter end_rel(const Elf_Shdr *sec) const {<br>
> +    return Elf_Rel_Iter(sec->sh_entsize,<br>
> ----------------<br>
> Rafael Ávila de Espíndola wrote:<br>
>> Having ELF in the name of this methods would be redundant, so I agree with dropping it, but why the switch away from CamelCase?<br>
> Because all the other iterator functions are like this.<br>
<br>
</div>This is one part of the codebase that is fairly inconsistent. We have<br>
alias_begin, beginSequence and begin_symbols. All of which don't<br>
follow the style :-(. Using beginFoo and endFoo still look to be the<br>
closest.<br></blockquote><div><br></div><div>Personally, the alias_begin style is the only convention that I (explicitly) remember ever seeing anywhere in the codebase to indicate STL-like semantics. All of the core LLVM classes use this style (Value::use_begin, Function::arg_begin, etc.); I don't see any reason to deviate.</div>
<div><br></div><div>-- Sean Silva </div></div></div></div>