Re: Don't use %z printf format length specified

2018-11-15 Thread Richard Biener
On Thu, Nov 15, 2018 at 2:31 PM Michael Matz wrote: > > Hi, > > On Thu, 15 Nov 2018, Richard Biener wrote: > > > > Okay, probably. Then consider the same patch with sprinkling casts to > > > long for all these arguments (the actual numbers will fit that type in > > > reality). I could of course

Re: Don't use %z printf format length specified

2018-11-15 Thread Michael Matz
Hi, On Thu, 15 Nov 2018, Richard Biener wrote: > > Okay, probably. Then consider the same patch with sprinkling casts to > > long for all these arguments (the actual numbers will fit that type in > > reality). I could of course also use PRIu64 but that makes my eyes > > bleed. > > Hmm. Can

Re: Don't use %z printf format length specified

2018-11-15 Thread Richard Biener
On Wed, Nov 14, 2018 at 6:24 PM Michael Matz wrote: > > Hi, > > On Wed, 14 Nov 2018, Alexander Monakov wrote: > > > On Wed, 14 Nov 2018, Michael Matz wrote: > > > > > Hi, > > > > > > it's not c++98 conforming and I get 1 million warnings when compiling. > > > Initially I had also casts to long at

Re: Don't use %z printf format length specified

2018-11-14 Thread Michael Matz
Hi, On Wed, 14 Nov 2018, Alexander Monakov wrote: > On Wed, 14 Nov 2018, Michael Matz wrote: > > > Hi, > > > > it's not c++98 conforming and I get 1 million warnings when compiling. > > Initially I had also casts to long at the various arguments, but I get no > > warning with this variant ei

Re: Don't use %z printf format length specified

2018-11-14 Thread Andreas Schwab
On Nov 14 2018, Michael Matz wrote: > Initially I had also casts to long at the various arguments, but I get no > warning with this variant either, so I removed them again. That is probably pure luck. size_t is not required to be the same as unsigned long. Andreas. -- Andreas Schwab, SUSE L

Re: Don't use %z printf format length specified

2018-11-14 Thread Alexander Monakov
On Wed, 14 Nov 2018, Michael Matz wrote: > Hi, > > it's not c++98 conforming and I get 1 million warnings when compiling. > Initially I had also casts to long at the various arguments, but I get no > warning with this variant either, so I removed them again. I'm currently > regstrapping this

Don't use %z printf format length specified

2018-11-14 Thread Michael Matz
Hi, it's not c++98 conforming and I get 1 million warnings when compiling. Initially I had also casts to long at the various arguments, but I get no warning with this variant either, so I removed them again. I'm currently regstrapping this, okay for trunk if successfull? Ciao, Michael.