https://bugzilla.gnome.org/show_bug.cgi?id=783901
Jonas Ådahl <[email protected]> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #366645|none |accepted-commit_now
status| |
--- Comment #41 from Jonas Ådahl <[email protected]> ---
Review of attachment 366645:
(In reply to Olivier Fourdan from comment #40)
...
>
> Note: I kept the curly braces on purpose for readability, even though these
> are not strictly necessary as there is only one statement left.
It's also the coding style (if any branch of a condition needs brackets, then
add brackets on all so they are horizontally aligned).
Anyhow, lgtm!
--
You are receiving this mail because:
You are on the CC list for the bug._______________________________________________
wayland-bugs mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
