lwIP - A Lightweight TCP/IP stack - Patches: patch #9450, New sockets function: lwip_poll
You are not allowed to post comments on this tracker with your current authentication level.
patch #9450: New sockets function: lwip_poll
Submitter: | Joan Lledó <jllledo> | ||
Submitted: | Thu 07 Sep 2017 04:43:13 PM UTC | ||
Category: | sockets/netconn | Priority: | 5 - Normal |
Status: | Done | Privacy: | Public |
Assigned to: | goldsimon | Open/Closed: | Closed |
Planned Release: | 2.1.0 |
Jump to the original submission
Thu 19 Oct 2017 07:05:19 PM UTC, comment #11: |
Joel Cunningham <jcunningham> |
Thu 19 Oct 2017 05:43:41 PM UTC, comment #10: Nice. I've already pushed it. |
Simon Goldschmidt <goldsimon> |
Mon 16 Oct 2017 08:04:53 PM UTC, comment #9: Attached is some additional poll cleans in regards to Open Group.
|
Joel Cunningham <jcunningham> |
Mon 16 Oct 2017 01:08:20 PM UTC, comment #8: Patch pushed in 0882e0ba894ab21a5fb6e7ce31a58fe5d3f43d32 with suggested nfds > 0 change, thanks! |
Joel Cunningham <jcunningham> |
Mon 16 Oct 2017 08:03:32 AM UTC, comment #7: Looks good.
|
Simon Goldschmidt <goldsimon> |
Fri 13 Oct 2017 05:04:30 PM UTC, comment #6: Ping! Did anyone have a chance to look at my cleanup patch (it's minor)? Thanks |
Joel Cunningham <jcunningham> |
Sun 08 Oct 2017 04:14:31 AM UTC, comment #5: I had a chance to review the poll changes and spotted a couple minor things to clean up:
|
Joel Cunningham <jcunningham> |
Thu 21 Sep 2017 08:25:50 PM UTC, comment #4: Some things have moved from sockets.c to sockets_priv.h. I thought git might handle this, but apparently it doesn't... |
Simon Goldschmidt <goldsimon> |
Thu 21 Sep 2017 08:21:17 PM UTC, comment #3: What's the error? I may be creating the patch with the wrong command. Or maybe the sources have changed since I wrote the patch and the lines aren't matching. |
Joan Lledó <jllledo> |
Thu 21 Sep 2017 08:20:45 PM UTC, comment #2: I've pushed the patch. Cleaned it up a bit, too. Having LWIP_SOCKET_POLL==2 is just not the lwIP way. I guess we have patch #9350 for that...
|
Simon Goldschmidt <goldsimon> |
Thu 21 Sep 2017 06:51:04 PM UTC, comment #1: Don't know why, but again, this patch didn't apply for me.
|
Simon Goldschmidt <goldsimon> |
Thu 07 Sep 2017 04:43:13 PM UTC, original submission:
Hello,
|
Joan Lledó <jllledo> |
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 8 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2017-10-16 | jcunningham | Attached File | - | Added 0001-sockets-poll-Open-Group-clean-ups.patch, #42171 | |
2017-10-08 | jcunningham | Attached File | - | Added 0001-sockets-poll-clean-ups.patch, #42092 | |
2017-09-21 | goldsimon | Status | None | Done | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
Planned Release | None | 2.1.0 | |||
2017-09-21 | goldsimon | Attached File | - | Added 0001-PATCH-New-sockets-function-lwip_poll.patch, #41860 | |
2017-09-07 | jllledo | Attached File | - | Added 0001-New-sockets-function-lwip_poll.patch, #41758 |
Powered by Savane 3.13-f8d8.
Corresponding source code
I didn't see any change in git. I went ahead and pushed my version of the commit (9c175835d5eab03c8b4213bb02a6e83f9925cf44) thanks!