This reverts commit 3ea491c0.

Big thanks at mvollmer (https://sourceforge.net/p/cjson/bugs/45/)

Do not always build static and dynamic

ARM Compilation

don't ignore malloc failure in cJSON_PrintBuffered

Currently, the static and dynamic version of the libraries are always
built as add_library is called twice. Instead, this patch will use the
standard CMake variable BUILD_SHARED_LIBS to know if the static or the
dynamic version must be built.
Signed-off-by: NFabrice Fontaine <fabrice.fontaine@orange.com>

Reformat cJSON_Utils.c and test.c

NOTE: This can change the assembly slightly, in my case it reordered two
instructions. This is due to the change from:

    which = (10 * which) + *pointer++ - '0';

to

    which = (10 * which) + (*pointer++ - '0');

This means that after the change, the subtraction runs before the
addition instead of after that. That shouldn't change the behavior
though.

Remove redundant test from cJSON_AddItemToArray

The test for whether "c" is null is redundant.  Since this is the else case of "(!c)", and the only way "c" is changed is by the instruction assigning it "c->next" on the condition here, simply verifying that "c->next" isn't null is sufficient.