diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2017-02-13 15:57:54 +1100 |
---|---|---|
committer | David Gibson <david@gibson.dropbear.id.au> | 2017-02-13 15:57:54 +1100 |
commit | 120775eb1cf39f8dcecd695c3ff1cfef8aeb669d (patch) | |
tree | 6b6102ee68ad131a407852a8cfb665fadae35057 | |
parent | 852e9ecbe1976927057402f8a8f71ee8e8a49098 (diff) | |
download | dtc-120775eb1cf39f8dcecd695c3ff1cfef8aeb669d.zip dtc-120775eb1cf39f8dcecd695c3ff1cfef8aeb669d.tar.gz dtc-120775eb1cf39f8dcecd695c3ff1cfef8aeb669d.tar.bz2 |
dtc: Use streq() in preference to strcmp()
dtc defines a streq() (string equality) macro to avoid the easy confusion
of the sense of strcmp() comparison for equality. A few places where we
don't use it have slipped in, so remove them.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
-rw-r--r-- | checks.c | 2 | ||||
-rw-r--r-- | livetree.c | 4 |
2 files changed, 3 insertions, 3 deletions
@@ -325,7 +325,7 @@ static void check_property_name_chars_strict(struct check *c, continue; /* Certain names are whitelisted */ - if (strcmp(name, "device_type") == 0) + if (streq(name, "device_type")) continue; /* @@ -242,7 +242,7 @@ void delete_property_by_name(struct node *node, char *name) struct property *prop = node->proplist; while (prop) { - if (!strcmp(prop->name, name)) { + if (streq(prop->name, name)) { delete_property(prop); return; } @@ -275,7 +275,7 @@ void delete_node_by_name(struct node *parent, char *name) struct node *node = parent->children; while (node) { - if (!strcmp(node->name, name)) { + if (streq(node->name, name)) { delete_node(node); return; } |