Re: Bug fix

[Michal H. <ms...@gm...>]

Hi,

On Thu, Aug 04, 2011 at 03:48:38AM +0200, Jozef wrote:
> 2michal: can you check the diff

Just a nit. Unified diff would be much easier to review (assuming that
you used CVS - cvs diff -up).

> - it fixes nasty bug in ur code.
>  the problem is that creating pdf hex string escaped characters
> which is wrong  (see attached pdfs)

The patch itself looks good. PDF specification says that:
"
Within a literal string, the backslash (\) is used as an escape
character for various purposes, such as to include newline characters,
nonprinting ASCII characters, unbalanced parentheses, or the backslash
character itself in the string.
"

I didn't get those documents thoug. The one that I would expect to be
correct decoded.zadanie.ok.po.zmene.pdf looks garbled.

Thanks
-- 
Michal Hocko