plamo 0.2.0 → 0.3.0

data/ext/plamo/plamo_request.h

@@ -2,6 +2,7 @@
 #define RUBY_PLAMO_REQUEST_H
 
 extern VALUE rb_cPlamoRequest;
+extern const rb_data_type_t rb_plamo_request_type;
 
 void Init_plamo_request(void);
 

data/ext/plamo/plamo_response.c

@@ -1,55 +1,57 @@
 #include "plamo.h"
-#include "wrapper.h"
 
 VALUE rb_cPlamoResponse;
 
-static void deallocate(Wrapper *wrapper) {
-  plamo_response_destroy(wrapper->inner);
-  free(wrapper);
+static void deallocate(void *plamo_response) {
+  plamo_response_destroy(plamo_response);
 }
 
+const rb_data_type_t rb_plamo_response_type = {
+  "Response",
+  {
+    NULL,
+    deallocate,
+    NULL,
+  },
+  NULL,
+  NULL,
+  0,
+};
+
 static VALUE allocate(VALUE klass) {
-  return Data_Wrap_Struct(klass, NULL, deallocate, malloc(sizeof(Wrapper)));
+  return TypedData_Wrap_Struct(klass, &rb_plamo_response_type, NULL);
 }
 
 static VALUE initialize(VALUE self) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  wrapper->inner = plamo_response_new();
+  DATA_PTR(self) = plamo_response_new();
   return self;
 }
 
 static VALUE status_code(VALUE self) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  return UINT2NUM(((PlamoResponse*)wrapper->inner)->status_code);
+  PlamoResponse *plamo_response;
+  TypedData_Get_Struct(self, PlamoResponse, &rb_plamo_response_type, plamo_response);
+  return UINT2NUM(plamo_response->status_code);
 }
 
 static VALUE set_status_code(VALUE self, VALUE code) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  ((PlamoResponse*)wrapper->inner)->status_code = FIX2UINT(code);
+  PlamoResponse *plamo_response;
+  TypedData_Get_Struct(self, PlamoResponse, &rb_plamo_response_type, plamo_response);
+  plamo_response->status_code = FIX2UINT(code);
   return code;
 }
 
 static VALUE header(VALUE self) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  VALUE rb_plamo_http_header = Data_Wrap_Struct(rb_cPlamoHttpHeader, NULL, free, malloc(sizeof(Wrapper)));
