[poppler] patch to make selection works with rawOrder

Daniel Garcia Moreno danigm at yaco.es
Thu Sep 30 13:28:17 PDT 2010


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.



More information about the poppler mailing list