rays 0.2.1 → 0.3.1

data/.doc/ext/rays/polygon.cpp

@@ -8,7 +8,7 @@
 #include "defs.h"
 
 
-RUCY_DEFINE_VALUE_OR_ARRAY_FROM_TO(Rays::Polygon)
+RUCY_DEFINE_VALUE_OR_ARRAY_FROM_TO(RAYS_EXPORT, Rays::Polygon)
 
 #define THIS  to<Rays::Polygon*>(self)
 
@@ -26,7 +26,7 @@ VALUE setup(VALUE self, VALUE args, VALUE loop, VALUE colors, VALUE texcoords)
 {
 	CHECK;
 
-	if (args[0].is_kind_of(Rays::polyline_class()))
+	if (args[0].is_a(Rays::polyline_class()))
 		*THIS = to<Rays::Polygon>(args.size(), args.as_array());
 	else
 	{
@@ -46,7 +46,7 @@ VALUE expand(VALUE self)
 	coord width         =             to<coord>         (argv[0]);
 	Rays::CapType cap   = argc >= 2 ? to<Rays::CapType> (argv[1]) : Rays::CAP_DEFAULT;
 	Rays::JoinType join = argc >= 3 ? to<Rays::JoinType>(argv[2]) : Rays::JOIN_DEFAULT;
-	coord ml            = argc >= 4 ? to<coord>         (argv[3]) : Rays::JOIN_DEFAULT_MITER_LIMIT;
+	coord ml            = argc >= 4 ? to<coord>         (argv[3]) : (coord) Rays::JOIN_DEFAULT_MITER_LIMIT;
 
 	Rays::Polygon polygon;
 	THIS->expand(&polygon, width, cap, join, ml);
@@ -83,7 +83,9 @@ VALUE get_at(VALUE self, VALUE index)
 	int i    = to<int>(index);
 	if (i < 0) i += size;
 
-	if (i < 0 || size <= i)
+	if (i < 0)
+		index_error(__FILE__, __LINE__);
+	if (i >= size)
 		index_error(__FILE__, __LINE__);
 
 	return value((*THIS)[i]);
@@ -116,10 +118,10 @@ VALUE op_add(VALUE self, VALUE obj)
 {
 	CHECK;
 
-	if (obj.is_kind_of(Rays::polyline_class()))
+	if (obj.is_a(Rays::polyline_class()))
 		return value(*THIS + to<Rays::Polyline&>(obj));
 
-	if (obj.is_kind_of(Rays::polygon_class()))
+	if (obj.is_a(Rays::polygon_class()))
 		return value(*THIS + to<Rays::Polygon&>(obj));
 
 	if (!obj.is_array())
@@ -131,7 +133,7 @@ VALUE op_add(VALUE self, VALUE obj)
 	for (const auto& polyline : to<Rays::Polygon&>(self))
 		polylines.emplace_back(polyline);
 
-	if (obj[0].is_kind_of(Rays::polyline_class()))
+	if (obj[0].is_a(Rays::polyline_class()))
 	{
 		each_poly<Rays::Polyline>(obj, [&](const auto& polyline)
 		{
@@ -375,14 +377,14 @@ namespace Rucy
 {
 
 
-	template <> Rays::Polygon
+	template <> RAYS_EXPORT Rays::Polygon
 	value_to<Rays::Polygon> (int argc, const Value* argv, bool convert)
 	{
 		if (convert)
 		{
 			if (argc <= 0)
 				return Rays::Polygon();
-			else if (argv->is_kind_of(Rays::polyline_class()))
+			else if (argv->is_a(Rays::polyline_class()))
 			{
 				if (argc == 1)
 					return Rays::Polygon(to<Rays::Polyline&>(*argv));

data/.doc/ext/rays/polyline.cpp

@@ -10,7 +10,7 @@
 #include "defs.h"
 
 
-RUCY_DEFINE_VALUE_OR_ARRAY_FROM_TO(Rays::Polyline)
+RUCY_DEFINE_VALUE_OR_ARRAY_FROM_TO(RAYS_EXPORT, Rays::Polyline)
 
 #define THIS  to<Rays::Polyline*>(self)
 
@@ -44,7 +44,7 @@ VALUE expand(VALUE self)
 	coord width         =             to<coord>         (argv[0]);
 	Rays::CapType cap   = argc >= 2 ? to<Rays::CapType> (argv[1]) : Rays::CAP_DEFAULT;
 	Rays::JoinType join = argc >= 3 ? to<Rays::JoinType>(argv[2]) : Rays::JOIN_DEFAULT;
-	coord ml            = argc >= 4 ? to<coord>         (argv[3]) : Rays::JOIN_DEFAULT_MITER_LIMIT;
+	coord ml            = argc >= 4 ? to<coord>         (argv[3]) : (coord) Rays::JOIN_DEFAULT_MITER_LIMIT;
 
 	Rays::Polygon polygon;
 	THIS->expand(&polygon, width, cap, join, ml);
@@ -102,7 +102,9 @@ VALUE get_at(VALUE self, VALUE index)
 	int i    = to<int>(index);
 	if (i < 0) i += size;
 
-	if (i < 0 || size <= i)
+	if (i < 0)
+		index_error(__FILE__, __LINE__);
+	if (i >= size)
 		index_error(__FILE__, __LINE__);
 
 	return value((*THIS)[i]);
@@ -206,7 +208,7 @@ namespace Rucy
 {
 
 
-	template <> Rays::Polyline
+	template <> RAYS_EXPORT Rays::Polyline
 	value_to<Rays::Polyline> (int argc, const Value* argv, bool convert)
 	{
 		assert(argc == 0 || (argc > 0 && argv));

data/.doc/ext/rays/rays.cpp

@@ -6,11 +6,11 @@
 #include "defs.h"
 
 
-RUCY_DEFINE_CONVERT_TO(Rays::CapType)
-RUCY_DEFINE_CONVERT_TO(Rays::JoinType)
-RUCY_DEFINE_CONVERT_TO(Rays::BlendMode)
-RUCY_DEFINE_CONVERT_TO(Rays::TexCoordMode)
-RUCY_DEFINE_CONVERT_TO(Rays::TexCoordWrap)
+RUCY_DEFINE_CONVERT_TO(RAYS_EXPORT, Rays::CapType)
+RUCY_DEFINE_CONVERT_TO(RAYS_EXPORT, Rays::JoinType)
+RUCY_DEFINE_CONVERT_TO(RAYS_EXPORT, Rays::BlendMode)
+RUCY_DEFINE_CONVERT_TO(RAYS_EXPORT, Rays::TexCoordMode)
+RUCY_DEFINE_CONVERT_TO(RAYS_EXPORT, Rays::TexCoordWrap)
 
 
 template <typename T>
@@ -102,7 +102,7 @@ namespace Rucy
 {
 
 
-	template <> Rays::CapType
+	template <> RAYS_EXPORT Rays::CapType
 	value_to<Rays::CapType> (int argc, const Value* argv, bool convert)
 	{
 		assert(argc > 0 && argv);
@@ -126,14 +126,16 @@ namespace Rucy
 		}
 
 		int type = value_to<int>(*argv, convert);
-		if (type < 0 || Rays::CAP_MAX <= type)
+		if (type < 0)
+			argument_error(__FILE__, __LINE__, "invalid cap type -- %d", type);
+		if (type >= Rays::CAP_MAX)
 			argument_error(__FILE__, __LINE__, "invalid cap type -- %d", type);
 
 		return (Rays::CapType) type;
 	}
 
 
-	template <> Rays::JoinType
+	template <> RAYS_EXPORT Rays::JoinType
 	value_to<Rays::JoinType> (int argc, const Value* argv, bool convert)
 	{
 		assert(argc > 0 && argv);
@@ -157,14 +159,16 @@ namespace Rucy
 		}
 
 		int type = value_to<int>(*argv, convert);
-		if (type < 0 || Rays::JOIN_MAX <= type)
+		if (type < 0)
+			argument_error(__FILE__, __LINE__, "invalid join type -- %d", type);
+		if (type >= Rays::JOIN_MAX)
 			argument_error(__FILE__, __LINE__, "invalid join type -- %d", type);
 
 		return (Rays::JoinType) type;
 	}
 
 
-	template <> Rays::BlendMode
+	template <> RAYS_EXPORT Rays::BlendMode
 	value_to<Rays::BlendMode> (int argc, const Value* argv, bool convert)
 	{
 		assert(argc > 0 && argv);
@@ -188,14 +192,16 @@ namespace Rucy
 		}
 
 		int mode = value_to<int>(*argv, convert);
-		if (mode < 0 || Rays::BLEND_MAX <= mode)
+		if (mode < 0)
+			argument_error(__FILE__, __LINE__, "invalid blend mode -- %d", mode);
+		if (mode >= Rays::BLEND_MAX)
 			argument_error(__FILE__, __LINE__, "invalid blend mode -- %d", mode);
 
 		return (Rays::BlendMode) mode;
 	}
 
 
-	template <> Rays::TexCoordMode
+	template <> RAYS_EXPORT Rays::TexCoordMode
 	value_to<Rays::TexCoordMode> (int argc, const Value* argv, bool convert)
 	{
 		assert(argc > 0 && argv);
@@ -219,14 +225,16 @@ namespace Rucy
 		}
 
 		int mode = value_to<int>(*argv, convert);
-		if (mode < 0 || Rays::TEXCOORD_MODE_MAX <= mode)
+		if (mode < 0)
+			argument_error(__FILE__, __LINE__, "invalid texcoord mode -- %d", mode);
+		if (mode >= Rays::TEXCOORD_MODE_MAX)
 			argument_error(__FILE__, __LINE__, "invalid texcoord mode -- %d", mode);
 
 		return (Rays::TexCoordMode) mode;
 	}
 
 
-	template <> Rays::TexCoordWrap
+	template <> RAYS_EXPORT Rays::TexCoordWrap
 	value_to<Rays::TexCoordWrap> (int argc, const Value* argv, bool convert)
 	{
 		assert(argc > 0 && argv);
@@ -250,7 +258,9 @@ namespace Rucy
 		}
 
 		int wrap = value_to<int>(*argv, convert);
-		if (wrap < 0 || Rays::TEXCOORD_WRAP_MAX <= wrap)
+		if (wrap < 0)
+			argument_error(__FILE__, __LINE__, "invalid texcoord wrap -- %d", wrap);
+		if (wrap >= Rays::TEXCOORD_WRAP_MAX)
 			argument_error(__FILE__, __LINE__, "invalid texcoord wrap -- %d", wrap);
 
 		return (Rays::TexCoordWrap) wrap;

data/.doc/ext/rays/shader.cpp

@@ -6,7 +6,7 @@
 #include "defs.h"
 
 
-RUCY_DEFINE_VALUE_OR_ARRAY_FROM_TO(Rays::Shader)
+RUCY_DEFINE_VALUE_OR_ARRAY_FROM_TO(RAYS_EXPORT, Rays::Shader)
 
 #define THIS  to<Rays::Shader*>(self)
 
@@ -139,7 +139,7 @@ VALUE set_uniform(VALUE self)
 			case 4: THIS->set_uniform(name, Af(0), Af(1), Af(2), Af(3)); break;
 		}
 	}
-	else if (argv[0].is_kind_of(Rays::image_class()))
+	else if (argv[0].is_a(Rays::image_class()))
 		THIS->set_uniform(name, to<Rays::Image&>(argv[0]));
 	else
 		argument_error(__FILE__, __LINE__);
@@ -189,7 +189,7 @@ namespace Rucy
 {
 
 
-	template <> Rays::Shader
+	template <> RAYS_EXPORT Rays::Shader
 	value_to<Rays::Shader> (int argc, const Value* argv, bool convert)
 	{
 		if (argc == 1 && argv->is_array())

data/.github/workflows/release-gem.yml

@@ -24,7 +24,7 @@ jobs:
         run: "ruby -I.github/workflows -rutils -e 'setup_dependencies'"
 
       - name: test
-        run: bundle exec rake quiet test
+        run: bundle exec rake quiet packages test
 
       - name: create gem
         id: gem

data/.github/workflows/test.yml

@@ -24,6 +24,9 @@ jobs:
       - name: setup dependencies
         run: "ruby -I.github/workflows -rutils -e 'setup_dependencies'"
 
+      - name: packages
+        run: bundle exec rake packages
+
       - name: lib
         run: bundle exec rake lib
 

data/ChangeLog.md

@@ -1,6 +1,17 @@
 # rays ChangeLog
 
 
+## [v0.3.1] - 2025-01-13
+
+- Update LICENSE
+- Add smooth flag for Image
+
+
+## [v0.3] - 2024-07-06
+
+- Support Windows
+
+
 ## [v0.2.1] - 2024-07-05
 
 - Add OpenGL_init() and OpenGL_fin()

data/Gemfile.lock

@@ -9,7 +9,7 @@ GEM
 
 PLATFORMS
   arm64-darwin-21
-  arm64-darwin-22
+  arm64-darwin-23
 
 DEPENDENCIES
   rake

data/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2019 xord.org
+Copyright (c) 2011 xord.org
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/Rakefile

@@ -1,6 +1,5 @@
 # -*- mode: ruby -*-
 
-
 %w[../xot ../rucy .]
   .map  {|s| File.expand_path "#{s}/lib", __dir__}
   .each {|s| $:.unshift s if !$:.include?(s) && File.directory?(s)}
@@ -15,6 +14,8 @@ require 'rays/extension'
 EXTENSIONS  = [Xot, Rucy, Rays]
 TESTS_ALONE = ['test/test_rays_init.rb']
 
+install_packages win32: %w[MINGW_PACKAGE_PREFIX-glew]
+
 use_external_library 'https://github.com/g-truc/glm',
   tag:     '0.9.9.8',
   srcdirs: 'NOSRC'
@@ -32,12 +33,25 @@ use_external_library 'https://github.com/mapbox/earcut.hpp',
 
 use_external_library 'https://github.com/andrewwillmott/splines-lib',
   commit:   '11e7240d57b0d22871aec3308186a5fcf915ba77',
-  excludes: 'Test\.cpp'
+  excludes: 'Test\.cpp',
+  &proc {
+    filter_file('Splines.cpp') do |cpp|
+      <<~EOS + cpp
+        #include <cstdint>
+      EOS
+    end
+  }
+
+if win32?
+  use_external_library 'https://github.com/nothings/stb',
+    commit:  'ae721c50eaf761660b4f90cc590453cdb0c2acd0',
+    srcdirs: 'NOSRC'
+end
 
 default_tasks :ext
 use_bundler
 build_native_library
 build_ruby_extension
-test_ruby_extension
+test_ruby_extension unless github_actions? && win32?
 generate_documents
 build_ruby_gem

data/VERSION

@@ -1 +1 @@
-0.2.1
+0.3.1

data/ext/rays/bitmap.cpp

@@ -7,7 +7,7 @@
 #include "defs.h"
 
 
-RUCY_DEFINE_VALUE_FROM_TO(Rays::Bitmap)
+RUCY_DEFINE_VALUE_FROM_TO(RAYS_EXPORT, Rays::Bitmap)
 
 #define THIS  to<Rays::Bitmap*>(self)
 
@@ -323,23 +323,23 @@ set_pixels (Rays::Bitmap* bmp, Value pixels)
 	}
 }
 
-static inline Value
-to_rgb_value (uint8_t r, uint8_t g, uint8_t b)
+static inline uint32_t
+to_rgb (uint8_t r, uint8_t g, uint8_t b)
 {
-	return value(
-		((uint) r) << 16 |
-		((uint) g) << 8  |
-		((uint) b));
+	return
+		((uint32_t) r) << 16 |
+		((uint32_t) g) << 8  |
+		((uint32_t) b);
 }
 
-static inline Value
-to_argb_value (uint8_t r, uint8_t g, uint8_t b, uint8_t a)
+static inline uint32_t
+to_argb (uint8_t r, uint8_t g, uint8_t b, uint8_t a)
 {
-	return value(
-		((uint) a) << 24 |
-		((uint) r) << 16 |
-		((uint) g) << 8  |
-		((uint) b));
+	return
+		((uint32_t) a) << 24 |
+		((uint32_t) r) << 16 |
+		((uint32_t) g) << 8  |
+		((uint32_t) b);
 }
 
 static void
@@ -398,7 +398,7 @@ get_pixels (auto* pixels, const Rays::Bitmap& bmp)
 			{
 				const auto* p = bmp.at<uint8_t>(0, y);
 				for (int x = 0; x < w; ++x, p += 3)
-					pixels->push_back(to_rgb_value(p[0], p[1], p[2]));
+					pixels->push_back(value(to_rgb(p[0], p[1], p[2])));
 			}
 			break;
 
@@ -408,7 +408,7 @@ get_pixels (auto* pixels, const Rays::Bitmap& bmp)
 			{
 				const auto* p = bmp.at<uint8_t>(0, y);
 				for (int x = 0; x < w; ++x, p += 4)
-					pixels->push_back(to_argb_value(p[0], p[1], p[2], p[3]));
+					pixels->push_back(value(to_argb(p[0], p[1], p[2], p[3])));
 			}
 			break;
 
@@ -418,7 +418,7 @@ get_pixels (auto* pixels, const Rays::Bitmap& bmp)
 			{
 				const auto* p = bmp.at<uint8_t>(0, y);
 				for (int x = 0; x < w; ++x, p += 4)
-					pixels->push_back(to_argb_value(p[1], p[2], p[3], p[0]));
+					pixels->push_back(value(to_argb(p[1], p[2], p[3], p[0])));
 			}
 			break;
 
@@ -427,7 +427,7 @@ get_pixels (auto* pixels, const Rays::Bitmap& bmp)
 			{
 				const auto* p = bmp.at<uint8_t>(0, y);
 				for (int x = 0; x < w; ++x, p += 3)
-					pixels->push_back(to_rgb_value(p[2], p[1], p[0]));
+					pixels->push_back(value(to_rgb(p[2], p[1], p[0])));
 			}
 			break;
 
@@ -437,7 +437,7 @@ get_pixels (auto* pixels, const Rays::Bitmap& bmp)
 			{
 				const auto* p = bmp.at<uint8_t>(0, y);
 				for (int x = 0; x < w; ++x, p += 4)
-					pixels->push_back(to_argb_value(p[2], p[1], p[0], p[3]));
+					pixels->push_back(value(to_argb(p[2], p[1], p[0], p[3])));
 			}
 			break;
 
@@ -447,7 +447,7 @@ get_pixels (auto* pixels, const Rays::Bitmap& bmp)
 			{
 				const auto* p = bmp.at<uint8_t>(0, y);
 				for (int x = 0; x < w; ++x, p += 4)
-					pixels->push_back(to_argb_value(p[3], p[2], p[1], p[0]));
+					pixels->push_back(value(to_argb(p[3], p[2], p[1], p[0])));
 			}
 			break;
 
@@ -538,17 +538,85 @@ get_pixels (auto* pixels, const Rays::Bitmap& bmp)
 	}
 }
 
-static
-RUCY_DEF1(set_pixels, pixels)
+static Value
+get_32bit_pixels_string (const Rays::Bitmap& bmp)
 {
-	CHECK;
+	// avoid SEGV caused by 32bit argb value on 'x64-mingw-ucrt' platform.
+
+	const auto& cs = bmp.color_space();
+	if (cs.bpp() != 32) return nil();
+
+	int w = bmp.width(), h = bmp.height();
 
-	if (sizeof(VALUE) <= 4)
+	std::vector<uint32_t> pixels;
+	pixels.reserve(w * h);
+
+	switch (cs.type())
 	{
-		not_implemented_error(
-			__FILE__, __LINE__, "Bitmap#pixels=() does not support 32-bit platforms");
+		case Rays::GRAY_32:
+		case Rays::ALPHA_32:
+			for (int y = 0; y < h; ++y)
+			{
+				const auto* p = bmp.at<uint32_t>(0, y);
+				for (int x = 0; x < w; ++x, ++p)
+					pixels.push_back(*p);
+			}
+			break;
+
+		case Rays::RGBA_8888:
+		case Rays::RGBX_8888:
+			for (int y = 0; y < h; ++y)
+			{
+				const auto* p = bmp.at<uint8_t>(0, y);
+				for (int x = 0; x < w; ++x, p += 4)
+					pixels.push_back(to_argb(p[0], p[1], p[2], p[3]));
+			}
+			break;
+
+		case Rays::ARGB_8888:
+		case Rays::XRGB_8888:
+			for (int y = 0; y < h; ++y)
+			{
+				const auto* p = bmp.at<uint8_t>(0, y);
+				for (int x = 0; x < w; ++x, p += 4)
+					pixels.push_back(to_argb(p[1], p[2], p[3], p[0]));
+			}
+			break;
+
+		case Rays::BGRA_8888:
+		case Rays::BGRX_8888:
+			for (int y = 0; y < h; ++y)
+			{
+				const auto* p = bmp.at<uint8_t>(0, y);
+				for (int x = 0; x < w; ++x, p += 4)
+					pixels.push_back(to_argb(p[2], p[1], p[0], p[3]));
+			}
+			break;
+
+		case Rays::ABGR_8888:
+		case Rays::XBGR_8888:
+			for (int y = 0; y < h; ++y)
+			{
+				const auto* p = bmp.at<uint8_t>(0, y);
+				for (int x = 0; x < w; ++x, p += 4)
+					pixels.push_back(to_argb(p[3], p[2], p[1], p[0]));
+			}
+			break;
+
+		default:
+			return nil();
 	}
 
+	return value(
+		(const char*) &pixels[0], pixels.size() * sizeof(uint32_t),
+		rb_ascii8bit_encoding());
+}
+
+static
+RUCY_DEF1(set_pixels, pixels)
+{
+	CHECK;
+
 	set_pixels(THIS, pixels);
 	return pixels;
 }
@@ -559,11 +627,10 @@ RUCY_DEF0(get_pixels)
 {
 	CHECK;
 
-	if (sizeof(VALUE) <= 4)
-	{
-		not_implemented_error(
-			__FILE__, __LINE__, "Bitmap#pixels() does not support 32-bit platforms");
-	}
+#ifdef RAYS_32BIT_PIXELS_STRING
+	Value str = get_32bit_pixels_string(*THIS);
+	if (str) return str;
+#endif
 
 	std::vector<VALUE> pixels;
 	get_pixels(&pixels, *THIS);
@@ -613,7 +680,7 @@ Init_rays_bitmap ()
 	cBitmap.define_method("height", height);
 	cBitmap.define_method("color_space", color_space);
 	cBitmap.define_method("pixels=", set_pixels);
-	cBitmap.define_method("pixels",  get_pixels);
+	cBitmap.define_method("pixels!", get_pixels);
 	cBitmap.define_method("[]=", set_at);
 	cBitmap.define_method("[]",  get_at);
 }

data/ext/rays/bounds.cpp

@@ -6,7 +6,7 @@
 #include "defs.h"
 
 
-RUCY_DEFINE_VALUE_OR_ARRAY_FROM_TO(Rays::Bounds)
+RUCY_DEFINE_VALUE_OR_ARRAY_FROM_TO(RAYS_EXPORT, Rays::Bounds)
 
 #define THIS  to<Rays::Bounds*>(self)
 
@@ -75,7 +75,7 @@ RUCY_DEFN(move_to)
 	CHECK;
 	check_arg_count(__FILE__, __LINE__, "Bounds#move_to", argc, 1, 2, 3);
 
-	if (argv[0].is_kind_of(Rays::point_class()))
+	if (argv[0].is_a(Rays::point_class()))
 		THIS->move_to(to<Rays::Point&>(argv[0]));
 	else
 	{
@@ -102,7 +102,7 @@ RUCY_DEFN(move_by)
 	CHECK;
 	check_arg_count(__FILE__, __LINE__, "Bounds#move_by", argc, 1, 2, 3);
 
-	if (argv[0].is_kind_of(Rays::point_class()))
+	if (argv[0].is_a(Rays::point_class()))
 		THIS->move_by(to<Rays::Point&>(argv[0]));
 	else
 	{
@@ -128,7 +128,7 @@ RUCY_DEFN(resize_to)
 	CHECK;
 	check_arg_count(__FILE__, __LINE__, "Bounds#resize_to", argc, 1, 2, 3);
 
-	if (argv[0].is_kind_of(Rays::point_class()))
+	if (argv[0].is_a(Rays::point_class()))
 		THIS->resize_to(to<Rays::Point&>(argv[0]));
 	else
 	{
@@ -155,7 +155,7 @@ RUCY_DEFN(resize_by)
 	CHECK;
 	check_arg_count(__FILE__, __LINE__, "Bounds#resize_by", argc, 1, 2, 3);
 
-	if (argv[0].is_kind_of(Rays::point_class()))
+	if (argv[0].is_a(Rays::point_class()))
 		THIS->resize_by(to<Rays::Point&>(argv[0]));
 	else
 	{
@@ -181,7 +181,7 @@ RUCY_DEFN(inset_by)
 	CHECK;
 	check_arg_count(__FILE__, __LINE__, "Bounds#inset_by", argc, 1, 2, 3);
 
-	if (argv[0].is_kind_of(Rays::point_class()))
+	if (argv[0].is_a(Rays::point_class()))
 		THIS->inset_by(to<Rays::Point&>(argv[0]));
 	else
 	{
@@ -501,7 +501,9 @@ RUCY_DEF2(set_at, index, value)
 	CHECK;
 
 	int i = index.as_i();
-	if (i < 0 || 1 < i)
+	if (i < 0)
+		index_error(__FILE__, __LINE__);
+	if (i > 1)
 		index_error(__FILE__, __LINE__);
 
 	(*THIS)[i] = to<Rays::Point&>(value);
@@ -515,7 +517,9 @@ RUCY_DEF1(get_at, index)
 	CHECK;
 
 	int i = index.as_i();
-	if (i < 0 || 1 < i)
+	if (i < 0)
+		index_error(__FILE__, __LINE__);
+	if (i > 1)
 		index_error(__FILE__, __LINE__);
 
 	return value((*THIS)[i]);
@@ -539,9 +543,9 @@ RUCY_DEF1(op_or, arg)
 	CHECK;
 
 	Rays::Bounds b = *THIS;
-	if (arg.is_kind_of(Rays::bounds_class()))
+	if (arg.is_a(Rays::bounds_class()))
 		b |= to<Rays::Bounds&>(arg);
-	else if (arg.is_kind_of(Rays::point_class()))
+	else if (arg.is_a(Rays::point_class()))
 		b |= to<Rays::Point&>(arg);
 	else
 		argument_error(__FILE__, __LINE__);
@@ -629,7 +633,7 @@ namespace Rucy
 {
 
 
-	template <> Rays::Bounds
+	template <> RAYS_EXPORT Rays::Bounds
 	value_to<Rays::Bounds> (int argc, const Value* argv, bool convert)
 	{
 		if (argc == 1 && argv->is_array())
@@ -644,7 +648,7 @@ namespace Rucy
 		{
 			if (argc == 0)
 				return Rays::Bounds();
-			else if (argv->is_kind_of(Rays::point_class()))
+			else if (argv->is_a(Rays::point_class()))
 			{
 				switch (argc)
 				{

data/ext/rays/camera.cpp

@@ -5,7 +5,7 @@
 #include "defs.h"
 
 
-RUCY_DEFINE_VALUE_FROM_TO(Rays::Camera)
+RUCY_DEFINE_VALUE_FROM_TO(RAYS_EXPORT, Rays::Camera)
 
 #define THIS  to<Rays::Camera*>(self)