Remove unnecessary copying of pointer before return in cbor_load() (#223

)

When building with TI ARM Clang compiler the initialization of
result_item generated a warning because of the goto statements earlier:

"error #548-D: transfer of control bypasses initialization of..."

This warning can be suppressed but since the copying of context.root is
unnecessary, we can simplify things and return the member right away.

src/cbor.c

@@ -100,9 +100,7 @@ cbor_item_t *cbor_load(cbor_data source, size_t source_size,
     }
   } while (stack.size > 0);
 
-  /* Move the result before free */
-  cbor_item_t *result_item = context.root;
-  return result_item;
+  return context.root;
 
 error:
   result->error.position = result->read;