Sam TH wrote: > > This looks really good. A couple quick comments: > > - _recognizeUCS/UTF8 should definitely be members of class. > IE_Imp_Text_Sniffer is probably the best choice. If I make the enum a member of the sniffer and the two functions static members, and make IE_Imp_Text a friend of the sniffer, would that be good? If the functions need to be non-static there may need to be some tricky pointer shuffling... Andrew. -- http://linguaphile.sourceforge.net _________________________________________________________ Do You Yahoo!? Get your free @yahoo.com address at http://mail.yahoo.com
- Re: Patch: Multi-encoding Text import/exp... Andrew Dunbar
- Re: Patch: Multi-encoding Text impor... Vlad Harchev
- Re: Patch: Multi-encoding Text impor... Dom Lachowicz
- Re: Patch: Multi-encoding Text impor... Andrew Dunbar
- Re: Patch: Multi-encoding Text impor... Vlad Harchev
- Re: Patch: Multi-encoding Text impor... Hubert Figuiere
- Re: Patch: Multi-encoding Text impor... Vlad Harchev
- Re: Patch: Multi-encoding Text impor... Andrew Dunbar
- Re: Patch: Multi-encoding Text impor... Hubert Figuiere
- Re: Patch: Multi-encoding Text impor... Vlad Harchev
- Re: Patch: Multi-encoding Text import/export Andrew Dunbar
- Re: Patch: Multi-encoding Text import/export Vlad Harchev
