[PorterStemmer] Rename vowelinstem to use underscores for PEP 0008 compliance

ExplodingCabbage · ExplodingCabbage · commit 221c19375ce6 · 2016-01-10T20:07:12.000Z
diff --git a/nltk/stem/porter.py b/nltk/stem/porter.py
@@ -229,8 +229,8 @@ def _measure(self, stem):
         # docstring above, which is in turn equivalent to `m`
         return cv_sequence.count('vc')
 
-    def _vowelinstem(self, stem):
-        """vowelinstem(stem) is TRUE <=> stem contains a vowel"""
+    def _contains_vowel(self, stem):
+        """_contains_vowel(stem) is TRUE <=> stem contains a vowel"""
         for i in range(len(stem)):
             if not self._cons(stem, i):
                 return True
@@ -397,8 +397,8 @@ def _step1b(self, word):
             
         rule_2_or_3_succeeded = False
         for rule in [
-            ('ed', '', self._vowelinstem),  # (*v*) ED  ->   
-            ('ing', '', self._vowelinstem), # (*v*) ING ->
+            ('ed', '', self._contains_vowel),  # (*v*) ED  ->   
+            ('ing', '', self._contains_vowel), # (*v*) ING ->
         ]:
             try:
                 intermediate_stem = self._replace_suffix_if(word, *rule)
@@ -446,7 +446,7 @@ def _step1c(self, word):
         'enjoy'. Step 1c is perhaps done too soon; but with this modification that
         no longer really matters.
 
-        Also, the removal of the vowelinstem(z) condition means that 'spy', 'fly',
+        Also, the removal of the contains_vowel(z) condition means that 'spy', 'fly',
         'try' ... stem to 'spi', 'fli', 'tri' and conflate with 'spied', 'tried',
         'flies' ...
         """
