[riot-notifications] [RIOT-OS/RIOT] shell_commands: adapt ping6 to original-ping-like implementation (#9523)

Martine Lenders notifications at github.com
Fri Feb 1 09:46:32 CET 2019


miri64 commented on this pull request.



> +
+        msg_receive(&msg);
+        switch (msg.type) {
+            case GNRC_NETAPI_MSG_TYPE_RCV: {
+                _handle_reply(&data, msg.content.ptr);
+                gnrc_pktbuf_release(msg.content.ptr);
+                break;
+            }
+            case _SEND_NEXT_PING:
+                _pinger(&data);
+                break;
+            case _PING_FINISH:
+                goto finish;
+            default:
+                /* requeue wrong packets */
+                msg_send(&msg, sched_active_pid);

Ok, I had in memory, that this line existed because someone asked for it, but apparently I introduced it myself in https://github.com/RIOT-OS/RIOT/pull/2555. Anyway, given that this is `main` and that we didn't face any problems with this until now, I think the line is here correctly.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/9523#discussion_r252970788
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190201/4f94fa69/attachment.html>


More information about the notifications mailing list