rb_sdl2 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6e1202e5d4354c491782ad3d005ff73685498fe74db86b27fb5dd89e96bbd6e0
-  data.tar.gz: d3544770d711514d14ea918eaacf135569e8ccb0329ec52f956ea0fdfddde46a
+  metadata.gz: 343018a331bb80be39191fae7cce6088e59f30522926e6d7da2e8ea0db468683
+  data.tar.gz: e1a9393769b54fe801ad6af026f3a34f5ed544d12aeb621f41b7771a31bda453
 SHA512:
-  metadata.gz: f7675ac774fd74152857edbad3a319f5700336dbe7cc375f80b2775ddf9f4695a3a07ecfd5c667252252cb7907539efd28662cae29d9ae7424fa2776edf7cc8e
-  data.tar.gz: 8ec901e68e62c38d850c4ddeac7526e23bd433f3ad238327f60f4ab8529ad4c3a779301b231a82ad0bc20a39341c8c079b853fd1955be34eb397ef5f8114a137
+  metadata.gz: 0a72ab5495e18557a202454284b55c2d30ebbcec6bf21d7487f95398b06ac0498cee3537ce28a9f8bc1ba75e7ef5eafed222d06dfd5a83809c7f806003131a23
+  data.tar.gz: fe2d518ee617debbaf5651bb875ababc54c6379db21ac80b68f0bfab40881a2879bd2e3af5790e0d2370a83942f07cecdeefdeecaea4f692fe17a5849a8553e8

data/CHANGELOG.md

@@ -12,6 +12,22 @@
 
 ### Security
 
+## [0.1.2] - 2021-10-19
+
+### Added
+- RWOps#inspect - Return Ruby File#inspect like string.
+
+### Changed
+- RWOps - The constructor can accept blocks.
+- RWOperator - Code cleanup.
+
+### Fixed
+- RWOps - some bugs.
+- RWOperator - some bugs.
+
+### Removed
+- RWOps.to_ptr - There is no use case.
+
 ## [0.1.1] - 2021-10-11
 
 ### Added

data/lib/rb_sdl2/rw_ops/rw_operator.rb

@@ -10,7 +10,7 @@ module RbSDL2
       def initialize(obj)
         # int (* close) (struct SDL_RWops * context);
         super(:int, [:pointer]) do |_context|
-          # close の際に _context ポインターを開放してはならない。
+          # close の際に _context ポインターを開放してはならない。ポインターは GC が管理する。
           (obj.close; 0) rescue -1
         end
       end
@@ -63,10 +63,13 @@ module RbSDL2
       end
     end
 
-    class RWopsPointer < ::FFI::AutoPointer
+    class RWOperatorPointer < RWOps::RWOpsPointer
       class << self
-        # SDL_AllocRW で確保したポインターのみ SDL_FreeRW で開放できる。
-        def release(ptr) = ::SDL2.SDL_FreeRW(ptr)
+        def release(ptr)
+          # オブジェクトはクローズしない。（GC 回収の時には close コールバックオブジェクトは消えている）
+          # SDL_AllocRW で確保したポインターのみ SDL_FreeRW で開放できる。
+          ::SDL2.SDL_FreeRW(ptr)
+        end
       end
     end
 
@@ -77,26 +80,26 @@ module RbSDL2
       # メソッドの戻り値は Ruby IO と同じ値を返せばよい。
       # メソッド内での例外は SDL のエラーに変換され、Ruby 側には反映されない。
       # obj がメソッド呼び出しに応答しない場合も安全である。その場合は SDL 側にエラーが通知される。
-      def new(io)
-        ptr = RWopsPointer.new(::SDL2.SDL_AllocRW)
+      def new(obj)
+        ptr = RWOperatorPointer.new(::SDL2.SDL_AllocRW)
         raise RbSDL2Error if ptr.null?
-        super(ptr, io)
+        super(ptr, obj)
       end
     end
 
     def initialize(ptr, obj)
-      st = ::SDL2::SDL_RWops.new(ptr)
-      st[:close] = @close = CloseCallback.new(obj)
-      st[:read]  = @read  = ReadCallback.new(obj)
-      st[:seek]  = @seek  = SeekCallback.new(obj)
-      st[:size]  = @size  = SizeCallback.new(obj)
-      st[:write] = @write = WriteCallback.new(obj)
       @obj = obj
