pdf-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [pdf-devel] Re: Modifications on pdf_token_read to get token boundar


From: jemarch
Subject: Re: [pdf-devel] Re: Modifications on pdf_token_read to get token boundaries
Date: Thu, 18 Jun 2009 00:17:32 +0200
User-agent: Wanderlust/2.14.0 (Africa) SEMI/1.14.6 (Maruoka) FLIM/1.14.8 (Shijō) APEL/10.6 Emacs/23.0.60 (i686-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO)

   >       The easiest way to implement it in the token reader would probably
   >       be to replace the "reader->state = X" statements with calls to a
   >       private function (e.g. enter_state) that would also store the
   >       token's starting position.  Then in flush_token, store this value
   >       when emitting a token.
   > 
   >    Many thanks for the hints :)
   > 
   > I just merged the type4+tokeniser patch in the trunk. All the unit
   > tests for the type 4 functions are passing.

   The code you committed changes beg_pos in flush_token even if no token
   is emitted, which could leave it pointing at an unexpected position
   after an error like EAGAIN.  I think that line of code belongs between
   "*token = new_tok;" and "finish:".

Right. Fixed.

-- 
Jose E. Marchesi  <address@hidden>
                  http://www.jemarch.net
GNU Project       http://www.gnu.org




reply via email to

[Prev in Thread] Current Thread [Next in Thread]