]> git.ipfire.org Git - thirdparty/dovecot/core.git/commit
global - migrate more complicated strncmp expressions to str_begins
authorPhil Carmody <phil@dovecot.fi>
Thu, 11 Jan 2018 13:35:42 +0000 (15:35 +0200)
committerTimo Sirainen <timo.sirainen@dovecot.fi>
Thu, 26 Apr 2018 09:41:29 +0000 (09:41 +0000)
commit5cef037a24b7c2d9b28a322d202833a65b31bbbf
treef2151f93a5af2243bffe7eab6f4de3bd4756d5de
parentc05976729405180c565aad464cb494c731a640b3
global - migrate more complicated strncmp expressions to str_begins

Simplify a bunch of verbose var=strlen(); strncmp(,,var) calls.
Fortunately, all of these examples use the length variable, so no
"unused value" warnings occur.

--- 8< --- strncmp2.cocci ---
@@
expression e1, e2;
identifier i2;
@@

  i2 = strlen(e2)
  ...
- strncmp(e1, e2, i2) == 0
+ str_begins(e1, e2)

@@
expression e1, e2;
identifier i2;
@@

  i2 = strlen(e2)
  ...
- strncmp(e1, e2, i2) != 0
+ !str_begins(e1, e2)

@@
expression e1, e2;
identifier i1;
@@

  i1 = strlen(e1)
  ...
- strncmp(e1, e2, i1) == 0
+ str_begins(e2, e1)

@@
expression e1, e2;
identifier i1;
@@

  i1 = strlen(e1)
  ...
- strncmp(e1, e2, i1) != 0
+ !str_begins(e2, e1)

--- 8< ---------------------

Signed-off-by: Phil Carmody <phil@dovecot.fi>
16 files changed:
src/auth/auth-cache.c
src/config/old-set-parser.c
src/doveadm/doveadm-cmd.c
src/doveadm/doveadm-mail.c
src/doveadm/dsync/dsync-brain-mailbox-tree.c
src/imap/cmd-notify.c
src/imap/cmd-rename.c
src/imap/imap-notify.c
src/lib-storage/index/imapc/imapc-list.c
src/lib-storage/index/imapc/imapc-storage.c
src/lib-storage/index/index-search-mime.c
src/lib-storage/index/shared/shared-list.c
src/lib-storage/list/mailbox-list-fs-iter.c
src/lib-storage/list/mailbox-list-index-backend.c
src/lib-storage/list/mailbox-list-iter.c
src/lib-storage/mailbox-list.c