[Mesa-dev] [PATCH mesa 01/21] vulkan: Add KHR_display extension using DRM [v4]

Keith Packard keithp at keithp.com
Tue Jun 12 02:39:21 UTC 2018


Jason Ekstrand <jason at jlekstrand.net> writes:

> I apologize for how nit-picky this is going to get but it's starting to
> bother me...

I'm glad you showed me what you wanted; always nice to have help! I've
reformatted and restructured the code along the lines that you've
suggested here for the entire series. In particular:

 * 80 column limit. Just make it work.
 * Declare variables at first use; loop control variables inside the
   loop
 * Add the nice list iterators and use list_ versions of the list macros

I think this touched pretty much every patch in the series, but the
result is more consistent looking, and doesn't require a 200 column
emacs window to see the code.

-- 
-keith
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180611/6da727fc/attachment.sig>


More information about the mesa-dev mailing list