[PATCH xrandr] xrandr: suppress misleading indentation warning

Adam Jackson ajax at nwnk.net
Fri Mar 24 15:11:50 UTC 2017


On Wed, 2017-01-18 at 08:52 +0100, Giuseppe Bilotta wrote:
> When printing out rotations, we print a space before any item other than
> the first, and set `first = False` in each block where we print.
> However, this is done in the same line as the conditional that checks if
> first is set, which may give the impression that the assignment is also
> under the conditional. This is not the case, and recent GCC warns about
> this.
> 
> Move the assignment to after we print the value we want to print, which
> (1) doesn't mislead about the indentation, and
> (2) makes logical sense as the _next_ entry is what won't be the first.

Merged this and the --dpi documentation patch, thanks!

remote: I: patch #133793 updated using rev 85e95db7120da3bdaf9efb3033be5f9338e6c328.
remote: I: patch #133794 updated using rev 215a01f1513f918e7295a8a477d4674f7b8085f0.
remote: I: 2 patch(es) updated to state Accepted.
To ssh://git.freedesktop.org/git/xorg/app/xrandr
   1b561d2..215a01f  master -> master

- ajax


More information about the xorg-devel mailing list