Re: [boost] Re: String Algo bug.

From: Pavol Droba (droba_at_[hidden])
Date: 2004-07-30 02:36:14

Hi

On Thu, Jul 29, 2004 at 10:58:14PM +0200, Stefan Slapeta wrote:
> 
> An optimization: I realized later that it can be costly to evaluate the eof
> states. I would save them in a local bool variable instead of calling the
> eof functions twice for each comparison.
> 
> ...return eof() || Other.eof() ? eof() == Other.eof() : etc.
> 

Thats true ... applied.

 
> I didn't investigate more then; today it compiles (I really don't understand
> why, I have to admit) _but_: now the same VC 8 regression test fails with
> exactly the same error (see below)! (A workaround was to split the function
> into some smaller ones.)
> 
> ..\libs\algorithm\string\test\replace_test.cpp(147) : fatal error C1509:
> compiler limit : too many exception handler states in function
> 'replace_test'. simplify function
> 
> 

I have splited the replace_test() function into smaller chunks. Let's see if it helps.

Regards.

Pavol