thingsboard-client-sdk icon indicating copy to clipboard operation
thingsboard-client-sdk copied to clipboard

More flexible RPC and Attributes subscription.

Open shkolnick-kun opened this issue 4 years ago • 3 comments

Hi TB team!

I've recently faced a problem with ThingsBoard-Arduino-MQTT-SDK when tried to connect "Knob control" widget to my esp8266 board. So I changed RPC callback signature to make callbcaks more flexible and enable any RPC-widgets to interact with ThingsBoard-Arduino-MQTT-SDK.

I've also added Attributes subscription methods to ThingsBoardSized class.

I hope you merge these changes.

Best regards, Paul.

shkolnick-kun avatar Dec 03 '20 05:12 shkolnick-kun

А в bool Attr_Subscribe(const Attr_Callback callback) callback может быть NULL?

Attr_Callback для массива

Там только один колбэк, массив как раз не нужен

NotNull_Attr_Callback для параметров функций, где callback обязан быть не NULL.

А вот это оно! С другой стороны, - там проверка на ноль перед вызовом.

shkolnick-kun avatar Dec 03 '20 10:12 shkolnick-kun

Ой, спутал RPC_Callback и Attr_Callback. Да, там нет массива.

fsb4000 avatar Dec 03 '20 10:12 fsb4000

Убрал topic, добавил тип данных, чтоб нельзя было NULL при подписке дать.

shkolnick-kun avatar Dec 03 '20 13:12 shkolnick-kun