Merge branch 'bugfix/esp_log_level_set_strcmp' into 'master'

projectgus · projectgus · commit 2e33dbf7662e · 2018-01-31T09:58:21.000+08:00
Fix esp_log_level_set string matching (github PR 1557)

See merge request idf/esp-idf!1872
diff --git a/components/log/log.c b/components/log/log.c
@@ -164,7 +164,7 @@ void esp_log_level_set(const char* tag, esp_log_level_t level)
 #ifdef LOG_BUILTIN_CHECKS
         assert(i == 0 || s_log_cache[(i - 1) / 2].generation < s_log_cache[i].generation);
 #endif
-        if (s_log_cache[i].tag == tag) {
+        if (strcmp(s_log_cache[i].tag,tag) == 0) {
             s_log_cache[i].level = level;
             break;
         }

Original file line number	Diff line number	Diff line change
`@@ -164,7 +164,7 @@ void esp_log_level_set(const char* tag, esp_log_level_t level)`
`164`	`164`	`#ifdef LOG_BUILTIN_CHECKS`
`165`	`165`	`assert(i == 0 \|\| s_log_cache[(i - 1) / 2].generation < s_log_cache[i].generation);`
`166`	`166`	`#endif`
`167`		`- if (s_log_cache[i].tag == tag) {`
	`167`	`+ if (strcmp(s_log_cache[i].tag,tag) == 0) {`
`168`	`168`	`s_log_cache[i].level = level;`
`169`	`169`	`break;`
`170`	`170`	`}`