On Sun, Nov 13, 2011 at 2:08 PM, Pawel Jakub Dawidek <p...@freebsd.org> wrote: > On Sat, Nov 12, 2011 at 12:16:23PM -0800, Garrett Cooper wrote: >> On Sat, Nov 12, 2011 at 12:01 PM, Alexander Motin <m...@freebsd.org> wrote: >> > Author: mav >> > Date: Sat Nov 12 20:01:30 2011 >> > New Revision: 227473 >> > URL: http://svn.freebsd.org/changeset/base/227473 >> > >> > Log: >> > Fix build on some archs after r227464. >> > >> > Modified: >> > head/sbin/geom/class/multipath/geom_multipath.c >> > >> > Modified: head/sbin/geom/class/multipath/geom_multipath.c >> > ============================================================================== >> > --- head/sbin/geom/class/multipath/geom_multipath.c Sat Nov 12 >> > 19:55:48 2011 (r227472) >> > +++ head/sbin/geom/class/multipath/geom_multipath.c Sat Nov 12 >> > 20:01:30 2011 (r227473) >> > @@ -133,7 +133,8 @@ mp_label(struct gctl_req *req) >> > uint8_t *sector, *rsector; >> > char *ptr; >> > uuid_t uuid; >> > - uint32_t secsize = 0, ssize, status; >> > + ssize_t secsize = 0, ssize; >> > + uint32_t status; >> > const char *name, *name2, *mpname; >> > int error, i, nargs, fd; >> > >> > @@ -161,8 +162,8 @@ mp_label(struct gctl_req *req) >> > disksize = msize; >> > } else { >> > if (secsize != ssize) { >> > - gctl_error(req, "%s sector size %u >> > different.", >> > - name, ssize); >> > + gctl_error(req, "%s sector size %ju >> > different.", >> > + name, (intmax_t)ssize); >> >> Shouldn't that be uintmax_t, not intmax_t ? > > No, ssize_t is signed. Although the best would be to use %zd for > ssize_t.
Thanks... Missed the leading s. -Garrett _______________________________________________ svn-src-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
