1216041 - add ranged iterator support to mozilla::{Array,RangedArray,EnumerationArray}

Status: RESOLVED FIXED

(Core :: MFBT, defect)

Description

[Cameron McCormack (:heycam)]

I want to use ranged for-loop syntax on an EnumeratedArray.  And I may as well add support to Array and RangedArray while I'm at it.

Comment 1

[Cameron McCormack (:heycam)]

Attachment: Add ranged iterator support to mozilla::{Array,RangedArray,EnumerationArray}.

Comment 2

[Nathan Froyd [:froydnj]]

Comment on attachment 8675513 [details] [diff] [review]
Add ranged iterator support to mozilla::{Array,RangedArray,EnumerationArray}.

Review of attachment 8675513 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with nitpicky consistency changes below.

::: mfbt/EnumeratedArray.h
@@ +77,5 @@
> +
> +  // Methods for range-based for loops.
> +  iterator begin() { return mArray.begin(); }
> +  const_iterator begin() const { return mArray.begin(); }
> +  const_iterator cbegin() const { return mArray.begin(); }

Nit: please make this cbegin() for consistency.  (Since you use the c* variants below.)

::: mfbt/EnumeratedRange.h
@@ +172,5 @@
>  
>  // Create a range to iterate from aBegin to aEnd, exclusive.
> +//
> +// (Once we can rely on std::underlying_type, we can remove the IntType
> +// template parameter.)

Feel free to fold these changes to this file into bug 1216038.

::: mfbt/RangedArray.h
@@ +46,5 @@
> +
> +  // Methods for range-based for loops.
> +  iterator begin() { return mArr.begin(); }
> +  const_iterator begin() const { return mArr.begin(); }
> +  const_iterator cbegin() const { return mArr.begin(); }

Here too.

Comment 3

[Cameron McCormack (:heycam)]

(In reply to Nathan Froyd [:froydnj] from comment #2)
> > +  const_iterator cbegin() const { return mArray.begin(); }
> 
> Nit: please make this cbegin() for consistency.  (Since you use the c*
> variants below.)

Sorry, yes I noticed these locally but forgot to update the patch.

> ::: mfbt/EnumeratedRange.h
> @@ +172,5 @@
> >  
> >  // Create a range to iterate from aBegin to aEnd, exclusive.
> > +//
> > +// (Once we can rely on std::underlying_type, we can remove the IntType
> > +// template parameter.)
> 
> Feel free to fold these changes to this file into bug 1216038.

Will do.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c3e202bf0f25

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c3e202bf0f25

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c3e202bf0f25