[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [patch #6521] lwip doesn't compile in 64_bit computers
From: |
Mike Kleshov |
Subject: |
[lwip-devel] [patch #6521] lwip doesn't compile in 64_bit computers |
Date: |
Wed, 18 Feb 2009 20:31:42 +0000 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.9.0.6) Gecko/2009011913 Firefox/3.0.6 (.NET CLR 3.5.30729) |
Follow-up Comment #5, patch #6521 (project lwip):
Oops, it appears Jared posted a comment on the lwip-devel mailing list rather
than here, in comments to this patch.
Anyway, I agree with Jared:
- %d is good for u8_t: ANSI C requires that u8_t is converted to int when
passed to printf and friends
- size_t needs a new define: I haven't dug deep into that, but it sounds like
a safe option
As for automated checking of format string consistency, shouldn't lint-like
tools do this? Splint, maybe? And I know for a fact that the RealView compiler
for ARM emits warnings in these cases.
- mike
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/patch/?6521>
_______________________________________________
Message sent via/by Savannah
http://savannah.nongnu.org/