lwip-devel
[Top][All Lists]
Advanced

[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/





reply via email to

[Prev in Thread] Current Thread [Next in Thread]