Thread (1 message) 1 message, 1 author, 2012-04-18

Re: [PATCH 10/11] batman-adv: print OGM seq numbers as unsigned long

From: Antonio Quartulli <hidden>
Date: 2012-04-18 07:24:50
Also in: batman

On Wed, Apr 18, 2012 at 09:01:35AM +0200, Antonio Quartulli wrote:
On Wed, Apr 18, 2012 at 08:47:00AM +0200, Antonio Quartulli wrote:
quoted
On Tue, Apr 17, 2012 at 10:47:46 -0400, David Miller wrote:
quoted
From: Antonio Quartulli <redacted>
Date: Tue, 17 Apr 2012 13:58:26 +0200
quoted
OGM sequence numbers are declared as uint32_t and so they have to printed
using %u instead of %d in order to avoid wrong representations.

Signed-off-by: Antonio Quartulli <redacted>
Fix this commit message subject, "%u" is "unsigned int" not
"unsigned long"

"unsigned long" would be "%lu"
There is a problem here. On my machine (x86_64) I have:

typedef unsigned int __u32;
typedef         __u32           uint32_t;


So I should use %u to print my uint32_t variable (as I reported in my commit
message).

Probably this is not the case on each and every architecture?
If so, how could I handle it?
Ok. After digging in the code again I understood that the problem is either in
my brain and in the commit subject.

I'll fix both.

Cheers,


-- 
Antonio Quartulli

..each of us alone is worth nothing..
Ernesto "Che" Guevara

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help