<div dir="ltr"><div><div><div><div>I'll look through for the parens, It might be in a seperate patch, but I think it deserves to be here, since PEP8 is all about readability.<br><br>The final one it complained about, but after reading PEP8 again I understand why, and can fix it. In this case it wants something like this:<br>
</div>variable = {<br></div>     key: value,<br></div>     key: value<br>}<br><br></div>With the important note that the closing brace (or parentheses or bracket) should either be on the last line of text, or should be on the next line even with the indent of the first line.<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 22, 2013 at 2:26 PM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 07/10/2013 03:19 PM, Dylan Baker wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
There are a 3 lines that are just over the recommended 79 character<br>
limit, however, breaking them made them harder to read so they were<br>
left.<br>
---<br>
  glapi/parse_glspec.py | 76 ++++++++++++++++++++++++------<u></u>---------------------<br>
  1 file changed, 36 insertions(+), 40 deletions(-)<br>
</blockquote>
<br>
<br>
<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><div class="im">
      m = GLES_VERSION_REGEXP.match(<u></u>category_name)<br>
      if m:<br>
          ones = int(m.group(1))<br>
          tenths = int(m.group(2))<br>
-        return 'GLES{0}.{1}'.format(ones, tenths), {<br>
-            'kind': 'GLES',<br>
-            'gl_10x_version': 10 * ones + tenths<br>
-            }<br>
+        return 'GLES{0}.{1}'.format(ones, tenths), {'kind': 'GLES',<br>
+                                                    'gl_10x_version':<br>
+                                                    10 * ones + tenths}<br>
<br>
</div></blockquote>
<br>
<br>
The original was significantly more readable. Maybe you could achieve PEP-8<br>
correctness while maintaining readability by using explicit parens around the<br>
tuple. Maybe like this:<br>
<br>
    return ('GLES{0}.{1}'.format(ones, tenths),<br>
            {'kind': 'GLES',<br>
             'gl_10x_version': 10 * ones + tenths})<br>
<br>
or this<br>
<br>
    return (<div class="im"><br>
        'GLES{0}.{1}'.format(ones, tenths),<br>
        {'kind': 'GLES',<br></div><div class="im">
         'gl_10x_version': 10 * ones + tenths}<br></div>
    )<br>
<br>
>From my interpretation of the PEP8 doc, both are PEP8 correct.<div class="im"><br>
<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
      extension_name = 'GL_' + category_name<br>
-    return extension_name, {<br>
-        'kind': 'extension',<br>
-        'extension_name': extension_name<br>
-        }<br>
+    return extension_name, {'kind': 'extension',<br>
+                            'extension_name': extension_name}<br>
</blockquote>
<br>
<br></div>
While your changing the formatting of tuples, could you add explicit<br>
parens to aid readability? There are several other locations in this<br>
patch where parens could be added. Or do you think that belongs to<br>
a separate patch?<div class="im"><br>
<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
          if name not in self.functions:<br>
-            self.functions[name] = {<br>
-                'return_type': return_type,<br>
-                'param_names': param_names,<br>
-                'param_types': param_types,<br>
-                'categories': [category],<br>
-                }<br>
+            self.functions[name] = {'return_type': return_type,<br>
+                                    'param_names': param_names,<br>
+                                    'param_types': param_types,<br>
+                                    'categories': [category]}<br>
</blockquote>
<br></div>
Did the PEP8 tool complain about this one?<br>
<br>
</blockquote></div><br></div>