Removed the unsafe nxt_memcmp() wrapper for memcmp(3).

The casts are unnecessary, since memcmp(3)'s arguments are 'void *'.
It might have been necessary in the times of K&R, where 'void *' didn't
ex

Removed the unsafe nxt_memcmp() wrapper for memcmp(3).

The casts are unnecessary, since memcmp(3)'s arguments are 'void *'.
It might have been necessary in the times of K&R, where 'void *' didn't
exist. Nowadays, it's unnecessary, and _very_ unsafe, since casts can
hide all classes of bugs by silencing most compiler warnings.

The changes from nxt_memcmp() to memcmp(3) were scripted:

$ find src/ -type f \
| grep '\.[ch]$' \
| xargs sed -i 's/nxt_memcmp/memcmp/'

Reviewed-by: Andrew Clayton <a.clayton@nginx.com>
Signed-off-by: Alejandro Colomar <alx@nginx.com>

show more ...

Implemented basic statistics API.

Constified numerous function parameters.

As was pointed out by the cppcheck[0] static code analysis utility we
can mark numerous function parameters as 'const'. This acts as a hint to
the compiler a

Constified numerous function parameters.

As was pointed out by the cppcheck[0] static code analysis utility we
can mark numerous function parameters as 'const'. This acts as a hint to
the compiler about our intentions and the compiler will tell us when we
deviate from them.

[0]: https://cppcheck.sourceforge.io/

show more ...

Added new array APIs that also work with non-arrays.

Similar to how C pointers to variables can always be considered as
pointers to the first element of an array of size 1 (see the
following code fo

Added new array APIs that also work with non-arrays.

Similar to how C pointers to variables can always be considered as
pointers to the first element of an array of size 1 (see the
following code for an example of how they are equivalent),
treating non-NXT_CONF_VALUE_ARRAY as if they were
NXT_CONF_VALUE_ARRAYs of size 1 allows for simpler and more
generic code.

void foo(ptrdiff_t sz, int arr[sz])
{
for (ptrdiff_t i = 0; i < sz; i++)
arr[i] = 0;
}

void bar(void)
{
int x;
int y[1];

foo(1, &x);
foo(1, y);
}

nxt_conf_array_elements_count_or_1():
Similar to nxt_conf_array_elements_count().
Return a size of 1 when input is non-array, instead of
causing undefined behavior. That value (1) makes sense
because it will be used as the limiter of a loop that
loops over the array and calls
nxt_conf_get_array_element_or_itself(), which will return
a correct element for such loops.

nxt_conf_get_array_element_or_itself():
Similar to nxt_conf_get_array_element().
Return the input pointer unmodified (i.e., a pointer to
the unique element of a hypothetical array), instead of
returning NULL, which wasn't very useful.

nxt_conf_array_qsort():
Since it's a no-op for non-arrays, this API can be reused.

show more ...

Added 'const' for read-only function parameter.

That parameter is not being modified in the function. Make it
'const' to allow passing 'static const' variables.

Fixed build with Clang 10, broken by 32578e837322.

This silences the -Wimplicit-int-float-conversion warning.

Configuration: support for rational numbers.

Configuration: fixed comments parsing.

Unclosed multi-line comments and "/" at the end of JSON shouldn't be allowed.

Configuration: stripping comments from the input JSON.

This allows to have JavaScript-like comments in the uploading JSON.

Fixed passing false in namespace flags.

This patch closes #328 in github.

Configuration: added ability to modify object members with slashes.

Example:

PUT/POST/DELETE /config/listeners/unix:%2Fpath%2Fto%2Fsocket

This follows a49ee872e83d.

Configuration: added ability to access object members with slashes.

Now URI encoding can be used to escape "/" in the request path:

GET /config/listeners/unix:%2Fpath%2Fto%2Fsocket/

Configuration: support for POST operations on arrays.

It allows to add an array element without specifying the index.

Configuration: support for manipulations of array elements.

Now PUT and DELETE operations also work on elements.
This closes #242 issue on GitHub.

Refactored nxt_conf_op_compile().

Now index is always initialized for create operations.

The changes in nxt_conf_op_compile() simplify adding upcoming support of
operations with arrays.

No functio

Refactored nxt_conf_op_compile().

Now index is always initialized for create operations.

The changes in nxt_conf_op_compile() simplify adding upcoming support of
operations with arrays.

No functional changes.

show more ...

Removed unused field from nxt_conf_op_s.

Introduced nxt_conf_array_qsort().

Introduced nxt_conf_array_elements_count().

Fixed discovering of modules on 64-bit big-endian systems.

The nxt_conf_map_object() function used nxt_int_t for NXT_CONF_MAP_INT, which
was 8 bytes long on 64-bit systems.

But the nxt_port_main_st

Fixed discovering of modules on 64-bit big-endian systems.

The nxt_conf_map_object() function used nxt_int_t for NXT_CONF_MAP_INT, which
was 8 bytes long on 64-bit systems.

But the nxt_port_main_start_worker_handler() used it to map into the int field
of the nxt_common_app_conf_t structure, which was 4 bytes. As the result, on
a 64-bit big-endian system all the meaningful module type numbers were assigned
into the gap above the "type" field.

The bug was discovered on IBM/S390x.

show more ...

Controller: addressing of array elements in requests.

Controller: certificates storage interface.

Deduplicated string value initializations.

Introduced nxt_length() macro.

Style: capitalized letters in hexadecimal literals.

nxt_lvlhsh_each() refactoring and nxt_lvlhsh_each_init().