On Friday, 2017-06-16 12:10:55 +0300, Grazvydas Ignotas wrote: > On Fri, Jun 16, 2017 at 1:53 AM, Eric Engestrom <[email protected]> wrote: > > My refactor missed the fact that `native_platform` is static. > > Add the proper guard around the detection code, as it might not be > > necessary, and only print the debug message when a detection was > > actually performed. > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101252 > > Fixes: 7adb9b094894a512c019 ("egl/display: remove unnecessary code and > > make it easier to read") > > Signed-off-by: Eric Engestrom <[email protected]> > > Reviewed-by: Grazvydas Ignotas <[email protected]> > > Ack-by: Emil Velikov <[email protected]> > > nit1: usually it's "Acked-by", but there are a few "Ack-by" in git > history too (2255 vs 3). > [...] > > nit2: you can make it const. Either way, my r-b still stands.
Fixed both and pushed, thanks and sorry it took so long :) Cheers, Eric _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
