On Tue, 11 Nov 2014 14:13:11 -0800, Joaquin M Lopez Munoz <joaquin@tid.es> wrote:
Mostafa <mostafa_working_away <at> yahoo.com> writes:
On Tue, 11 Nov 2014 12:30:17 -0800, Joaquin M Lopez Munoz <joaquin <at> tid.es> wrote:
https://github.com/boostorg/multi_index/ commit/e18ece1ec13a8050aa5e75a522b8833dc37df37c
Does this look any better?
Yes, thanks! I just have two nitpicks:
1) line 1076: I think the following: "sequenced index iterator's value_type is <i>constant</i>" is a little more clear. IMO the previous wording gave the impression that only const_iteratorS are available.
Well, "constant iterator" is standardese for precisely what it's meant.
Ok, I wasn't aware of that. Maybe it's just me, but I've always found that term to be ambiguous. Does it mean "cntr::iterator const" or "cntr::const_iterator"?