Re: [patch 1/7] kernel: fix bad TextSimpleOperator::getRawText

[Jozef M. <mis...@ho...>]

ack

> getRawText is intended only for low level operations when one wants to  
> work
> directly with the font dependent data (character codes) which can be  
> different
> than what is printed in the end (typical examples are pdfs created by
> OpenOffice which uses char codes from 1 to the number of distinct  
> characters
> and each character is then assigned to one such a code in the font  
> unicode map).
>
> CContentStream::replaceText is not such a function, obviously, so it has  
> to
> use getFontText instead.
> Index: pdfedit-patches/src/kernel/ccontentstream.cc
> ===================================================================
> --- pdfedit-patches.orig/src/kernel/ccontentstream.cc	2010-06-09  
> 16:23:30.000000000 +0200
> +++ pdfedit-patches/src/kernel/ccontentstream.cc	2010-06-09  
> 16:23:56.000000000 +0200
> @@ -659,7 +659,7 @@ CContentStream::replaceText (const std::
>  		boost::shared_ptr<TextSimpleOperator> _cur
>  				= boost::dynamic_pointer_cast<TextSimpleOperator, PdfOperator>  
> (tit.getCurrent());
>  		std::string tmp;
> -		_cur->getRawText (tmp);
> +		_cur->getFontText (tmp);
>  		string replaced = boost::replace_all_copy (tmp, what, with);
>  		if (tmp != replaced)
>  		{
>
>
>
> ------------------------------------------------------------------------------
> ThinkGeek and WIRED's GeekDad team up for the Ultimate
> GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the
> lucky parental unit.  See the prize list and enter to win:
> http://p.sf.net/sfu/thinkgeek-promo
> _______________________________________________
> Pdfedit-devel mailing list
> Pdf...@li...
> https://lists.sourceforge.net/lists/listinfo/pdfedit-devel
>


-- 
Using Opera's revolutionary e-mail client: http://www.opera.com/mail/