From d61e6adfcc82d4a1b3b3882f07b39c77b42eafc1 Mon Sep 17 00:00:00 2001
From: Josh Haberman <jhaberman@gmail.com>
Date: Thu, 20 Aug 2015 16:41:32 -0700
Subject: Return TypedData_Wrap_Struct directly.

Change-Id: I6cf77f01370204ad4bc7b345a040a9a3de1706a0
---
 ruby/ext/google/protobuf_c/map.c            | 3 +--
 ruby/ext/google/protobuf_c/repeated_field.c | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

(limited to 'ruby')

diff --git a/ruby/ext/google/protobuf_c/map.c b/ruby/ext/google/protobuf_c/map.c
index 5043f395..92fc7286 100644
--- a/ruby/ext/google/protobuf_c/map.c
+++ b/ruby/ext/google/protobuf_c/map.c
@@ -167,10 +167,9 @@ void Map_free(void* _self) {
 
 VALUE Map_alloc(VALUE klass) {
   Map* self = ALLOC(Map);
-  VALUE ret = TypedData_Wrap_Struct(klass, &Map_type, self);
   memset(self, 0, sizeof(Map));
   self->value_type_class = Qnil;
-  return ret;
+  return TypedData_Wrap_Struct(klass, &Map_type, self);
 }
 
 static bool needs_typeclass(upb_fieldtype_t type) {
diff --git a/ruby/ext/google/protobuf_c/repeated_field.c b/ruby/ext/google/protobuf_c/repeated_field.c
index 72687f27..83afbc91 100644
--- a/ruby/ext/google/protobuf_c/repeated_field.c
+++ b/ruby/ext/google/protobuf_c/repeated_field.c
@@ -606,13 +606,12 @@ void RepeatedField_free(void* _self) {
  */
 VALUE RepeatedField_alloc(VALUE klass) {
   RepeatedField* self = ALLOC(RepeatedField);
-  VALUE ret = TypedData_Wrap_Struct(klass, &RepeatedField_type, self);
   self->elements = NULL;
   self->size = 0;
   self->capacity = 0;
   self->field_type = -1;
   self->field_type_class = Qnil;
-  return ret;
+  return TypedData_Wrap_Struct(klass, &RepeatedField_type, self);
 }
 
 VALUE RepeatedField_init(int argc, VALUE* argv, VALUE self) {
-- 
cgit v1.2.3