[Mesa-dev] [PATCH mesa 2/4] vulkan: add VK_EXT_display_control [v8]

Keith Packard keithp at keithp.com
Thu Jun 21 03:21:27 UTC 2018


Jason Ekstrand <jason at jlekstrand.net> writes:

>> +   /* Look for a DPMS property if we haven't already found one */
>> +   for (int p = 0; connector->dpms_property == 0 && p < 
>> drm_connector->count_props; p++) {
>
> I'm guessing this is well over 80 characters

Sorry I missed this -- I haven't managed to configure my editor to
highlight long lines.

>> +   if (counters)
>> +      result = wsi_display_surface_get_surface_counters(
>> +         icd_surface,
>> +         &counters->supported_surface_counters);
>
> Please put braces around multi-line if statements.

Ok.

-- 
-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/20180620/f28b00ec/attachment.sig>


More information about the mesa-dev mailing list