Merge pull request #121 from phord/master

brentru · web-flow · commit 0a7b4313a24f · 2020-06-02T10:44:29.000-04:00
Don't compare nullptr string - Fixes #120
diff --git a/src/AdafruitIO_Group.cpp b/src/AdafruitIO_Group.cpp
@@ -345,7 +345,7 @@ void AdafruitIO_Group::call(AdafruitIO_Data *d) {
 
   while (cur_cb) {
 
-    if (strcmp(cur_cb->feed, d->feedName()) == 0 || cur_cb->feed == NULL) {
+    if (cur_cb->feed == NULL || strcmp(cur_cb->feed, d->feedName()) == 0) {
       cur_cb->dataCallback(d);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -345,7 +345,7 @@ void AdafruitIO_Group::call(AdafruitIO_Data *d) {`
`345`	`345`
`346`	`346`	`while (cur_cb) {`
`347`	`347`
`348`		`- if (strcmp(cur_cb->feed, d->feedName()) == 0 \|\| cur_cb->feed == NULL) {`
	`348`	`+ if (cur_cb->feed == NULL \|\| strcmp(cur_cb->feed, d->feedName()) == 0) {`
`349`	`349`	`cur_cb->dataCallback(d);`
`350`	`350`	`}`
`351`	`351`