On Fri, Jul 17, 2015 at 2:06 PM, Pekka Paalanen <[email protected]> wrote:
> On Wed, 1 Jul 2015 12:29:00 +0200 > Marek Chalupa <[email protected]> wrote: > > > sizeof operator returns size_t, which need not to be unsigned int, but > can > > be long (long) unsigned int. > > So here the correct fix should be either to use %zu (since C99) or cast > it > > to unsigned long and use %lu > > < SardemFF7> Why has sizeof() anything do to with your patch? "vtnr" > is an "unsigned int" and is not a size_t or anything O_o > Oops, sorry all. Don't know why I thought it is size_t ... > > ;-) > > Thanks, > pq > > > On Sat, Jun 20, 2015 at 1:28 AM, Bryce Harrington <[email protected] > > > > wrote: > > > > > Addresses this warning found by Denis Denisov: > > > > > > [src/logind-util.c:702]: (warning) %d in format string (no. 1) > > > requires 'int' but the argument type is 'unsigned int'. > > > > > > Signed-off-by: Bryce Harrington <[email protected]> > > > --- > > > src/logind-util.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/src/logind-util.c b/src/logind-util.c > > > index 5c95874..006a1ae 100644 > > > --- a/src/logind-util.c > > > +++ b/src/logind-util.c > > > @@ -702,7 +702,7 @@ weston_logind_setup_vt(struct weston_logind *wl) > > > sigset_t mask; > > > struct wl_event_loop *loop; > > > > > > - snprintf(buf, sizeof(buf), "/dev/tty%d", wl->vtnr); > > > + snprintf(buf, sizeof(buf), "/dev/tty%u", wl->vtnr); > > > buf[sizeof(buf) - 1] = 0; > > > > > > wl->vt = open(buf, O_RDWR|O_CLOEXEC|O_NONBLOCK); > > > -- > > > 1.9.1 > > > > > > _______________________________________________ > > > wayland-devel mailing list > > > [email protected] > > > http://lists.freedesktop.org/mailman/listinfo/wayland-devel > > > > > Regards, Marek
_______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