-  Wrapper *plamo_http_header_wrapper;
-  Data_Get_Struct(rb_plamo_http_header, Wrapper, plamo_http_header_wrapper);
-  plamo_http_header_wrapper->inner = ((PlamoResponse*)wrapper->inner)->header;
+  PlamoResponse *plamo_response;
+  TypedData_Get_Struct(self, PlamoResponse, &rb_plamo_response_type, plamo_response);
+  VALUE rb_plamo_http_header = TypedData_Wrap_Struct(rb_cPlamoHttpHeader, &rb_plamo_http_header_type, plamo_response->header);
   return rb_plamo_http_header;
 }
 
 static VALUE body(VALUE self) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  if (((PlamoResponse*)wrapper->inner)->body != NULL) {
-    VALUE rb_plamo_byte_array = Data_Wrap_Struct(rb_cPlamoByteArray, NULL, free, malloc(sizeof(Wrapper)));
-    Wrapper *plamo_byte_array_wrapper;
-    Data_Get_Struct(rb_plamo_byte_array, Wrapper, plamo_byte_array_wrapper);
-    plamo_byte_array_wrapper->inner = ((PlamoResponse*)wrapper->inner)->body;
+  PlamoResponse *plamo_response;
+  TypedData_Get_Struct(self, PlamoResponse, &rb_plamo_response_type, plamo_response);
+  if (plamo_response->body != NULL) {
+    VALUE rb_plamo_byte_array = TypedData_Wrap_Struct(rb_cPlamoByteArray, &rb_plamo_byte_array_type, plamo_response->body);
     return rb_plamo_byte_array;
   } else {
     return Qnil;
@@ -57,12 +59,11 @@ static VALUE body(VALUE self) {
 }
 
 static VALUE set_body(VALUE self, VALUE rb_body) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-
+  PlamoResponse *plamo_response;
+  TypedData_Get_Struct(self, PlamoResponse, &rb_plamo_response_type, plamo_response);
   if (strcmp("String", rb_class2name(rb_obj_class(rb_body))) == 0) {
     struct RString *rstring = RSTRING(rb_body);
-    ((PlamoResponse*)wrapper->inner)->body = plamo_byte_array_new(RSTRING_PTR(rstring), RSTRING_LEN(rstring));
+    plamo_response->body = plamo_byte_array_new(RSTRING_PTR(rstring), RSTRING_LEN(rstring));
   } else {
     VALUE *array = RARRAY_PTR(rb_body);
     size_t len = RARRAY_LEN(rb_body);
@@ -70,10 +71,9 @@ static VALUE set_body(VALUE self, VALUE rb_body) {
     for (size_t i = 0; i < len; i++) {
       buf[i] = NUM2CHR(array[i]);
     }
-    ((PlamoResponse*)wrapper->inner)->body = plamo_byte_array_new(buf, len);
+    plamo_response->body = plamo_byte_array_new(buf, len);
     free(array);
   }
-
   return self;
 }
 

data/ext/plamo/plamo_response.h

@@ -2,6 +2,7 @@
 #define RUBY_PLAMO_RESPONSE_H
 
 extern VALUE rb_cPlamoResponse;
+extern const rb_data_type_t rb_plamo_response_type;
 
 void Init_plamo_response(void);
 

data/ext/plamo/plamo_string_array.c

@@ -1,35 +1,53 @@
 #include "plamo.h"
-#include "wrapper.h"
 
 VALUE rb_cPlamoStringArray;
 
-static void deallocate(Wrapper *wrapper) {
-  plamo_string_array_destroy(wrapper->inner);
-  free(wrapper);
+static void deallocate(void *plamo_string_array) {
+  plamo_string_array_destroy(plamo_string_array);
 }
 
+const rb_data_type_t rb_plamo_string_array_type = {
+  "StringArray",
+  {
+    NULL,
+    deallocate,
+    NULL,
+  },
+  NULL,
+  NULL,
+  0,
+};
+
 static VALUE allocate(VALUE klass) {
-  return Data_Wrap_Struct(klass, NULL, deallocate, malloc(sizeof(Wrapper)));
+  return TypedData_Wrap_Struct(klass, &rb_plamo_string_array_type, NULL);
 }
 
 static VALUE initialize(VALUE self) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  wrapper->inner = plamo_string_array_new();
+  DATA_PTR(self) = plamo_string_array_new();
   return self;
 }
 
+static VALUE length(VALUE self) {
+  PlamoStringArray *plamo_string_array;
+  TypedData_Get_Struct(self, PlamoStringArray, &rb_plamo_string_array_type, plamo_string_array);
+  return SIZET2NUM(plamo_string_array_length(plamo_string_array));
+}
+
 static VALUE push(VALUE self, VALUE rb_string) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  plamo_string_array_add(wrapper->inner, StringValueCStr(rb_string));
+  PlamoStringArray *plamo_string_array;
+  TypedData_Get_Struct(self, PlamoStringArray, &rb_plamo_string_array_type, plamo_string_array);
+  if (OBJ_FROZEN(self)) {
+    rb_exc_raise(rb_eFrozenError);
+    return Qnil;
+  }
+  plamo_string_array_add(plamo_string_array, StringValueCStr(rb_string));
   return Qnil;
 }
 
 static VALUE first(VALUE self) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  const char *str = plamo_string_array_get_first(wrapper->inner);
+  PlamoStringArray *plamo_string_array;
+  TypedData_Get_Struct(self, PlamoStringArray, &rb_plamo_string_array_type, plamo_string_array);
+  const char *str = plamo_string_array_get_first(plamo_string_array);
   if (str != NULL) {
     return rb_str_new2(str);
   } else {
@@ -38,9 +56,9 @@ static VALUE first(VALUE self) {
 }
 
 static VALUE last(VALUE self) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  const char *str = plamo_string_array_get_last(wrapper->inner);
+  PlamoStringArray *plamo_string_array;
+  TypedData_Get_Struct(self, PlamoStringArray, &rb_plamo_string_array_type, plamo_string_array);
+  const char *str = plamo_string_array_get_last(plamo_string_array);
   if (str != NULL) {
     return rb_str_new2(str);
   } else {
@@ -49,9 +67,9 @@ static VALUE last(VALUE self) {
 }
 
 static VALUE get_at(VALUE self, VALUE index) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  const char *str = plamo_string_array_get_at(wrapper->inner, FIX2ULONG(index));
+  PlamoStringArray *plamo_string_array;
+  TypedData_Get_Struct(self, PlamoStringArray, &rb_plamo_string_array_type, plamo_string_array);
+  const char *str = plamo_string_array_get_at(plamo_string_array, FIX2ULONG(index));
   if (str != NULL) {
     return rb_str_new2(str);
   } else {
@@ -64,16 +82,16 @@ static void execute_each(const char *str) {
 }
 
 static VALUE each(VALUE self) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  plamo_string_array_for_each(wrapper->inner, execute_each);
+  PlamoStringArray *plamo_string_array;
+  TypedData_Get_Struct(self, PlamoStringArray, &rb_plamo_string_array_type, plamo_string_array);
+  plamo_string_array_for_each(plamo_string_array, execute_each);
   return Qnil;
 }
 
 static VALUE delete_at(VALUE self, VALUE index) {
-  Wrapper *wrapper;
-  Data_Get_Struct(self, Wrapper, wrapper);
-  if (plamo_string_array_remove_at(wrapper->inner, FIX2ULONG(index))) {
+  PlamoStringArray *plamo_string_array;
+  TypedData_Get_Struct(self, PlamoStringArray, &rb_plamo_string_array_type, plamo_string_array);
+  if (plamo_string_array_remove_at(plamo_string_array, FIX2ULONG(index))) {
     return Qtrue;
   } else {
     return Qfalse;
@@ -84,6 +102,8 @@ void Init_plamo_string_array(void) {
   rb_cPlamoStringArray = rb_define_class_under(rb_mPlamo, "StringArray", rb_cObject);
   rb_define_alloc_func(rb_cPlamoStringArray, allocate);
   rb_define_method(rb_cPlamoStringArray, "initialize", initialize, 0);
+  rb_define_method(rb_cPlamoStringArray, "length", length, 0);
+  rb_define_alias(rb_cPlamoStringArray, "size", "length");
   rb_define_method(rb_cPlamoStringArray, "push", push, 1);
   rb_define_method(rb_cPlamoStringArray, "first", first, 0);
   rb_define_method(rb_cPlamoStringArray, "last", last, 0);
@@ -91,4 +111,3 @@ void Init_plamo_string_array(void) {
   rb_define_method(rb_cPlamoStringArray, "each", each, 0);
   rb_define_method(rb_cPlamoStringArray, "delete_at", delete_at, 1);
 }
-

data/ext/plamo/plamo_string_array.h

@@ -2,6 +2,7 @@
 #define RUBY_PLAMO_STRING_ARRAY_H
 
 extern VALUE rb_cPlamoStringArray;
+extern const rb_data_type_t rb_plamo_string_array_type;
 
 void Init_plamo_string_array(void);
 

data/lib/plamo/version.rb

@@ -1,3 +1,3 @@
 module Plamo
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: plamo
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Shogo Otake
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-10-14 00:00:00.000000000 Z
+date: 2019-11-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -60,6 +60,7 @@ extensions:
 - ext/plamo/extconf.rb
 extra_rdoc_files: []
 files:
+- ".github/FUNDING.yml"
 - ".gitignore"
 - Gemfile
 - Gemfile.lock
@@ -75,6 +76,16 @@ files:
 - ext/plamo/plamo_app.h
 - ext/plamo/plamo_byte_array.c
 - ext/plamo/plamo_byte_array.h
+- ext/plamo/plamo_form_data.c
+- ext/plamo/plamo_form_data.h
+- ext/plamo/plamo_form_data_field.c
+- ext/plamo/plamo_form_data_field.h
+- ext/plamo/plamo_form_data_field_array.c
+- ext/plamo/plamo_form_data_field_array.h
+- ext/plamo/plamo_form_data_file.c
+- ext/plamo/plamo_form_data_file.h
+- ext/plamo/plamo_form_urlencoded.c
+- ext/plamo/plamo_form_urlencoded.h
 - ext/plamo/plamo_http_header.c
 - ext/plamo/plamo_http_header.h
 - ext/plamo/plamo_http_query.c
@@ -87,7 +98,6 @@ files:
 - ext/plamo/plamo_response.h
 - ext/plamo/plamo_string_array.c
 - ext/plamo/plamo_string_array.h
-- ext/plamo/wrapper.h
 - lib/plamo.rb
 - lib/plamo/version.rb
 - plamo.gemspec

data/ext/plamo/wrapper.h

@@ -1,8 +0,0 @@
-#ifndef RUBY_WRAPPER_H
-#define RUBY_WRAPPER_H
-
-typedef struct Wrapper {
-  void *inner;
-} Wrapper;
-
-#endif /* RUBY_WRAPPER_H */