This is the mail archive of the libstdc++@gcc.gnu.org mailing list for the libstdc++ project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch] String: move and centralize exp growth


On Wed, 2004-01-28 at 03:46, Paolo Carlini wrote:
> Hi,
> 
> this is the idea implemented! Honestly, I'm very happy with it: now all
> the "magic numbers" are only in one place (instead of three!), and the
> encapsulation is much better.
> 
> I have tested the patch on x86-linux, I would like to do some additional
> tests and, if everything goes well, commit to mainline (only, of course!)
> very soon.

I noticed this in the patch:

*************** namespace std
*** 157,165 ****
  
        const size_type __dnew =
static_cast<size_type>(std::distance(__beg, __end));

Is it really intended, or would you rather use difference_type on the
lhs of the assignment?




-- 
	-Dhruv Matani.
http://www.geocities.com/dhruvbird/




Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]