rmagick 7.0.0 → 7.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0b8378b6521fc857bfee81cb0b0d234cbacd0ea753d2f9be41180f1d4f833086
-  data.tar.gz: 6492cec6f6802500827e3e0d3b0c2c30b5ab7c000b3c4da68cf5af5dd3b856c1
+  metadata.gz: dde5ecba6c9f64a65d65582b49338a8d29e8385b3734f3a959b4c690f0c9ea66
+  data.tar.gz: d133faa5b9ab9a6647b5de590538bdcebb86aacaedf1153c0bb1ac5235a63de7
 SHA512:
-  metadata.gz: 70fdb4d672961ba027ec56396813d81aa7101e32d3d0b73a42e28e12b779777e27a63e061d2fdcb82ee5ada9ceede1d3cf1f4124b51a62962e6c1eef656a3f66
-  data.tar.gz: b436f377627611cab9ca86f0b59e2fc8a9a9e8cbd3071fa06036ac841fe136717430851f2043aeaf31175835afab285c5dd2cd3efd5f8de51a8f58cdb38bb595
+  metadata.gz: 200d4179fbb2a170ac0e49ca3da399717d76eb5b66a66743fcefc3f4c75a107ecd90296c9c69446b8c7e690c966340c34b64f0c561a3826a86d3bb0d61a8eec7
+  data.tar.gz: e301f6a39bac7a1302630bbba806b225381f07716e4514c555d1d39a4e773106bcfc6f61a95cb06ff1aff11044461e448797407585db40518edab77d4f996c48

data/CHANGELOG.md

@@ -3,6 +3,14 @@
 All notable changes to this project are documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## RMagick 7.0.1
+
+Bug Fixes
+
+* Image_gray_q changes the colour space of the image, instead of returning whether it's grayscale (#1788)
+* Check for errors from ParseGeometry() (#1786)
+* Fail explicitly when trying to install on JRuby (#1784)
+
 ## RMagick 7.0.0
 
 Breaking Changes

data/ext/RMagick/extconf.rb

@@ -42,6 +42,7 @@ module RMagick
     def initialize
       @stdout = $stdout.dup
 
+      exit_failure("JRuby is not supported.\nSee https://github.com/jruby/jruby/wiki/C-Extension-Alternatives") if RUBY_ENGINE == 'jruby'
       exit_failure("No longer support MSWIN environment.") if RUBY_PLATFORM.include?('mswin')
 
       setup_pkg_config_path
@@ -283,7 +284,6 @@ module RMagick
         _aligned_msize
       ]
       imagemagick_api = [
-        'SetImageGray', # 6.9.1-10
         'SetMagickAlignedMemoryMethods' # 7.0.9-0
       ]
 

data/ext/RMagick/rmimage.cpp

@@ -7929,6 +7929,13 @@ has_image_attribute(VALUE self, MagickBooleanType (attr_test)(const Image *))
 }
 #endif
 
+static MagickBooleanType
+Image_is_gray_colorspace_type(const Image *image, ExceptionInfo *exception)
+{
+    ColorspaceType colorspace = GetImageColorspaceType(image, exception);
+    return (colorspace == GRAYColorspace || colorspace == LinearGRAYColorspace) ? MagickTrue : MagickFalse;
+}
+
 
 /**
  * Return true if all the pixels in the image have the same red, green, and blue intensities.
@@ -7938,11 +7945,7 @@ has_image_attribute(VALUE self, MagickBooleanType (attr_test)(const Image *))
 VALUE
 Image_gray_q(VALUE self)
 {
-#if defined(HAVE_SETIMAGEGRAY)
-    return has_attribute(self, (MagickBooleanType (*)(const Image *, ExceptionInfo *))SetImageGray);
-#else
-    return has_attribute(self, IsGrayImage);
-#endif
+    return has_attribute(self, Image_is_gray_colorspace_type);
 }
 
 
@@ -11355,15 +11358,24 @@ Image_random_threshold_channel(int argc, VALUE *argv, VALUE self)
     exception = AcquireExceptionInfo();
 
 #if defined(IMAGEMAGICK_7)
-    BEGIN_CHANNEL_MASK(new_image, channels);
     {
         GeometryInfo geometry_info;
 
-        ParseGeometry(thresholds, &geometry_info);
-        GVL_STRUCT_TYPE(RandomThresholdImage) args = { new_image, geometry_info.rho, geometry_info.sigma, exception };
-        CALL_FUNC_WITHOUT_GVL(GVL_FUNC(RandomThresholdImage), &args);
+        if (ParseGeometry(thresholds, &geometry_info) == NoValue && *thresholds)
+        {
+            (void) ThrowMagickException(exception, GetMagickModule(), OptionError,
+                                        "InvalidArgument", "'%s'", thresholds);
+        }
+        else
+        {
+            BEGIN_CHANNEL_MASK(new_image, channels);
+            {
+                GVL_STRUCT_TYPE(RandomThresholdImage) args = { new_image, geometry_info.rho, geometry_info.sigma, exception };
+                CALL_FUNC_WITHOUT_GVL(GVL_FUNC(RandomThresholdImage), &args);
+            }
+            END_CHANNEL_MASK(new_image);
+        }
     }
-    END_CHANNEL_MASK(new_image);
 #else
     GVL_STRUCT_TYPE(RandomThresholdImageChannel) args = { new_image, channels, thresholds, exception };
     CALL_FUNC_WITHOUT_GVL(GVL_FUNC(RandomThresholdImageChannel), &args);

data/ext/RMagick/rmkinfo.cpp

@@ -180,12 +180,17 @@ KernelInfo_builtin(VALUE self, VALUE what, VALUE geometry)
     KernelInfo *kernel;
     KernelInfoType kernel_type;
     GeometryInfo info;
+    const char *geom_str;
 #if defined(IMAGEMAGICK_7)
     ExceptionInfo *exception;
 #endif
 
     VALUE_TO_ENUM(what, kernel_type, KernelInfoType);
-    ParseGeometry(StringValueCStr(geometry), &info);
+    geom_str = StringValueCStr(geometry);
+    if (ParseGeometry(geom_str, &info) == NoValue && *geom_str)
+    {
+        rb_raise(rb_eArgError, "invalid geometry string");
+    }
 
 #if defined(IMAGEMAGICK_7)
     exception = AcquireExceptionInfo();

data/lib/rmagick/version.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Magick
-  VERSION = '7.0.0'
+  VERSION = '7.0.1'
   MIN_RUBY_VERSION = '3.2.0'
   MIN_IM6_VERSION = '6.9.0'
   MIN_IM7_VERSION = '7.1.0'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rmagick
 version: !ruby/object:Gem::Version
-  version: 7.0.0
+  version: 7.0.1
 platform: ruby
 authors:
 - Tim Hunter
@@ -143,7 +143,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements:
 - ImageMagick 6.9.0+ (for ImageMagick 6) or 7.1.0+ (for ImageMagick 7)
-rubygems_version: 4.0.11
+rubygems_version: 4.0.10
 specification_version: 4
 summary: Ruby binding to ImageMagick
 test_files: []