sparsam 0.2.9 → 0.2.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 34cdf5f3a73cda371b8a1275369af0063e525d0951f44c23a12e02e6dfa3c519
-  data.tar.gz: f5d7efaf6389491e9ceab33839b828fb26757d8b84b8cc3ba9cf87b509716e13
+  metadata.gz: c759ead3b6f09dd4e6b1e1754dc0443ea782e37fd717617d4069e5952605bc85
+  data.tar.gz: 3734e8cfa8796efdcfe78a702a536672bf88e5e442fdedb1e59b60602b34daaa
 SHA512:
-  metadata.gz: c2482ca31e496f3e1fb2597eaf7eb8f83e2f5545d7b2f8165840c5b394b4f63ce522bff6c2d18e11287803cc739ad279b97cf45f97efa1bbdd2f29c5594c8fc2
-  data.tar.gz: 62c14e98d040e76fdf6d9a886c0b138e25f0e8b9b901445b000f6c9fed12947c7537749a24f5601fc84d4e920b6b18b816fbcbe946aa525242de57d269698abd
+  metadata.gz: 9eec207eba73e7fdb9a53edf2de172f34d640cfc15d484804204b929ecdc4caf223eb76a01be496487734ca1af54760fc1e7ad9cf4622e27183e4c4db24fa323
+  data.tar.gz: 27a9faadcf29db90b72d14a1b775c10ac9004a31f98562f1906c771a8f923124f1290feef5df5e2e8aefc39912ccaedc2b783b8461573eab71b2afc9770d6f4a

data/ext/serializer.cpp

@@ -1,7 +1,5 @@
-extern "C" {
 #include <ruby.h>
 #include <ruby/intern.h>
-}
 #include <ruby/encoding.h>
 #include <stdio.h>
 #include <thrift/protocol/TBinaryProtocol.h>
@@ -51,6 +49,8 @@ VALUE SetClass;
 KlassFieldsCache klassCache;  // consider the memory leaked.
 std::unordered_set<VALUE> unions;
 
+#define SPARSAM_CHECK_TYPE(x, t, outer_struct, field_sym) if (!(RB_TYPE_P(x, t))) { raise_type_mismatch(outer_struct, field_sym, t, x); }
+
 void *serializer_create() { return (void *)(new ThriftSerializer()); }
 
 void serializer_free(void *data) {
@@ -185,12 +185,6 @@ static inline long raise_type_mismatch_as_value(VALUE outer_struct,
   return 0;
 }
 
-static inline void Sparsam_Check_Type(VALUE x, int t, VALUE outer_struct,
-                                      VALUE field_sym) {
-  if (!(RB_TYPE_P(x, t))) {
-    raise_type_mismatch(outer_struct, field_sym, t, x);
-  }
-}
 
 static inline VALUE make_ruby_bool(bool val) { return val ? Qtrue : Qfalse; }
 
@@ -513,7 +507,7 @@ void ThriftSerializer::writeAny(TType ttype, FieldInfo *field_info,
     HANDLE_TYPE(BYTE, Byte, CONVERT_FIXNUM(byte_convert, protocol::T_BYTE))
 
     case protocol::T_STRING: {
-      Sparsam_Check_Type(actual, T_STRING, outer_struct, field_sym);
+      SPARSAM_CHECK_TYPE(actual, T_STRING, outer_struct, field_sym);
 
       string data = string(StringValuePtr(actual), RSTRING_LEN(actual));
       if (field_info->isBinaryString) {
@@ -525,7 +519,7 @@ void ThriftSerializer::writeAny(TType ttype, FieldInfo *field_info,
     }
 
     case protocol::T_LIST: {
-      Sparsam_Check_Type(actual, T_ARRAY, outer_struct, field_sym);
+      SPARSAM_CHECK_TYPE(actual, T_ARRAY, outer_struct, field_sym);
 
       long length = RARRAY_LEN(actual);
       TType elem = field_info->elementType->ftype;
@@ -556,7 +550,7 @@ void ThriftSerializer::writeAny(TType ttype, FieldInfo *field_info,
     }
 
     case protocol::T_MAP: {
-      Sparsam_Check_Type(actual, T_HASH, outer_struct, field_sym);
+      SPARSAM_CHECK_TYPE(actual, T_HASH, outer_struct, field_sym);
 
       TType keyTType = field_info->keyType->ftype,
             valueTType = field_info->elementType->ftype;

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sparsam
 version: !ruby/object:Gem::Version
-  version: 0.2.9
+  version: 0.2.10
 platform: ruby
 authors:
 - Airbnb Thrift Developers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-01-14 00:00:00.000000000 Z
+date: 2022-05-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec