From 45458fe9eb6fedee95621a607fd94077cdb852e0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Beat=20K=C3=BCng?= Date: Fri, 19 Jan 2018 11:48:42 +0100 Subject: [PATCH] uORBManager: add comment about not having to set the priority --- src/modules/uORB/uORBManager.cpp | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/src/modules/uORB/uORBManager.cpp b/src/modules/uORB/uORBManager.cpp index 63f7880816..36c26ff6dc 100644 --- a/src/modules/uORB/uORBManager.cpp +++ b/src/modules/uORB/uORBManager.cpp @@ -435,6 +435,17 @@ int uORB::Manager::node_open } } + /* + else if (advertiser) { + * We have a valid fd and are an advertiser. + * This can happen if the topic is already subscribed/published, and orb_advertise() is called, + * where instance==nullptr. + * We would need to set the priority here (via px4_ioctl(fd, ...) and a new IOCTL), but orb_advertise() + * uses ORB_PRIO_DEFAULT, and a subscriber also creates the node with ORB_PRIO_DEFAULT. So we don't need + * to do anything here. + } + */ + if (fd < 0) { errno = EIO; return ERROR;