diff options
author | Gabriel Caruso <carusogabriel34@gmail.com> | 2018-08-19 01:32:00 -0300 |
---|---|---|
committer | Peter Kokot <peterkokot@gmail.com> | 2018-12-03 01:22:14 +0100 |
commit | cdd8368d6f3f000a2e30a6be9976c5539f10fe85 (patch) | |
tree | 8f680d0c1fb552fdcbdbf3d49427c811fa789bf9 /ext/imap/php_imap.c | |
parent | 6c16f9b69c1951392d5d0ac89d3fbd2b318af118 (diff) | |
download | php-git-cdd8368d6f3f000a2e30a6be9976c5539f10fe85.tar.gz |
Clean up unnecessary ternary expressions and simplify some returns
- Simplify conditions
- Use ZEND_HASH_APPLY_* instead of hard-coded booleans
- Use ZEND_NORMALIZE_BOOL
- Drop sign in favor of ZEND_NORMALIZE_BOOL
Diffstat (limited to 'ext/imap/php_imap.c')
-rw-r--r-- | ext/imap/php_imap.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/ext/imap/php_imap.c b/ext/imap/php_imap.c index 9e626a4cfa..2863657827 100644 --- a/ext/imap/php_imap.c +++ b/ext/imap/php_imap.c @@ -4074,11 +4074,8 @@ int _php_imap_mail(char *to, char *subject, char *message, char *headers, char * } fprintf(sendmail, "\n%s\n", message); ret = pclose(sendmail); - if (ret == -1) { - return 0; - } else { - return 1; - } + + return ret != -1; } else { php_error_docref(NULL, E_WARNING, "Could not execute mail delivery program"); return 0; |