abstriker 0.1.0 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 58fc310fb1c93359f11f08ece41ac5447b228166230656b8a481a21efb13ff42
-  data.tar.gz: c941dc1843f1bdb93cb567fa3e2a0c7fa9379b765832a9a01ccfe19ba9dd32f7
+  metadata.gz: 9084bf87aeeb1ffa9f9ec57310d709c57f30ad039154c384720793eea694c632
+  data.tar.gz: 6ee676d5a9a1229d38ed3a4ae2b7a4e1fe07cd3f5b1100954e0f42cdd1cb9e66
 SHA512:
-  metadata.gz: aa481fe912ed555b398e61ab3f64b9f3a51e3cada25148143ad8e7587bceacf3fb3afb5208b0b47327c174915168d9577bcb62b87ca629719f5001f5c94cd934
-  data.tar.gz: cd96e6d7772f00590e3a260a42602d52d7e14916e32c699c82f6b22930b7b1a962e1a901bf5bcab509c59ef12d79d7111784058248f0393a60ea1833f512973e
+  metadata.gz: 01d1026f053c0fe15266eeeba49e0795f87537e6393a0c326e8ccb6e26756792e087705a0a121c442494d46253b4eb985a668e7408077620b19faa999af30673
+  data.tar.gz: a5f838cca75e71629eb62645a78ae851962c361290d448ec5d14bec066a6dfc2d45be57e4cbf917920ce800ae6e9f6cb77e9fa511ded96793ef86c263281fb8e

data/README.md

@@ -1,4 +1,5 @@
 # Abstriker
+[![Build Status](https://travis-ci.org/joker1007/abstriker.svg?branch=master)](https://travis-ci.org/joker1007/abstriker)
 
 This gem adds `abstract` syntax. that is similar to Java's one.
 `abstract` modified method requires subclass implementation.
@@ -48,6 +49,29 @@ end # => raise
 If you want to disable Abstriker, write `Abstriker.disable = true` at first line.
 If Abstriker is disabled, TracePoint never runs, and so there is no overhead of VM instruction.
 
+### Caution
+
+Must not call `include` or `extend` abstracted module outer class definition.
+
+ex.
+
+```ruby
+module A1
+  extend Abstriker
+
+  abstract def foo
+  end
+end
+
+class A2;
+end
+
+A2.include(A1) # => Don't do this
+```
+
+This case leaves enabled TracePoint.
+It is very high overhead.
+
 ### Examples
 
 #### include module

data/lib/abstriker.rb

@@ -52,11 +52,23 @@ module Abstriker
     private
 
     def detect_event_type
-      caller_info = caller_locations(3, 1)[0]
-      if caller_info.label.match?(/block/)
-        [:end, :raise]
-      elsif caller_info.label.match?(/initialize/) || caller_info.label.match?(/new/)
-        [:b_call, :b_return, :raise]
+      callers = caller_locations(2, 3)
+      if callers[0].label == "inherited"
+        caller_info = callers[2]
+        if caller_info.label.match?(/initialize/) || caller_info.label.match?(/new/)
+          [:c_return, :b_call, :b_return, :raise]
+        else
+          [:end, :raise]
+        end
+      elsif callers[0].label == "included" || callers[0].label
+        caller_info = callers[2]
+        if caller_info.label.match?(/<class/) || caller_info.label.match?(/<module/)
+          [:end, :raise]
+        else
+          [:c_return, :b_call, :b_return, :raise]
+        end
+      else
+        raise "Not detect event_type"
       end
     end
 
@@ -77,11 +89,15 @@ module Abstriker
           block_count += 1 if t.event == :b_call
           block_count -= 1 if t.event == :b_return
 
-          if t.self == klass && (t.event == :end || t.event == :b_return && block_count.zero?)
+          t_self = t.self
+          target_end_event = t_self == klass && t.event == :end
+          target_b_return_event = t_self == klass && t.event == :b_return && block_count.zero?
+          target_c_return_event = t_self == Class && t.event == :c_return && t.method_id == :new
+          if target_end_event || target_b_return_event || target_c_return_event
             klass.ancestors.drop(1).each do |mod|
               Abstriker.abstract_methods[mod]&.each do |fmeth_name|
                 meth = klass.instance_method(fmeth_name)
-                unless meth&.owner == klass
+                if meth.owner == mod
                   tp.disable
                   klass.instance_variable_set("@__abstract_trace_point", nil)
                   raise Abstriker::NotImplementedError.new(klass, meth)
@@ -113,11 +129,15 @@ module Abstriker
           block_count += 1 if t.event == :b_call
           block_count -= 1 if t.event == :b_return
 
-          if t.self == klass && (t.event == :end || t.event == :b_return && block_count.zero?)
+          t_self = t.self
+          target_end_event = t_self == klass && t.event == :end
+          target_b_return_event = t_self == klass && t.event == :b_return && block_count.zero?
+          target_c_return_event = t_self == Class && t.event == :c_return && t.method_id == :new
+          if target_end_event || target_b_return_event || target_c_return_event
             klass.singleton_class.ancestors.drop(1).each do |mod|
               Abstriker.abstract_methods[mod]&.each do |fmeth_name|
                 meth = klass.singleton_class.instance_method(fmeth_name)
-                unless meth&.owner == klass.singleton_class
+                if meth.owner == mod
                   tp.disable
                   klass.instance_variable_set("@__abstract_singleton_trace_point", nil)
                   raise Abstriker::NotImplementedError.new(klass, meth)

data/lib/abstriker/version.rb

@@ -1,3 +1,3 @@
 module Abstriker
-  VERSION = "0.1.0"
+  VERSION = "0.1.2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: abstriker
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
 platform: ruby
 authors:
 - joker1007
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2018-01-25 00:00:00.000000000 Z
+date: 2018-01-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler