mediainfo-native 0.2.9 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cc1cb6b37d320316c8b909401511361a3354cc25a7a246d6aa5a25fc5005d767
-  data.tar.gz: c495092814ed874dca479e8e130bd80c77465a69b001c54d9ea0487e8b5e5a2a
+  metadata.gz: c240897a580eb3de9faaf15a827f96d20d310b0532cb2b01b74bf481bbb31760
+  data.tar.gz: 41596ad2d43cb9141732bac08d7e4cf1a4939b1df6b50739edf4d6f44ea243e3
 SHA512:
-  metadata.gz: 68f7cbb0f8b8d829a0be7c2d7ea98e676df82ccd0c80bc47d6277ffefed468426a7bcf8c288743428fa6778b44363f3c9dffa63d76208396f523761d8ab70898
-  data.tar.gz: 183be53bc08407becb4d67cde69ebfbc1e5705025725679d1f0ce318fd0296dffea6d950157a9efe70efc5cdc332a8cba9abc6abd40a9f3d4100c65f4caba337
+  metadata.gz: 34d30843f971abf092d7db9676176e7c9a76ea531156d1b053f58025a6cfb2063b0bb24d5ce6b1b1309d96d503e2788428e37aebe9bd78e1652d32d64b2a3086
+  data.tar.gz: 579b8b1185b0222f9ac99150a66fd29183b4a3fe91d301fff38610a076aa3e6a620a2a1bdca9cd17899ff019c604b832ec21f19d53b207114c8d9cfdc1060114

data/ext/mediainfo_native/MediaInfoDLL.h

@@ -15,9 +15,12 @@
 #ifndef MediaInfoDLLH
 #define MediaInfoDLLH
 
-
+#ifndef UNICODE
 #define UNICODE
+#endif
+#ifndef _UNICODE
 #define _UNICODE
+#endif
 
 //***************************************************************************
 // Platforms (from libzen)

data/ext/mediainfo_native/basestream.cpp

@@ -8,7 +8,7 @@ namespace MediaInfoNative
 
 #define GET_BASESTREAM(var) \
   BaseStream* var; \
-  Data_Get_Struct(self, BaseStream, var)
+  TypedData_Get_Struct(self, BaseStream, &basestream_type, var)
 
 extern "C"
 {
@@ -18,6 +18,17 @@ extern "C"
     delete ((BaseStream*) ptr);
   }
 
+  static const rb_data_type_t basestream_type = {
+    "BaseStream",
+    {
+      nullptr, // mark function
+      bs_free, // free function
+      nullptr,  // memsize function
+    },
+    nullptr, nullptr,
+    RUBY_TYPED_FREE_IMMEDIATELY
+  };
+
   static VALUE bs_lookup(VALUE self, VALUE key)
   {
     GET_BASESTREAM(bs);
@@ -31,7 +42,9 @@ VALUE stream_klasses[STREAM_TYPE_MAX];
 void Init_BaseStream(VALUE mMediaInfoNative)
 {
   VALUE cBaseStream = rb_define_class_under(mMediaInfoNative, "BaseStream", rb_cObject);
+  rb_undef_alloc_func(cBaseStream);
   VALUE cBaseStreamWithFramerate = rb_define_class_under(mMediaInfoNative, "BaseStreamWithFramerate", cBaseStream);
+  rb_undef_alloc_func(cBaseStreamWithFramerate);
 
   stream_klasses[GENERAL] = rb_define_class_under(mMediaInfoNative, "GeneralStream", cBaseStream);
   stream_klasses[VIDEO]   = rb_define_class_under(mMediaInfoNative, "VideoStream", cBaseStreamWithFramerate);
@@ -42,7 +55,8 @@ void Init_BaseStream(VALUE mMediaInfoNative)
   stream_klasses[MENU]    = rb_define_class_under(mMediaInfoNative, "MenuStream", cBaseStream);
 
   for(unsigned int st = 0; st < STREAM_TYPE_MAX; ++st) {
-    rb_define_method(stream_klasses[st], "lookup", (VALUE(*)(...)) bs_lookup, 1);
+    rb_undef_alloc_func(stream_klasses[st]);
+    rb_define_method(stream_klasses[st], "lookup", RUBY_METHOD_FUNC(bs_lookup), 1);
   }
 }
 
@@ -55,7 +69,7 @@ BaseStream::BaseStream(StreamType _type, unsigned int _idx, MediaInfoWrapper* _w
 
 VALUE BaseStream::wrap()
 {
-  return Data_Wrap_Struct(stream_klasses[type], 0, bs_free, this);
+  return TypedData_Wrap_Struct(stream_klasses[type], &basestream_type, this);
 }
 
 VALUE BaseStream::lookup(VALUE key) const

data/ext/mediainfo_native/mediainfo_wrapper.cpp

@@ -12,7 +12,7 @@ namespace MediaInfoNative
 
 #define GET_WRAPPER(var) \
   MediaInfoWrapper* var; \
-  Data_Get_Struct(self, MediaInfoWrapper, var)
+  TypedData_Get_Struct(self, MediaInfoWrapper, &mediainfo_wrapper_type, var)
 
 typedef struct {
   MediaInfoWrapper* miw;
@@ -23,13 +23,22 @@ typedef struct {
 extern "C"
 {
 
-  typedef VALUE(*RUBYFUNC)(...);
-
   static void miw_free(void* ptr) 
   {
     delete ((MediaInfoWrapper*) ptr);
   }
 
+  static const rb_data_type_t mediainfo_wrapper_type = {
+    "MediaInfoWrapper",
+    {
+      nullptr, // mark function
+      miw_free, // free function
+      nullptr,  // memsize function
+    },
+    nullptr, nullptr, // parent, data
+    RUBY_TYPED_FREE_IMMEDIATELY
+  };
+
   static VALUE miw_new(VALUE klass, VALUE args)
   {
     if(RARRAY_LEN(args) > 1)
@@ -46,7 +55,7 @@ extern "C"
     }
 
     MediaInfoWrapper* miw = new MediaInfoWrapper(ignore_continuous_file_names);
-    return Data_Wrap_Struct(klass, 0, miw_free, miw);
+    return TypedData_Wrap_Struct(klass, &mediainfo_wrapper_type, miw);
   }
 
   static VALUE miw_close(VALUE self)
@@ -79,7 +88,7 @@ extern "C"
     }
 
     if(rb_block_given_p())
-      return rb_ensure((RUBYFUNC) rb_yield, Qnil, (RUBYFUNC) miw_close, self);
+      return rb_ensure(rb_yield, Qnil, miw_close, self);
     else
       return Qnil;
   }
@@ -123,16 +132,17 @@ extern "C"
 void Init_MediaInfoWrapper(VALUE mMediaInfoNative)
 {
   VALUE cMediaInfo = rb_define_class_under(mMediaInfoNative, "MediaInfo", rb_cObject);
+  rb_undef_alloc_func(cMediaInfo);
 
-  rb_define_singleton_method(cMediaInfo, "new", (RUBYFUNC) miw_new, -2);
+  rb_define_singleton_method(cMediaInfo, "new", RUBY_METHOD_FUNC(miw_new), -2);
 
-  rb_define_method(cMediaInfo, "close", (RUBYFUNC) miw_close, 0);
-  rb_define_method(cMediaInfo, "open", (RUBYFUNC) miw_open, 1);
-  rb_define_method(cMediaInfo, "is_open?", (RUBYFUNC) miw_is_open, 0);
-  rb_define_method(cMediaInfo, "streams", (RUBYFUNC) miw_streams, 0);
+  rb_define_method(cMediaInfo, "close", RUBY_METHOD_FUNC(miw_close), 0);
+  rb_define_method(cMediaInfo, "open", RUBY_METHOD_FUNC(miw_open), 1);
+  rb_define_method(cMediaInfo, "is_open?", RUBY_METHOD_FUNC(miw_is_open), 0);
+  rb_define_method(cMediaInfo, "streams", RUBY_METHOD_FUNC(miw_streams), 0);
 
-  rb_define_method(cMediaInfo, "inform", (RUBYFUNC) miw_inform, 1);
-  rb_define_method(cMediaInfo, "option", (RUBYFUNC) miw_option, 0);
+  rb_define_method(cMediaInfo, "inform", RUBY_METHOD_FUNC(miw_inform), 1);
+  rb_define_method(cMediaInfo, "option", RUBY_METHOD_FUNC(miw_option), 0);
 }
 
 /* ************************** MediaInfoWrapper ****************************** */
@@ -157,6 +167,9 @@ MediaInfoDLL::stream_t convertToMediaInfoStreamType(StreamType type)
     return MediaInfoDLL::Stream_Image;
   case MENU:
     return MediaInfoDLL::Stream_Menu;
+  default:
+    rb_raise(rb_eRuntimeError, "Invalid stream type: %d", (int)type);
+    __builtin_unreachable();
   }
 }
 

