From c3cac7a521857747771f661d607004f58470e0cc Mon Sep 17 00:00:00 2001 From: Mattias Andrée Date: Sat, 3 Apr 2021 10:31:02 +0200 Subject: Use strcmpnul MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Mattias Andrée --- set-contact-organisations.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) (limited to 'set-contact-organisations.c') 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; } } -- cgit v1.2.3-70-g09d2