aboutsummaryrefslogtreecommitdiffstats
path: root/set-contact-organisations.c
diff options
context:
space:
mode:
Diffstat (limited to 'set-contact-organisations.c')
-rw-r--r--set-contact-organisations.c13
1 files changed, 6 insertions, 7 deletions
diff --git a/set-contact-organisations.c b/set-contact-organisations.c
index 17db004..d1fbeaf 100644
--- a/set-contact-organisations.c
+++ b/set-contact-organisations.c
@@ -53,8 +53,7 @@ main(int argc, char *argv[])
for (; contact.organisations[i]; i++);
} else if (update_title) {
for (; contact.organisations[i]; i++) {
- if (contact.organisations[i]->organisation &&
- !strcmp(contact.organisations[i]->organisation, argv[1])) {
+ if (!strcmpnul(contact.organisations[i]->organisation, argv[1])) {
free(contact.organisations[i]->title);
contact.organisations[i]->title = estrdup(argv[2]);
goto save;
@@ -62,7 +61,7 @@ main(int argc, char *argv[])
}
} else if (update_organisation) {
for (; contact.organisations[i]; i++) {
- if (contact.organisations[i]->title && !strcmp(contact.organisations[i]->title, argv[2])) {
+ if (!strcmpnul(contact.organisations[i]->title, argv[2])) {
free(contact.organisations[i]->organisation);
contact.organisations[i]->organisation = estrdup(argv[1]);
goto save;
@@ -70,16 +69,16 @@ main(int argc, char *argv[])
}
} else if (argc == 3) {
for (; contact.organisations[i]; i++)
- if (contact.organisations[i]->organisation && !strcmp(contact.organisations[i]->organisation, argv[1]))
- if (contact.organisations[i]->title && !strcmp(contact.organisations[i]->title, argv[2]))
+ if (!strcmpnul(contact.organisations[i]->organisation, argv[1]))
+ if (!strcmpnul(contact.organisations[i]->title, argv[2]))
break;
} else if (remove_by_organisation) {
for (; contact.organisations[i]; i++)
- if (contact.organisations[i]->organisation && !strcmp(contact.organisations[i]->organisation, argv[1]))
+ if (!strcmpnul(contact.organisations[i]->organisation, argv[1]))
break;
} else {
for (; contact.organisations[i]; i++)
- if (contact.organisations[i]->title && !strcmp(contact.organisations[i]->title, argv[1]))
+ if (!strcmpnul(contact.organisations[i]->title, argv[1]))
break;
}
}