data/lib/mediainfo-native/attr_readers.rb

@@ -10,20 +10,22 @@ module MediaInfoNative
     end
 
     def mediainfo_attr_reader(attribute, mediainfo_key)
-      attribute_before_type_cast = "#{attribute}_before_type_cast"
+      # Sanitize for instance variable (remove ?)
+      ivar_name = attribute.to_s.tr('?', '')
+      attribute_before_type_cast = "#{ivar_name}_before_type_cast"
 
-      define_method attribute_before_type_cast do
+      define_method "#{attribute}_before_type_cast" do
         instance_variable_get("@#{attribute_before_type_cast}") || instance_variable_set("@#{attribute_before_type_cast}", lookup(mediainfo_key))
       end
 
       define_method attribute do
-        if v = instance_variable_get("@#{attribute}")
+        if v = instance_variable_get("@#{ivar_name}")
           v
         else
-          v = send(attribute_before_type_cast)
+          v = send("#{attribute}_before_type_cast")
           v = yield v if v and block_given?
 
-          instance_variable_set("@#{attribute}", v)
+          instance_variable_set("@#{ivar_name}", v)
         end
       end
 

data/lib/mediainfo-native/streams/general.rb

@@ -16,6 +16,9 @@ module MediaInfoNative
     mediainfo_int_reader :datasize, 'DataSize'
     mediainfo_int_reader :footersize, 'FooterSize'
     alias_method :writing_application, :encoded_application_string
+    mediainfo_attr_reader :streamable?, 'IsStreamable' do |value|
+      value.empty? ? nil : value == 'Yes'
+    end
 
     # Since MediaInfo v0.7.76 encoded_application is replaced by
     # encoded_application_string. So lets check which one is empty.

data/lib/mediainfo-native/version.rb

@@ -1,3 +1,3 @@
 module MediaInfoNative
-  VERSION = '0.2.9'.freeze
+  VERSION = '0.3.0'.freeze
 end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: mediainfo-native
 version: !ruby/object:Gem::Version
-  version: 0.2.9
+  version: 0.3.0
 platform: ruby
 authors:
 - Projective Technology GmbH
-autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-03-01 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake-compiler
@@ -71,7 +70,6 @@ homepage: https://github.com/projectivetech/mediainfo-native
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -86,8 +84,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.32
-signing_key: 
+rubygems_version: 3.6.7
 specification_version: 4
 summary: Native bindings for mediainfo
 test_files: []