On Wed, 19 Dec 2018 22:00:38 +0200 Serhey Popovych <serhe.popov...@gmail.com> wrote:
> Stephen Hemminger wrote: > > > The argument to print_0xhex is converted to unsigned long long > > so the format string give for normal printout has to be some > > variant of %llx. Otherwise, bogus values will be printed on > > 32 bit platforms. > > Sorry it is too late and change is merged as commit 90c5c969f0b9 > ("fix print_0xhex on 32 bit") but I want to ask following: > > $ printf '0x%llx != %#llx\n' 0 0 > 0x0 != 0 > > So we potentially can get "tos 0" vs "tos 0x0" previously. Is that > expected and will not cause any compatibility problems? > > It is clear that 0 is always zero, but some code may rely on 0x form > even for zero. What do you think? > > Thanks. > > Hmm. that is a surprise. we can change back if it matters. Was just trying to be consistent. Since most code that scraps is going to use strtoul(str, NULL, 16) it shouldn't matter.
pgpQI7hGtm0Uf.pgp
Description: OpenPGP digital signature