Skip to content

Commit 9309f2b

Browse files
committed
- fix build and warning (introduced by my last commit)
1 parent 294401c commit 9309f2b

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

ext/imap/php_imap.c

+5-6
Original file line numberDiff line numberDiff line change
@@ -1276,19 +1276,18 @@ PHP_FUNCTION(imap_append)
12761276
int folder_len, message_len, internal_date_len = 0, flags_len = 0;
12771277
pils *imap_le_struct;
12781278
STRING st;
1279-
1280-
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rss|ss", &streamind, &folder, &folder_len, &message, &message_len, &flags, &flags_len, &internal_date, &internal_date_len) == FAILURE) {
1281-
return;
1282-
}
1283-
12841279
char* regex = "/[0-3][0-9]-((Jan)|(Feb)|(Mar)|(Apr)|(May)|(Jun)|(Jul)|(Aug)|(Sep)|(Oct)|(Nov)|(Dec))-[0-9]{4} [0-2][0-9]:[0-5][0-9]:[0-5][0-9] [+-][0-9]{4}/";
1285-
int regex_len = strlen(regex);
1280+
const int regex_len = strlen(regex);
12861281
pcre_cache_entry *pce; /* Compiled regex */
12871282
zval *subpats = NULL; /* Parts (not used) */
12881283
long regex_flags = 0; /* Flags (not used) */
12891284
long start_offset = 0; /* Start offset (not used) */
12901285
int global = 0;
12911286

1287+
if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rss|ss", &streamind, &folder, &folder_len, &message, &message_len, &flags, &flags_len, &internal_date, &internal_date_len) == FAILURE) {
1288+
return;
1289+
}
1290+
12921291
if (internal_date) {
12931292
/* Make sure the given internal_date string matches the RFC specifiedformat */
12941293
if ((pce = pcre_get_compiled_regex_cache(regex, regex_len TSRMLS_CC))== NULL) {

0 commit comments

Comments
 (0)