skroutz / elasticsearch-skroutz-greekstemmer

Greek Stemmer for elasticsearch
https://www.skroutz.gr
74 stars 34 forks source link

Missing Parenthesis #3

Closed markpapadakis closed 10 years ago

markpapadakis commented 10 years ago

You should use parenthesis to resolve ambiguity here: else if (len > 4 && endsWith(..) .. to else if (len > 4 && (endsWith(..) ...

https://github.com/skroutz/elasticsearch-skroutz-greekstemmer/blob/master/src/main/java/org/elasticsearch/index/analysis/SkroutzGreekStemmer.java#L813

astathopoulos commented 10 years ago

Thank you for reporting it!