rjb 1.0.3 → 1.0.4

data/ext/rjb.c

@@ -15,7 +15,7 @@
  * $Id: rjb.c 8 2006-11-20 19:17:22Z arton $
  */
 
-#define RJB_VERSION "1.0.3"
+#define RJB_VERSION "1.0.4"
 
 #include "ruby.h"
 #include "st.h"
@@ -1885,6 +1885,7 @@ static VALUE rjb_newinstance_s(int argc, VALUE* argv, VALUE self)
     VALUE ret = Qnil;
     struct jv_data* ptr;
     JNIEnv* jenv = rjb_attach_current_thread();
+    int found = 0;
 
     rb_scan_args(argc, argv, "1*", &vsig, &rest);
     sig = StringValueCStr(vsig);
@@ -1897,11 +1898,15 @@ static VALUE rjb_newinstance_s(int argc, VALUE* argv, VALUE self)
 	    if ((*pc)->arg_count == argc - 1
 		&& !strcmp(sig, (*pc)->method_signature))
 	    {
+	        found = 1;
 		ret = createinstance(jenv, argc - 1, argv + 1, &ptr->idata, *pc);
 		break;
 	    }
 	}
     }
+    if (!found) {
+	rb_raise(rb_eRuntimeError, "Constructor not found");
+    }
     return ret;
 }
 
@@ -1911,12 +1916,13 @@ static VALUE rjb_newinstance(int argc, VALUE* argv, VALUE self)
     struct jv_data* ptr;
     struct cls_constructor** pc;
     JNIEnv* jenv = rjb_attach_current_thread();
+    int found = 0;
 
     Data_Get_Struct(self, struct jv_data, ptr);
 
     if (ptr->constructors)
     {
-        int i, found = 0;
+        int i;
 	char* psig;
 	for (pc = ptr->constructors; *pc; pc++)
 	{
@@ -1941,6 +1947,9 @@ static VALUE rjb_newinstance(int argc, VALUE* argv, VALUE self)
 	    }
 	}
     }
+    if (!found) {
+	rb_raise(rb_eRuntimeError, "Constructor not found");
+    }
     return ret;
 }
 

data/test/test.rb

@@ -411,5 +411,15 @@ class TestRjb < Test::Unit::TestCase
     assert_equal(nil, test.callWithArraies(nil, nil, nil, nil, nil, nil,
                                            nil, nil))
   end
+
+  def test_failed_constructor_call()
+    test = import('java.lang.String')
+    begin
+      s = test.new('a', 'b', 'c')
+      flunk('no exception')
+    rescue RuntimeError => e
+      assert(e)
+    end
+  end
 end
 

metadata

@@ -3,8 +3,8 @@ rubygems_version: 0.9.0
 specification_version: 1
 name: rjb
 version: !ruby/object:Gem::Version 
-  version: 1.0.3
-date: 2006-11-21 00:00:00 +09:00
+  version: 1.0.4
+date: 2007-05-06 00:00:00 +09:00
 summary: Ruby Java bridge
 require_paths: 
 - lib
@@ -30,20 +30,20 @@ authors:
 - arton
 files: 
 - ext/RBridge.java
+- ext/load.c
 - ext/riconv.c
-- ext/rjb.c
 - ext/rjbexception.c
-- ext/load.c
+- ext/rjb.c
 - ext/jniwrap.h
+- ext/rjb.h
 - ext/riconv.h
 - ext/jp_co_infoseek_hp_arton_rjb_RBridge.h
-- ext/rjb.h
 - ext/depend
 - data/rjb/jp/co/infoseek/hp/arton/rjb/RBridge.class
 - lib/rjb.rb
 - samples/filechooser.rb
-- test/test.rb
 - test/gctest.rb
+- test/test.rb
 - test/jp/co/infoseek/hp/arton/rjb/ExtBase.class
 - test/jp/co/infoseek/hp/arton/rjb/Test.class
 - test/jp/co/infoseek/hp/arton/rjb/IBase.class