-      @ptr = ptr
+      @st = ::SDL2::SDL_RWops.new(ptr).tap do |st|
+        st[:close] = @close = CloseCallback.new(obj)
+        st[:read]  = @read  = ReadCallback.new(obj)
+        st[:seek]  = @seek  = SeekCallback.new(obj)
+        st[:size]  = @size  = SizeCallback.new(obj)
+        st[:write] = @write = WriteCallback.new(obj)
+      end
     end
 
     def __getobj__ = @obj
 
-    def to_ptr = @ptr
+    def to_ptr = @st.to_ptr
   end
 end

data/lib/rb_sdl2/rw_ops.rb

@@ -1,11 +1,12 @@
 module RbSDL2
   class RWOps
-    class Releaser < ::FFI::AutoPointer
+    class RWOpsPointer < ::FFI::AutoPointer
       class << self
         def release(ptr)
           # SDL_RWclose は RWOps構造体を開放する。そのため呼び出しは1回しかできない。
           # ::FFI::AutoPointer を使うことで2重開放を防ぐ。
-          raise RbSDL2Error if ::SDL2.SDL_RWclose(ptr) < 0
+          err = ::SDL2.SDL_RWclose(ptr)
+          raise RbSDL2Error if err < 0
         end
       end
 
@@ -26,16 +27,28 @@ module RbSDL2
               else
                 ::SDL2.SDL_RWFromMem(mem, size)
               end
+        ptr = RWOpsPointer.new(ptr)
         raise RbSDL2Error if ptr.null?
+        ptr.autorelease = autoclose
         obj = allocate
-        obj.__send__(:initialize, ptr, mem, autoclose: autoclose)
+        obj.__send__(:initialize, ptr, mem)
+        if block_given?
+          begin
+            yield(obj)
+          ensure
+            obj.close
+          end
+        else
+          obj
+        end
       end
 
       # mode は一般的なファイルAPIと同じ文字列が使用できる。
       def new(file, _mode = "rb", autoclose: true, mode: _mode)
-        ptr = ::SDL2.SDL_RWFromFile(file.to_s, mode)
+        ptr = RWOpsPointer.new(::SDL2.SDL_RWFromFile(file.to_s, mode))
         raise RbSDL2Error if ptr.null?
-        obj = super(ptr, autoclose: autoclose)
+        ptr.autorelease = autoclose
+        obj = super(ptr, file)
         if block_given?
           begin
             yield(obj)
@@ -48,37 +61,49 @@ module RbSDL2
       end
       alias open new
 
-      def to_ptr(ptr)
-        obj = allocate
-        obj.__send__(:initialize, ptr, autoclose: false)
-        obj
-      end
-
       require_relative 'rw_ops/rw_operator'
 
-      def with_object(obj)
-        rw = RWOperator.new(obj)
+      # io 引数には Ruby の IO オブジェクトのように振る舞うオブジェクトを与える。
+      # オブジェクトは自動的にクローズされない。（close が呼ばれた場合はクローズする）
+      # autoclose オプション引数に false を与えて、RWOps#to_ptr から取り出したポインターを
+      # C のスコープへ渡す場合、ポインターが利用されている間 RWOps オブジェクトを生存させる必要がある。
+      def with_object(io, autoclose: true)
+        rw = RWOperator.new(io)
+        ptr = rw.to_ptr
+        ptr.autorelease = autoclose
         obj = allocate
-        obj.__send__(:initialize, rw.to_ptr, rw, autoclose: false)
+        obj.__send__(:initialize, ptr, rw)
+        if block_given?
+          begin
+            yield(obj)
+          ensure
+            obj.close
+          end
+        else
+          obj
+        end
       end
     end
 
-    def initialize(ptr, obj = nil, autoclose:)
+    def initialize(ptr, obj = nil)
       @obj = obj
-      @ptr = Releaser.new(ptr)
-      self.autoclose = autoclose
+      @ptr = ptr
     end
 
     def autoclose=(bool)
       @ptr.autorelease = bool
     end
 
-    def autoclose? = @ptr.autorelease
+    def autoclose? = @ptr.autorelease?
 
     def close = @ptr.free
 
     def closed? = @ptr.released?
 
+    def inspect
+      "#<#{self.class.name}:#{@obj.inspect}>"
+    end
+
     def read(length = nil)
       raise IOError if closed?
       len = length.nil? ? size : length.to_i

data/rb_sdl2.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |spec|
   spec.name          = "rb_sdl2"
-  spec.version       = "0.1.1"
+  spec.version       = "0.1.2"
   spec.author        = "shinokaro"
   spec.email         = "shinokaro@hotmail.co.jp"
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rb_sdl2
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - shinokaro
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-11 00:00:00.000000000 Z
+date: 2021-10-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi