diff options
| author | Klaus Silveira <contact@klaussilveira.com> | 2012-04-02 22:54:57 -0300 |
|---|---|---|
| committer | Klaus Silveira <contact@klaussilveira.com> | 2012-04-02 22:54:57 -0300 |
| commit | 16bd11062ec87b1478321c964d5739a9887b8ae0 (patch) | |
| tree | afcba2b4000288020f59618c0605e851edfaa2a4 /main | |
| parent | 14f120dbd9caf25704f4d4f3c9b36729d954c6e5 (diff) | |
| download | php-git-16bd11062ec87b1478321c964d5739a9887b8ae0.tar.gz | |
Small performance improvement. The current code is correct, but if it is used inside a long loop or long strings, it's inefficient.
Diffstat (limited to 'main')
| -rw-r--r-- | main/rfc1867.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/main/rfc1867.c b/main/rfc1867.c index 5da3a99355..3ca2c1131b 100644 --- a/main/rfc1867.c +++ b/main/rfc1867.c @@ -408,7 +408,7 @@ static int multipart_buffer_headers(multipart_buffer *self, zend_llist *header T /* get lines of text, or CRLF_CRLF */ - while( (line = get_line(self TSRMLS_CC)) && strlen(line) > 0 ) + while( (line = get_line(self TSRMLS_CC)) && line[0] != '\0' ) { /* add header to table */ char *key = line; @@ -979,7 +979,7 @@ SAPI_API SAPI_POST_HANDLER_FUNC(rfc1867_post_handler) /* {{{ */ continue; } - if (strlen(filename) == 0) { + if (filename[0] == '\0') { #if DEBUG_FILE_UPLOAD sapi_module.sapi_error(E_NOTICE, "No file uploaded"); #endif @@ -1063,12 +1063,12 @@ SAPI_API SAPI_POST_HANDLER_FUNC(rfc1867_post_handler) /* {{{ */ if (!cancel_upload && !end) { #if DEBUG_FILE_UPLOAD - sapi_module.sapi_error(E_NOTICE, "Missing mime boundary at the end of the data for file %s", strlen(filename) > 0 ? filename : ""); + sapi_module.sapi_error(E_NOTICE, "Missing mime boundary at the end of the data for file %s", filename[0] != '\0' ? filename : ""); #endif cancel_upload = UPLOAD_ERROR_C; } #if DEBUG_FILE_UPLOAD - if (strlen(filename) > 0 && total_bytes == 0 && !cancel_upload) { + if (filename[0] != '\0' && total_bytes == 0 && !cancel_upload) { sapi_module.sapi_error(E_WARNING, "Uploaded file size 0 - file [%s=%s] not saved", param, filename); cancel_upload = 5; } |