rzo1 commented on code in PR #561: URL: https://github.com/apache/opennlp/pull/561#discussion_r1419124646
########## opennlp-tools/src/main/java/opennlp/tools/stemmer/snowball/arabicStemmer.java: ########## @@ -30,2094 +30,1284 @@ OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE */ -// This file was generated automatically by the Snowball to Java compiler - +// Generated by Snowball (build from 867c4ec70debd4daa7fb4d5a9f7759b47887d0b9) package opennlp.tools.stemmer.snowball; - - /** - * This class was automatically generated by a Snowball to Java compiler - * It implements the stemming algorithm defined by a snowball script. - */ - +/** + * This class implements the stemming algorithm defined by a snowball script. + * <p> + * Generated by Snowball (build from 867c4ec70debd4daa7fb4d5a9f7759b47887d0b9) - <a href="https://github.com/snowballstem/snowball";>https://github.com/snowballstem/snowball</a> + * </p> + */ +@SuppressWarnings("unused") public class arabicStemmer extends AbstractSnowballStemmer { Review Comment: Yes. Normally, it should start with a capitalized letter. For the generation, it does not really matter how we name the result. It could be `ArabicStemmer` with capitalized letters. If we decide to change, it might break users as these classes are in the `public` api :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
