Date: Tue, 20 Dec 2016 10:14:31 +0800 From: Sepherosa Ziehau <sepherosa@gmail.com> To: Ravi Pokala <rpokala@mac.com> Cc: Ian Lepore <ian@freebsd.org>, Warner Losh <imp@bsdimp.com>, Dimitry Andric <dim@freebsd.org>, src-committers <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> Subject: Re: svn commit: r310171 - head/sys/sys Message-ID: <CAMOc5cxOX1LtB_HuZz8TRYbrQT_Jz_h5HYmveUKOXtSfsvERCg@mail.gmail.com> In-Reply-To: <F65A3D59-6FCD-4EEC-B2D2-40DB682B716D@panasas.com> References: <201612161949.uBGJnMol059217@repo.freebsd.org> <CAMOc5cw8cBUN5UZjcRJ-vM5WE0vmD2AGDZ-raKTraFz7LBR2hg@mail.gmail.com> <9BD5034F-55A6-48F6-A391-A0877FF49702@panasas.com> <CANCZdfoN_NNsU2bW2mvLifSCoNhYj-XNvNYLosMsbmnbG9JfYQ@mail.gmail.com> <1482175209.48539.9.camel@freebsd.org> <F65A3D59-6FCD-4EEC-B2D2-40DB682B716D@panasas.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Dec 20, 2016 at 4:37 AM, Ravi Pokala <rpokala@mac.com> wrote: > -----Original Message----- >> From: <owner-src-committers@freebsd.org> on behalf of Ian Lepore <ian@freebsd.org> >> Date: 2016-12-19, Monday at 11:20 >> To: Warner Losh <imp@bsdimp.com>, Ravi Pokala <rpokala@mac.com> >> Cc: Sepherosa Ziehau <sepherosa@gmail.com>, Dimitry Andric <dim@freebsd.org>, src-committers <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> >> Subject: Re: svn commit: r310171 - head/sys/sys >> >> On Mon, 2016-12-19 at 11:58 -0700, Warner Losh wrote: >>> >>> ... >>> >>> Are there other precedence for avoiding the SCN macros in the tree as >>> well, or is this new art? >>> >>> Warner >> >> There was another commit recently the fixed the same kind of scanf >> error by making the variable fit the scanf type (changing uint64_t to >> an explicit long long unsigned, iirc). I don't know if that alone >> counts as a precedent, but IMO it's a more palatible fix than the >> SCN/PRI ugliness. > > With all apologies to Churchill, SCN/PRI are the worst way to address this in a machine-independent way, except for all the other ways that have been tried from time to time. :-P > If no objection comes, I'd commit the posted patch as it is tomorrow in my time. Thanks, sephe -- Tomorrow Will Never Die
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAMOc5cxOX1LtB_HuZz8TRYbrQT_Jz_h5HYmveUKOXtSfsvERCg>