lwIP - A Lightweight TCP/IP stack - Patches: patch #7536, Fix bad MEM_SIZE check when...
You are not allowed to post comments on this tracker with your current authentication level.
patch #7536: Fix bad MEM_SIZE check when setting type of mem_size_t
Submitter: | Johan Englund <johanenglund> | ||
Submitted: | Thu 28 Apr 2011 09:02:26 AM UTC | ||
Category: | None | Priority: | 5 - Normal |
Status: | Done | Privacy: | Public |
Assigned to: | goldsimon | Open/Closed: | Closed |
Planned Release: | None |
Fri 29 Apr 2011 11:45:00 AM UTC, comment #3: |
Simon Goldschmidt <goldsimon> |
Thu 28 Apr 2011 01:06:09 PM UTC, comment #2: Right you are! A bit too trigger happy, so sorry! |
Johan Englund <johanenglund> |
Thu 28 Apr 2011 12:26:42 PM UTC, comment #1: That's an l (L) not a 1 (one). IMO lower-case l should never be used for long constants. |
Bill Auerbach <billauerbach> |
Thu 28 Apr 2011 09:02:26 AM UTC, original submission:
Compares with 640001 instead of 64000. |
Johan Englund <johanenglund> |
Attached Files
Depends on the following items: None found
Items that depend on this one: None found
Carbon-Copy List
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 4 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2011-04-29 | goldsimon | Status | None | Done | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2011-04-28 | johanenglund | Attached File | - | Added mem.h.patch, #23317 |
Powered by Savane 3.13-d3ae.
Corresponding source code
In terms of readability, I agree and have changed it that way.