Re: Patch: Fix for Bug 1164, 2nd try

[Hubert Figuiere]

According to Dom Lachowicz <[EMAIL PROTECTED]>:
> > >  Thank you for research. That added chunk:
> > >                                else
> > >                                        m_mbtowc.initialize();
> > >  should be definitely removed.
> > 
> > I'm not an iconv expert but I thought we needed something like
> > that to reset the internal state after trying and failing to
> > convert a character.  Something *like* that - but not exactly that...
> 
> This stops a few SEGVs. The iconv_t pointer was being corrupted and the
> internal state of the MbToWc most definitately needs to be reset.


Perhaps should we have a reinitialize () method that does the dirty work
without discarding everything ? Shouldn't we ?


Hub