Re: PDFTextStripper : can't change the default TextPositionComparator

[Sébastien Dailly]

Le 08/11/2011 16:53, Raimi Rufai a écrit :

With Generics, things might look like this instead perhaps ...

  private Comparator<TextPosition>  textPositionComparator=  new
TextPositionComparator();

public Comparator<TextPosition>  getTextPositionComparator() {
                        return textPositionComparator;
}

public void setgetTextPositionComparator(Comparator<TextPosition>
comparator) {
           textPositionComparator = comparator;
}



On Tue, Nov 8, 2011 at 10:50 AM, Raimi Rufai<rru...@gmail.com>  wrote:

Hi Sebastien,

It might be more flexible to inject an instance of rather than the class
of the Comparator. For comparators that take parameters, your current
solution won't work. In other words, you would have:

private Comparator<TextPosition>  textPositionComparator=  new
TextPositionComparator();

public Comparator<TextPosition>  getTextPositionComparator() {
                        return textPositionComparator;
}

public void setgetTextPositionComparator(Comparator<TextPosition>
comparator) {
           textPositionComparator = comparator;
}

What do you think?

I Raimi,

I think your solution is better. I'm opening an issue in Jira for that.

--
Sébastien