I'm struggling to understand the motivation for this patch. krh has always said that you need to think of uint and int as two entirely separate types -- mixing both in math will likely screw up. You can see this in other places -- widths are often expressed as signed ints in the protocol, not unsigned ints.
On Fri, Oct 23, 2015 at 11:53 PM, Giulio Camuffo <[email protected]> wrote: > 2015-10-24 1:29 GMT+03:00 Bryce Harrington <[email protected]>: >> Signed-off-by: Bryce Harrington <[email protected]> >> --- >> src/compositor.h | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/src/compositor.h b/src/compositor.h >> index 6bb6222..4443c72 100644 >> --- a/src/compositor.h >> +++ b/src/compositor.h >> @@ -619,9 +619,9 @@ enum weston_capability { >> */ >> struct weston_backend_output_config { >> uint32_t transform; >> - int32_t width; >> - int32_t height; >> - int scale; >> + uint32_t width; >> + uint32_t height; >> + uint32_t scale; >> }; > > Reviewed-by: Giulio Camuffo <[email protected]> > >> >> /* Configuration struct for a backend. >> -- >> 1.9.1 >> > _______________________________________________ > wayland-devel mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/wayland-devel -- Jasper _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
