COMMON Fixes to return values, errno, leaks
    
    Started looking at the ticket #107 related to
    traverse functions. Realized that the return values
    are not consistent. That ovelapped with the work
    that I wanted to do for ticket #103 - errno cleanup.
    So I (across collection, INI and ELAPI):
    * Made the return codes consistent (where found)
    * Removed errno where it is not needed
    While was testing used valgrind and found a nasty
    problem when the value was added to collection with
    overwriting duplicates the count was decreased improperly.
    Fixing collection.c to not decrease count made
    valgrind happy. While I was debugging this
    I also spotted several build warnings in trace
    statements when the " exp ? v1 : v2 " was used.
    Fixed those.
    In ini_config.c there was a trace stament that used
    variable after it was freed. Removed trace stament.