[poppler] patch to make selection works with rawOrder

Daniel Garcia Moreno danigm at yaco.es
Fri Oct 1 00:08:21 PDT 2010


El jue, 30-09-2010 a las 22:32 +0100, Albert Astals Cid escribió:
> A Dijous, 30 de setembre de 2010, Daniel Garcia Moreno va escriure:
> > El mar, 28-09-2010 a las 23:19 +0100, Albert Astals Cid escribió:
> > > A Dissabte, 18 de setembre de 2010, Daniel Garcia Moreno va escriure:
> > > > El vie, 17-09-2010 a las 19:33 +0100, Albert Astals Cid escribió:
> > > > > A Divendres, 17 de setembre de 2010, Daniel Garcia Moreno va escriure:
> > > > > > I've modified my patch to works as normal when !rawOrder and when
> > > > > > TextOutputDev.rawOrder makes selection with "raw" algorithm.
> > > > > > 
> > > > > > This solution doesn't break current one and adds new functionality
> > > > > > to poppler.
> > > > > > 
> > > > > > What do you think about it?, it's unneeded or it's a good feature
> > > > > > to add to poppler? I think it's useful, because I do it :P and
> > > > > > because it works for me...
> > > > > 
> > > > > Not sure i understand you, are you suggesting that in raw mode we
> > > > > sort words (using secondaryCmp)?
> > > > > 
> > > > > Then it's not raw anymore, is it?
> > > > 
> > > > secondaryCmp is used to group text in lines in getSelectionText and to
> > > > group selection region by lines in getSelectionRegion, it doesn't
> > > > change text order.
> > > 
> > > Question do you really need to "select" the text (i.e update the text
> > > painted on screen)? Or you only want to "retrieve" the text (i.e. get
> > > the raw text of the page)?
> > 
> > Yes, I need it, because it works as expected with tables.
> 
> You can't answer "Yes" to two questions ;-)
> 
> Are you answering "Yes" to the first or to the second?

My answer was for the first question. Sorry for the misunderstand.



More information about the poppler mailing list