lwIP - A Lightweight TCP/IP stack - Patches: patch #6861, Support for proper system...
You are not allowed to post comments on this tracker with your current authentication level.
patch #6861: Support for proper system termination in lwIP
Submitter: | Laura Garrett <laura4garrett> | ||
Submitted: | Thu 09 Jul 2009 01:18:31 PM UTC | ||
Category: | None | Priority: | 5 - Normal |
Status: | Wont Do | Privacy: | Public |
Assigned to: | goldsimon | Open/Closed: | Closed |
Planned Release: | None |
Wed 07 Oct 2009 06:13:50 PM UTC, comment #4: |
Simon Goldschmidt <goldsimon> |
Tue 14 Jul 2009 02:35:06 PM UTC, comment #3:
There are exceptions. We have a battery powered GPRS terminal which acquires information and goes online only for a minute or two every day in order to send it. I bet there are much users that have similar needs.
It can be surrounded by something like #if LWIP_SHUTDOWN_SUPPORT
How about schedule it for 1.4?
|
Iordan Neshev <iordan_neshev> |
Thu 09 Jul 2009 04:49:49 PM UTC, comment #2: Seems like it's meant as local reference to the global 'mbox', which could have to be deleted when the stack is shut down, although that part seems to be missing (at least in the diff).
|
Simon Goldschmidt <goldsimon> |
Thu 09 Jul 2009 03:02:59 PM UTC, comment #1: Can you explain more about why all the local_mbox stuff is necessary? |
Kieran Mansley <kieranm> |
Thu 09 Jul 2009 01:18:31 PM UTC, original submission:
Hi,
|
Laura Garrett <laura4garrett> |
Depends on the following items: None found
Items that depend on this one: None found
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 5 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2009-10-07 | goldsimon | Open/Closed | Open | Closed | |
Assigned to | None | goldsimon | |||
Status | None | Wont Do | |||
2009-07-09 | laura4garrett | Attached File | - | Added terminate.diff, #18394 | |
Carbon-Copy | - | Added -email is unavailable- |
Powered by Savane 3.13-d3ae.
Corresponding source code
Still, the patch doesn't do what you want: we rely pretty much on implicit variable initialization (for variables initialized to NULL, zero or something like that). And mailboxes, semaphores and so on aren't deleted, as I already mentioned.
As such and as this patch bloats the code even if shutdown is not used, I vote against it.
Please repoen if you disagree.