cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Conal Tuohy" <Conal.Tu...@vuw.ac.nz>
Subject RE: Recent changes to LuceneIndexTransformer
Date Sun, 27 Jul 2003 23:22:14 GMT
Carsten wrote:

> After the last changes to LuceneIndexTransformer, a fragment
> of the code looks like this:
> 
>                 if (analyzerClassname == null)
>                     analyzerClassname = this.analyzerClassname;
> 
>                 String sMergeFactor =
>                     
> atts.getValue(LUCENE_QUERY_MERGE_FACTOR_ATTRIBUTE);
>                 mergeFactor = this.mergeFactor;
> 
> Now, there is no local variable analyzerClassname nor mergeFactor,
> so the assignments are useless. Looking at the code, I would guess
> that there is something wrong with it.
> 
> Does anybody know what?

Yes ... it was me. These used to be local variables as well but I removed them when refactoring
Vadim's code. I will fix it. In the meantime, the lines of code are at least not harmful.

Con

Mime
View raw message