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 _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel
