split_args(): Do not insist on checking the return value.
authorAndre Noll <maan@systemlinux.org>
Mon, 25 Jun 2012 18:59:17 +0000 (20:59 +0200)
committerAndre Noll <maan@systemlinux.org>
Mon, 25 Jun 2012 18:59:17 +0000 (20:59 +0200)
commitcc858ad54f6f5d5d35ea6473b92aad9186c8e363
tree30d00cf547e101367cb84b82c9bccc2064624843
parentc6efe1f82dce725cbf20ca56a6d9e4e493351acd
split_args(): Do not insist on checking the return value.

Currently, split_args() of string.c has the __must_check attribute
which instructs gcc to warn whenever the return value of this function
is ignored by the caller.

However, since the returned array is NULL terminated anyway, there
are situations the return value my safely be ignored, for example if
the returned array is passed to execvp(), which does not receive a
length argument.

This patch removes the __must_check attribute and fixes a "set but
not used" warning on newer gcc versions.
select.c
string.c
string.h