r203179 - [C++11] Replacing iterators redecls_begin() and redecls_end() with iterator_range redecls(). Updating all of the usages of the iterators with range-based for loops, which allows the begin/end forms to be removed entirely.

Aaron Ballman aaron at aaronballman.com
Thu Mar 6 16:12:02 PST 2014


On Thu, Mar 6, 2014 at 7:10 PM, Tobias Grosser <tobias at grosser.es> wrote:
> On 03/07/2014 12:45 AM, Aaron Ballman wrote:
>>
>> Author: aaronballman
>> Date: Thu Mar  6 17:45:36 2014
>> New Revision: 203179
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=203179&view=rev
>> Log:
>> [C++11] Replacing iterators redecls_begin() and redecls_end() with
>> iterator_range redecls(). Updating all of the usages of the iterators with
>> range-based for loops, which allows the begin/end forms to be removed
>> entirely.
>
>
> Hi Aaron,
>
> I wonder if you could use 'auto const &' in some of these cases?
>
> http://llvm.org/docs/CodingStandards.html#beware-unnecessary-copies-with-auto

I'll look into it once I've gotten the build bots green again. Thank
you for the reminder. :-)

~Aaron



More information about the cfe-commits mailing list