11 Nov
2014
11 Nov
'14
10:29 p.m.
On Tue, 11 Nov 2014 14:13:11 -0800, Joaquin M Lopez Munoz
Mostafa
writes: On Tue, 11 Nov 2014 12:30:17 -0800, Joaquin M Lopez Munoz
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"?