2016-04-25 22:34 GMT+08:00 Pedro Giffuni <p...@freebsd.org>: > Marcelo, please revert. > > And check your script, of course. >
Fixed thanks! The problem is the weird way the expression was wrote before. Best, > > Pedro. > > > On 23/04/2016 04:23, luke wrote: > > On Fri, Apr 22, 2016 at 11:43 AM, Marcelo Araujo < <ara...@freebsd.org> > ara...@freebsd.org> wrote: > >> Author: araujo >> Date: Fri Apr 22 03:43:06 2016 >> New Revision: 298443 >> URL: https://svnweb.freebsd.org/changeset/base/298443 >> >> Log: >> Use macro MAX() from sys/param.h. >> >> MFC after: 2 weeks. >> >> Modified: >> head/usr.bin/stat/stat.c >> >> Modified: head/usr.bin/stat/stat.c >> >> ============================================================================== >> --- head/usr.bin/stat/stat.c Fri Apr 22 03:37:27 2016 (r298442) >> +++ head/usr.bin/stat/stat.c Fri Apr 22 03:43:06 2016 (r298443) >> @@ -1025,7 +1025,7 @@ format1(const struct stat *st, >> * >> * Nanoseconds: long. >> */ >> - (void)snprintf(tmp, sizeof(tmp), "%dld", prec > 9 ? 9 : >> prec); >> + (void)snprintf(tmp, sizeof(tmp), "%dld", MAX(9, prec)); >> (void)strcat(lfmt, tmp); >> >> /* >> _______________________________________________ >> svn-src-head@freebsd.org mailing list >> https://lists.freebsd.org/mailman/listinfo/svn-src-head >> To unsubscribe, send any mail to " <svn-src-head-unsubscr...@freebsd.org> >> svn-src-head-unsubscr...@freebsd.org" >> > > > Hi, > > Should this be MIN() ? > > (void)snprintf(tmp, sizeof(tmp), "%dld", MIN(prec, 9)); > > > -- > Chang-Hsien Tsai > > > -- -- Marcelo Araujo (__)ara...@freebsd.org \\\'',)http://www.FreeBSD.org <http://www.freebsd.org/> \/ \ ^ Power To Server. .\. /_) _______________________________________________ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
