Compiler warning in osrf_utf8.c

[Jason Stephenson]

Compiling osrf_utf8.c generates the following warning:

osrf_utf8.c:510:29: warning: utf8_char may be used uninitialized in this function [-Wuninitialized]

[Dan Scott (denials)]

Also, using clang as the compiler results in 4 warnings like the following:

osrf_list.c:106:23: warning: comparison of unsigned expression < 0 is always false [-Wtautological-compare]
        if(!list || position < 0) return NULL;
                    ~~~~~~~~ ^ ~

(Explanation: "position" is an unsigned int; thus the comparison to < 0 can never evaluate to true).

[Dan Scott (denials)]

Pushed two commits, one to fix each warning, to user/dbs/lp954059_compiler_warnings

[Galen Charlton (gmc)]

collab/gmcharlt/lp954059_compiler_warnings in the OpenSRF working repo contains sign-offs of the two patches plus a follow-up to clean up some callers of osrfList* functions that should be using an unsigned for list indexes.

[Dan Scott (denials)]

Thanks, Galen! Pushed to master and rel_2_1.