comparison src/http/ngx_http_script.c @ 6861:e4590dfd97ff

Win32: support 64-bit compilation with MSVC. There are lots of C4244 warnings (conversion from 'type1' to 'type2', possible loss of data), so they were disabled. The same applies to C4267 warnings (conversion from 'size_t' to 'type', possible loss of data), most notably - conversion from ngx_str_t.len to ngx_variable_value_t.len (which is unsigned:28). Additionally, there is at least one case when it is not possible to fix the warning properly without introducing win32-specific code: recv() on win32 uses "int len", while POSIX defines "size_t len". The ssize_t type now properly defined for 64-bit compilation with MSVC. Caught by warning C4305 (truncation from '__int64' to 'ssize_t'), on "cutoff = NGX_MAX_SIZE_T_VALUE / 10" in ngx_atosz()). Several C4334 warnings (result of 32-bit shift implicitly converted to 64 bits) were fixed by adding explicit conversions. Several C4214 warnings (nonstandard extension used: bit field types other than int) in ngx_http_script.h fixed by changing bit field types from uintptr_t to unsigned.
author Maxim Dounin <mdounin@mdounin.ru>
date Sat, 24 Dec 2016 18:01:14 +0300
parents af642539cd53
children 39ff6939266e
comparison
equal deleted inserted replaced
6860:f18c285c2e59 6861:e4590dfd97ff
354 #if (NGX_PCRE) 354 #if (NGX_PCRE)
355 ngx_uint_t n; 355 ngx_uint_t n;
356 356
357 n = sc->source->data[i] - '0'; 357 n = sc->source->data[i] - '0';
358 358
359 if (sc->captures_mask & (1 << n)) { 359 if (sc->captures_mask & ((ngx_uint_t) 1 << n)) {
360 sc->dup_capture = 1; 360 sc->dup_capture = 1;
361 } 361 }
362 362
363 sc->captures_mask |= 1 << n; 363 sc->captures_mask |= (ngx_uint_t) 1 << n;
364 364
365 if (ngx_http_script_add_capture_code(sc, n) != NGX_OK) { 365 if (ngx_http_script_add_capture_code(sc, n) != NGX_OK) {
366 return NGX_ERROR; 366 return NGX_ERROR;
367 } 367 }
368 368