webkitpy: Slightly adjusting pylintrc allow names.

third_party/WebKit/Tools/Scripts/webkitpy/pylintrc

 # Copyright (c) 2012 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 # copyright notice, this list of conditions and the following disclaimer
@@ skipping 185 matching lines @@
 argument-rgx=[a-z_][a-z0-9_]{1,30}$
 
 # Regular expression which should only match correct variable names.
 variable-rgx=[a-z_][a-z0-9_]{1,30}$
 
 # Regular expression which should only match correct list comprehension or
 # generator expression variable names.
 inlinevar-rgx=[A-Za-z_][A-Za-z0-9_]*$
 
 # Good variable names which should always be accepted, separated by a comma.
-good-names=i,j,k,ex,Run,_,_log
+# These short names are acceptable in the following cases;

[qyearsley, 2017/03/08 15:04:11]

Nit: semicolon -> colon

[mithro, 2017/03/09 03:01:08]

Done.

+#  _        - unused value
+#  i/j/k/x  - for XXX in range():

[qyearsley, 2017/03/08 15:04:11]

Bikeshedding: I think in most cases where you'd want to use `x`, there's
probably a better variable name.

[mithro, 2017/03/09 03:01:09]

Actually the case where x makes sense, y and z also make sense. IE Where the
values are coordinates.

+#  e/ex/err - exception XXXError as e:

[qyearsley, 2017/03/08 15:04:11]

I'm not sure if ex and err need to be in the below list (since lowercase names
longer than two characters are OK according to variable-rgx above)

Also I personally feel that in that case it's nicer to use a word (e.g. error)
as the variable name.

[mithro, 2017/03/09 03:01:09]

I removed ex and exp but as the following is extremely common in Python code,

   exception XXError as e:

By the consistency rule I think we should aim to use that.

+#  f/fd     - with open('blah') as f:
+#  _log/Run - webkitpy functions
+good-names=_,i,j,k,x,e,ex,err,f,fd,_log,Run
 
 # Bad variable names which should always be refused, separated by a comma.
 bad-names=foo,bar,baz,toto,tutu,tata
 
 # Regular expression which should only match functions or classes name which do
 # not require a docstring.
 no-docstring-rgx=__.*__
 
 
 [DESIGN]
@@ skipping 59 matching lines @@
 # Create a graph of internal dependencies in the given file (report RP0402 must
 # not be disabled).
 int-import-graph=
 
 
 [EXCEPTIONS]
 
 # Exceptions that will emit a warning when being caught. Defaults to
 # "Exception".
 overgeneral-exceptions=Exception