fix more build warnings

Author: remicollet

enum.c

@@ -26,7 +26,7 @@ PHP_METHOD(protocolbuffers_enum, isValid)
 	return;
 #else
 	long value;
-	zval *result, *result2;
+	zval *result;
 
 	if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC,
 		"l", &value) == FAILURE) {
@@ -71,15 +71,15 @@ PHP_METHOD(protocolbuffers_enum, getName)
 	return;
 #else
 	long value;
-	zval *result, *result2;
+	zval *result;
 
 	if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC,
 		"l", &value) == FAILURE) {
 		return;
 	}
 
 	if (zend_call_method_with_0_params(NULL, EG(called_scope), NULL, "getenumdescriptor", &result)) {
-		zval *values, **entry, **key;
+		zval *values, **entry;
 		HashPosition pos;
 
 		if (!instanceof_function_ex(Z_OBJCE_P(result), php_protocol_buffers_enum_descriptor_class_entry, 0 TSRMLS_CC)) {

enum_descriptor_builder.c

@@ -27,7 +27,7 @@ PHP_METHOD(protocolbuffers_enum_descriptor_builder, __construct)
 PHP_METHOD(protocolbuffers_enum_descriptor_builder, addValue)
 {
 	zval *instance = getThis();
-	zval *value, **fields, *name, *value2;
+	zval *value, **fields, *name;
 	zend_bool force = 0;
 	char *property;
 	int property_len;

helper.c

@@ -35,7 +35,7 @@ static void php_protocolbuffers_helper_debug_zval(zval **value TSRMLS_DC)
 	zval *val = *value;
 
 	php_printf("{\n");
-	php_printf("  address: 0x%x,\n", (unsigned int)val);
+	php_printf("  address: 0x%lx,\n", (unsigned long)val);
 	php_printf("  type: %d,\n", val->type);
 	php_printf("  is_ref: %d,\n", PZVAL_IS_REF(val));
 	php_printf("  refcount: %d,\n", Z_REFCOUNT_PP(value));
@@ -44,14 +44,14 @@ static void php_protocolbuffers_helper_debug_zval(zval **value TSRMLS_DC)
 	php_printf("    double: %f,\n", val->value.dval);
 	if (val->type == 4) {
 		php_printf("    ht: {\n");
-		php_printf("      address: 0x%x,\n", (unsigned int)val->value.ht);
+		php_printf("      address: 0x%lx,\n", (unsigned long)val->value.ht);
 		php_printf("      num_of_elements: %d,\n", (unsigned int)val->value.ht->nNumOfElements);
 		php_printf("      next_free_elements: %d,\n", (unsigned int)val->value.ht->nNextFreeElement);
 		php_printf("    },\n");
 	}
 	php_printf("    object: {\n");
 	php_printf("      handle: 0x%x,\n", val->value.obj.handle);
-	php_printf("      handlers: 0x%x,\n", (unsigned int)val->value.obj.handlers);
+	php_printf("      handlers: 0x%lx,\n", (unsigned long)val->value.obj.handlers);
 	php_printf("    },\n");
 	php_printf("  }\n");
 	php_printf("}\n");

message.c

@@ -1571,7 +1571,6 @@ PHP_METHOD(protocolbuffers_message, clearExtension)
 
 	if (zend_hash_find(htt, n, n_len, (void **)&e) == SUCCESS) {
 		zval *tmp;
-		ulong hval;
 		if (is_mangled) {
 			efree(n);
 		}
@@ -1676,7 +1675,6 @@ PHP_METHOD(protocolbuffers_message, jsonSerialize)
 PHP_METHOD(protocolbuffers_message, toArray)
 {
 	zval *instance = getThis(), *result = NULL;
-	zend_class_entry **json;
 
 	if (php_protocolbuffers_jsonserialize(INTERNAL_FUNCTION_PARAM_PASSTHRU, 1, Z_OBJCE_P(instance), instance, &result) == 0) {
 		RETURN_ZVAL(result, 0, 1);

serializer.c

@@ -723,13 +723,13 @@ int php_protocolbuffers_fetch_element(INTERNAL_FUNCTION_PARAMETERS, php_protocol
 
 	if (zend_hash_find(hash, name, name_len, (void **)&tmp) == SUCCESS) {
 		*output = *tmp;
-		return 0;
 	} else {
 		if (scheme->required > 0) {
 			zend_throw_exception_ex(php_protocol_buffers_invalid_protocolbuffers_exception_class_entry, 0 TSRMLS_CC, "the class does not declared required property `%s`. probably you missed declaration", scheme->name);
 			return 1;
 		}
 	}
+	return 0;
 }
 
 int php_protocolbuffers_encode_message(INTERNAL_FUNCTION_PARAMETERS, zval *klass, php_protocolbuffers_scheme_container *container, php_protocolbuffers_serializer **serializer)

unknown_field_set.c

@@ -66,7 +66,7 @@ void php_protocolbuffers_unknown_field_set_properties_init(zval *object TSRMLS_D
 
 void php_protocolbuffers_unknown_field_clear(INTERNAL_FUNCTION_PARAMETERS, zval *instance)
 {
-	zval *fields = NULL, **prior_fields = NULL;
+	zval *fields = NULL;
 	char *name = {0};
 	int name_len = 0;