comparison src/core/ngx_resolver.c @ 4474:41f640a693de

Time parsing cleanup. Nuke NGX_PARSE_LARGE_TIME, it's not used since 0.6.30. The only error ngx_parse_time() can currently return is NGX_ERROR, check it explicitly and make sure to cast it to appropriate type (either time_t or ngx_msec_t) to avoid signedness warnings on platforms with unsigned time_t (notably QNX).
author Maxim Dounin <mdounin@mdounin.ru>
date Mon, 13 Feb 2012 15:41:11 +0000
parents d620f497c50f
children be6c250b827b
comparison
equal deleted inserted replaced
4473:3b877a45d343 4474:41f640a693de
158 s.len = names[i].len - 6; 158 s.len = names[i].len - 6;
159 s.data = names[i].data + 6; 159 s.data = names[i].data + 6;
160 160
161 r->valid = ngx_parse_time(&s, 1); 161 r->valid = ngx_parse_time(&s, 1);
162 162
163 if (r->valid == NGX_ERROR) { 163 if (r->valid == (time_t) NGX_ERROR) {
164 ngx_conf_log_error(NGX_LOG_EMERG, cf, 0, 164 ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
165 "invalid parameter: %V", &names[i]); 165 "invalid parameter: %V", &names[i]);
166 return NULL; 166 return NULL;
167 } 167 }
168 168