[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [patch #7683] u32_t should be time_t in some cases
From: |
Simon Goldschmidt |
Subject: |
[lwip-devel] [patch #7683] u32_t should be time_t in some cases |
Date: |
Fri, 09 Dec 2011 09:18:32 +0000 |
User-agent: |
Mozilla/5.0 (Windows NT 5.1; rv:8.0) Gecko/20100101 Firefox/8.0 |
Follow-up Comment #3, patch #7683 (project lwip):
> I wondered if LWIP actually makes assumptions about times being 32-bits
Yes, it does. 32 bit are enough in most cases as we only need the time
difference. If timers are called often enough, even 16 bit would suffice. The
only problem I could think of with that would be SNMP, but I'm not sure.
> I used the UNIX port in lwip-contrib as a starting point, so the
> change could be made there.
I guess that port might either rely on time being 32 bit (I would doubt that,
however), or the original makefiles don't spit out a warning when implicitly
converting. Would you mind creating a patch to see what would need to be
changed?
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/patch/?7683>
_______________________________________________
Nachricht geschickt von/durch Savannah
http://savannah.nongnu.org/
- [lwip-devel] [patch #7683] u32_t should be time_t in some cases, Mike Pontillo, 2011/12/08
- [lwip-devel] [patch #7683] u32_t should be time_t in some cases, Simon Goldschmidt, 2011/12/09
- [lwip-devel] [patch #7683] u32_t should be time_t in some cases, Mike Pontillo, 2011/12/09
- [lwip-devel] [patch #7683] u32_t should be time_t in some cases,
Simon Goldschmidt <=
- [lwip-devel] [patch #7683] u32_t should be time_t in some cases, Mike Pontillo, 2011/12/12
- [lwip-devel] [patch #7683] u32_t should be time_t in some cases, Simon Goldschmidt, 2011/12/12
- [lwip-devel] [patch #7683] u32_t should be time_t in some cases, Mike Pontillo, 2011/12/12
- [lwip-devel] [patch #7683] u32_t should be time_t in some cases, Mike Pontillo, 2011/12/12
- [lwip-devel] [patch #7683] unix port is missing some explicit casts, Simon Goldschmidt, 2011/12/13