Hm, it looks like the code is published under LGPL (see the end of the
PDF).
Wouldn't it have to be published under ASF License before it could be
included in Lucene distribution?

Otis


--- [EMAIL PROTECTED] wrote:
> Hi,
> 
> I just added the Highlight terms functionality by Maik Schreiber to
> the 
> contributions section.
> This code essentially gets all the terms for a given query, and then 
> provides an interface to highlight the text.
> 
> In order to provide this functionality he had to make some changes to
> 
> the core Lucene code.
> 
> I am suggesting providing the a query.getTerms() method. After review
> 
> and Maik approval, we might even want to use his code.
> 
> I would suggest making these changes after 1.2 is released.
> 
> To check out his code and PDF describing the changes, go to
> http://www.iq-computing.de/lucene/highlight.htm
> 
> Please provide feedback.
> 
> Thanks
> 
> --Peter
> 
> ----LIST OF SUGGESTED CHANGES----
> He documented it very well. Here are the list of changes.
> 
> org.apache.lucene.search.BooleanQuery - add the following method:
> public BooleanClause[] getClauses()
> {
> return (BooleanClause[]) clauses.toArray(new BooleanClause[0]);
> }
> 
> org.apache.lucene.search.MultiTermQuery - mark getQuery() public
> 
> org.apache.lucene.search.PhraseQuery - add the following method:
> public Term[] getTerms()
> {
> return (Term[]) terms.toArray(new Term[0]);
> }
> 
> org.apache.lucene.search.PrefixQuery - mark getQuery() public
> 
> org.apache.lucene.search.RangeQuery - mark getQuery() public
> 
> org.apache.lucene.search.TermQuery - add the following method:
> public Term getTerm()
> {
> return term;
> }
> 


__________________________________________________
Do You Yahoo!?
Got something to say? Say it better with Yahoo! Video Mail 
http://mail.yahoo.com

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to