Review messages for E713 and E714

Author: florentx

docs/intro.rst

@@ -308,9 +308,9 @@ This is the current list of error and warning codes:
 +----------+----------------------------------------------------------------------+
 | E712 (^) | comparison to True should be 'if cond is True:' or 'if cond:'        |
 +----------+----------------------------------------------------------------------+
-| E713     | evaluating membership should be 'elem not in collection'             |
+| E713     | test for membership should be 'not in'                               |
 +----------+----------------------------------------------------------------------+
-| E714     | testing unequal identities should be 'x is not y'                    |
+| E714     | test for object identity should be 'is not'                          |
 +----------+----------------------------------------------------------------------+
 | E721     | do not compare types, use 'isinstance()'                             |
 +----------+----------------------------------------------------------------------+

pep8.py

@@ -986,13 +986,11 @@ def comparison_negative(logical_line):
     """
     match = COMPARE_NEGATIVE_REGEX.search(logical_line)
     if match:
+        pos = match.start(1)
         if match.group(2) == 'in':
-            msg = ("E713: Use the 'not in' "
-                   "operator for collection membership evaluation")
+            yield pos, "E713 test for membership should be 'not in'"
         else:
-            msg = ("E714: Use the 'is not' "
-                   "operator when testing for unequal identities")
-        yield match.start(1), msg
+            yield pos, "E714 test for object identity should be 'is not'"
 
 
 def comparison_type(logical_line):