rmagick 5.5.0 → 6.0.0

data/CHANGELOG.md

@@ -3,6 +3,40 @@
 All notable changes to this project are documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## RMagick 6.0.0
+
+Improvements
+
+- Improve compatibility of Image#pixel_color value for ImageMagick 6 and 7 (#1591)
+- Add missing constants (#1580)
+- Loosen ImageMagick version check between compiled and runtime (#1526)
+
+Bug Fixes
+
+- Transform image according to Info#colorspace (#1594)
+- Sync Image::Info attributes to image object (#1593)
+- Fix install error on Windows MINGW environment (#1588)
+- Fix header checks in order to use aligned_malloc expectedly (#1579)
+
+Breaking Changes
+
+- Change method that returns a color name to return a hex string (#1592)
+  - The following methods return RGBA hex string as color name. The hex string length is according to color depth.
+    - `Image#background_color`
+    - `Image#border_color`
+    - `Image#colormap`
+    - `Image#matte_color`
+    - `Image#transparent_color`
+    - `Info#background_color`
+    - `Info#border_color`
+    - `Info#matte_color`
+    - `Info#transparent_color`
+  - Change default value of argument
+    - The hex argument of `Pixel#to_color` has `true` by default.
+- Drop ruby-mswin environment support on Windows (#1587)
+- Drop Ruby 2.x support (#1540)
+- Drop ImageMagick 6.7 support (#1539)
+
 ## RMagick 5.5.0
 
 Improvements

data/Gemfile

@@ -4,21 +4,15 @@ source 'https://rubygems.org'
 gemspec
 
 gem 'pry', '~> 0.14'
-gem 'rake-compiler', '~> 1.0'
-gem 'rspec', '~> 3.8'
+gem 'rake-compiler', '~> 1.2'
+gem 'rspec', '~> 3.13'
 gem 'rspec_junit_formatter', '~> 0.6.0'
-gem 'simplecov', '~> 0.16.1'
-gem 'yard', '~> 0.9.24'
+gem 'simplecov', '~> 0.22.0'
+gem 'yard', '~> 0.9.36'
 
-if RUBY_PLATFORM !~ /mswin|mingw/
-  gem 'rubocop', '~> 0.81.0'
-  gem 'rubocop-rspec', '~> 1.38.1'
-  gem 'rubocop-performance', '~> 1.5.2'
-end
+gem 'rubocop', '~> 1.63'
+gem 'rubocop-performance', '~> 1.21'
+gem 'rubocop-rspec', '~> 2.29'
 
-if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('3.0.0')
-  # For CI
-  gem 'rbs', '~> 3.4'
-
-  gem 'steep', '~> 1.6'
-end
+gem 'rbs', '~> 3.4'
+gem 'steep', '~> 1.6'

data/README.md

@@ -42,14 +42,14 @@ These prerequisites are required for the latest version of RMagick.
 - RMagick 5.4.0 or later requires a C++ compiler.
 
 **Ruby**
-- Version 2.3 or later.
+- Version 3.0 or later.
 
 You can get Ruby from <https://www.ruby-lang.org>.
 
 Ruby must be able to build C-Extensions (e.g. MRI, Rubinius, not JRuby)
 
 **ImageMagick**
-- Version 6.7.7 or later (6.x.x).
+- Version 6.8.9 or later (6.x.x).
 - Version 7.0.8 or later (7.x.x). Require RMagick 4.1.0 or later.
 
 You can get ImageMagick from <https://imagemagick.org>.
@@ -138,7 +138,7 @@ gem install rmagick
 ### Versioning
 
 RMagick is versioned according to Semantic Versioning. For stable version
-compatible with Ruby 1.8+, use `~> 2.0`. Versions >= 3 work on Ruby >= 2.x
+compatible with Ruby 3.0+, use `~> 3.0`. Versions >= 6 work on Ruby >= 3.x
 only.
 
 Using RMagick

data/Rakefile

@@ -71,7 +71,7 @@ namespace :website do
     now = now.strftime('%m/%d/%y')
 
     lines.each do |line|
-      line.gsub!(/0\.0\.0/, Magick::VERSION)
+      line.gsub!("0.0.0", Magick::VERSION)
       line.gsub!(%r{YY/MM/DD}, now)
     end
     lines

data/before_install_linux.sh

@@ -2,8 +2,6 @@
 
 set -euox pipefail
 
-gem install bundler -v 2.3.26 # Bundler 2.4.x has dropped support for Ruby 2.3
-
 if [ -v STYLE_CHECKS ]; then
   set +ux
   exit 0
@@ -15,22 +13,14 @@ if [ ! -v IMAGEMAGICK_VERSION ]; then
   exit 1
 fi
 
-sudo apt-get clean
-sudo apt-get update
-
 # remove all existing imagemagick related packages
 sudo apt-get autoremove -y imagemagick* libmagick* --purge
 
 # install build tools, ImageMagick delegates
 sudo apt-get install -y build-essential libx11-dev libxext-dev zlib1g-dev \
-  liblcms2-dev libpng-dev libjpeg-dev libfreetype6-dev libxml2-dev \
+  liblcms2-dev libpng-dev libjpeg-dev libfreetype6-dev \
   libtiff5-dev libwebp-dev liblqr-1-0-dev vim gsfonts ghostscript
 
-if [ ! -d /usr/include/freetype ]; then
-  # If `/usr/include/freetype` is not existed, ImageMagick 6.7 configuration fails about Freetype.
-  sudo ln -sf /usr/include/freetype2 /usr/include/freetype
-fi
-
 project_dir=$(pwd)
 build_dir="${project_dir}/build-ImageMagick/ImageMagick-${IMAGEMAGICK_VERSION}"
 if [ -v CONFIGURE_OPTIONS ]; then

data/before_install_osx.sh

@@ -2,8 +2,6 @@
 
 set -euox pipefail
 
-gem install bundler -v 2.3.26 # Bundler 2.4.x has dropped support for Ruby 2.3
-
 if [ -v STYLE_CHECKS ]; then
   set +ux
   exit 0
@@ -17,10 +15,10 @@ fi
 
 export HOMEBREW_NO_AUTO_UPDATE=true
 brew uninstall --force imagemagick imagemagick@6
-brew install wget ghostscript freetype jpeg little-cms2 openexr libomp libpng libtiff liblqr libtool zlib webp
+brew install wget ghostscript freetype libtool jpeg little-cms2 openexr libomp libpng libtiff liblqr zlib webp zstd
 
-export LDFLAGS="-L$(brew --prefix libxml2)/lib -L$(brew --prefix zlib)/lib -L$(brew --prefix glib)/lib -L$(brew --prefix openexr)/lib"
-export CPPFLAGS="-I$(brew --prefix libxml2)/include -I$(brew --prefix zlib)/include -I$(brew --prefix glib)/include/glib-2.0 -I$(brew --prefix glib)/lib/glib-2.0/include -I$(brew --prefix openexr)/include/OpenEXR"
+export LDFLAGS="-L$(brew --prefix jpeg)/lib -L$(brew --prefix little-cms2)/lib -L$(brew --prefix openexr)/lib -L$(brew --prefix libomp)/lib -L$(brew --prefix libpng)/lib -L$(brew --prefix libtiff)/lib -L$(brew --prefix liblqr)/lib -L$(brew --prefix zlib)/lib -L$(brew --prefix webp)/lib -L$(brew --prefix zstd)/lib"
+export CPPFLAGS="-I$(brew --prefix jpeg)/include -I$(brew --prefix openexr)/include/OpenEXR -I$(brew --prefix libtiff)/include -I$(brew --prefix zlib)/include -I$(brew --prefix zstd)/include -I$(brew --prefix glib)/include/glib-2.0 -I$(brew --prefix glib)/lib/glib-2.0/include"
 
 project_dir=$(pwd)
 build_dir="${project_dir}/build-ImageMagick/ImageMagick-${IMAGEMAGICK_VERSION}"
@@ -43,7 +41,7 @@ build_imagemagick() {
   fi
 
   cd "${build_dir}"
-  ./configure --prefix=/usr/local "${options}" --without-raw --without-jxl --without-openjp2
+  ./configure --prefix=/usr/local "${options}" --without-raw
   make -j
 }
 
@@ -52,7 +50,7 @@ if [ ! -d "${build_dir}" ]; then
 fi
 
 cd "${build_dir}"
-make install -j
+sudo make install -j
 cd "${project_dir}"
 
 set +ux

data/ext/RMagick/extconf.rb

@@ -4,21 +4,12 @@ require 'rubygems'
 require 'mkmf'
 require 'pkg-config'
 
-module MakeMakefile
-  # Use the C++ compiler to retrieve the information needed to create a Makefile for mswin environment.
-  remove_const(:CONFTEST_C)
-  CONFTEST_C = "#{CONFTEST}.cpp"
-end
-
 module RMagick
   class Extconf
     require 'rmagick/version'
     RMAGICK_VERS = ::Magick::VERSION
     MIN_RUBY_VERS = ::Magick::MIN_RUBY_VERSION
 
-    # ImageMagick 6.7 package
-    IM6_7_PACKAGES = ['ImageMagick'].freeze
-
     # ImageMagick 6.8+ packages
     IM6_PACKAGES = %w[
       ImageMagick-6.Q64HDRI
@@ -48,6 +39,8 @@ module RMagick
     def initialize
       @stdout = $stdout.dup
 
+      exit_failure("No longer support MSWIN environment.") if RUBY_PLATFORM.include?('mswin')
+
       setup_pkg_config_path
       assert_can_compile!
       configure_compile_options
@@ -55,7 +48,7 @@ module RMagick
     end
 
     def setup_pkg_config_path
-      return if RUBY_PLATFORM =~ /mswin|mingw/
+      return if RUBY_PLATFORM.include?('mingw')
 
       if find_executable('brew')
         append_pkg_config_path("#{`brew --prefix imagemagick`.strip}/lib/pkgconfig")
@@ -93,11 +86,25 @@ module RMagick
       else
         exit_failure "Can't install RMagick #{RMAGICK_VERS}. Can't find magick/MagickCore.h."
       end
+
+      if have_header('malloc.h')
+        headers << 'malloc.h'
+      elsif have_header('malloc/malloc.h')
+        headers << 'malloc/malloc.h'
+      end
     end
 
     def configure_compile_options
       # Magick-config is not available on Windows
-      if RUBY_PLATFORM !~ /mswin|mingw/
+      if RUBY_PLATFORM.include?('mingw') # mingw
+
+        dir_paths = search_paths_for_windows
+        $CPPFLAGS += %( -I"#{dir_paths[:include]}")
+        $CPPFLAGS += ' -x c++ -std=c++11 -Wno-register'
+        $LDFLAGS += %( -L"#{dir_paths[:root]}" -lucrt)
+        $LDFLAGS += (im_version_at_least?('7.0.0') ? ' -lCORE_RL_MagickCore_' : ' -lCORE_RL_magick_')
+
+      else
 
         check_multiple_imagemagick_versions
         check_partial_imagemagick_versions
@@ -119,31 +126,9 @@ module RMagick
           $LDFLAGS = "#{original_ldflags} #{ldflags}"
         end
 
-        configure_archflags_for_osx($magick_package) if RUBY_PLATFORM =~ /darwin/ # osx
-
-      elsif RUBY_PLATFORM =~ /mingw/ # mingw
-
-        dir_paths = search_paths_for_library_for_windows
-        $CPPFLAGS += %( -I"#{dir_paths[:include]}")
-        $CPPFLAGS += ' -x c++ -std=c++11 -Wno-register'
-        $LDFLAGS += %( -L"#{dir_paths[:lib]}")
-        $LDFLAGS << ' -lucrt' if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.4.0')
-
-        have_library(im_version_at_least?('7.0.0') ? 'CORE_RL_MagickCore_' : 'CORE_RL_magick_')
+        configure_archflags_for_osx($magick_package) if RUBY_PLATFORM.include?('darwin') # osx
 
-      else # mswin
-
-        dir_paths = search_paths_for_library_for_windows
-        $CPPFLAGS << %( -I"#{dir_paths[:include]}")
-        $LDFLAGS << %( -libpath:"#{dir_paths[:lib]}")
-        $LDFLAGS << ' -libpath:ucrt' if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.4.0')
-
-        $LOCAL_LIBS += ' ' + (im_version_at_least?('7.0.0') ? 'CORE_RL_MagickCore_.lib' : 'CORE_RL_magick_.lib')
-
-        $CPPFLAGS += ' /std:c++11'
       end
-      ruby_version = RUBY_VERSION.split('.')
-      $CPPFLAGS += " -DRUBY_VERSION_MAJOR=#{ruby_version[0]} -DRUBY_VERSION_MINOR=#{ruby_version[1]}"
       $CPPFLAGS += ' $(optflags) $(debugflags)'
     end
 
@@ -181,11 +166,6 @@ module RMagick
     def determine_imagemagick_package
       packages = [installed_im7_packages, installed_im6_packages].flatten
 
-      if packages.empty?
-        # ImageMagick 6.7 does not have package file like ImageMagick-6.Q16.pc
-        packages = detect_imagemagick_packages(IM6_7_PACKAGES)
-      end
-
       if packages.empty?
         exit_failure "Can't install RMagick #{RMAGICK_VERS}. Can't find ImageMagick with pkg-config\n"
       end
@@ -269,10 +249,10 @@ module RMagick
     # issue #169
     # set ARCHFLAGS appropriately for OSX
     def configure_archflags_for_osx(magick_package)
-      return unless PKGConfig.libs_only_L(magick_package).match(%r{-L(.+)/lib})
+      return unless PKGConfig.libs_only_L(magick_package) =~ %r{-L(.+)/lib}
 
       imagemagick_dir = Regexp.last_match(1)
-      command = Dir.glob(File.join(imagemagick_dir, "bin/*")).select { |file| File.executable? file }.first
+      command = Dir.glob(File.join(imagemagick_dir, "bin/*")).find { |file| File.executable? file }
       fileinfo = `file #{command}`
 
       # default ARCHFLAGS
@@ -286,28 +266,27 @@ module RMagick
       $ARCH_FLAG = archflags.join(' ') unless archflags.empty?
     end
 
-    def search_paths_for_library_for_windows
+    def search_paths_for_windows
       msg = 'searching PATH for the ImageMagick library...'
       Logging.message msg
       message msg + "\n"
 
-      found_lib = false
+      found = false
       dir_paths = {}
 
       paths = ENV['PATH'].split(File::PATH_SEPARATOR)
       paths.each do |dir|
-        lib = File.join(dir, 'lib')
-        lib_file = File.join(lib, im_version_at_least?('7.0.0') ? 'CORE_RL_MagickCore_.lib' : 'CORE_RL_magick_.lib')
-        next unless File.exist?(lib_file)
+        dll = File.join(dir, im_version_at_least?('7.0.0') ? 'CORE_RL_MagickCore_.dll' : 'CORE_RL_magick_.dll')
+        next unless File.exist?(dll)
 
         dir_paths[:include] = File.join(dir, 'include')
-        dir_paths[:lib] = lib
+        dir_paths[:root] = dir
 
-        found_lib = true
+        found = true
         break
       end
 
-      return dir_paths if found_lib
+      return dir_paths if found
 
       exit_failure <<~END_MINGW
         Can't install RMagick #{RMAGICK_VERS}.
@@ -322,7 +301,7 @@ module RMagick
       assert_has_dev_libs!
 
       # Check for compiler. Extract first word so ENV['CXX'] can be a program name with arguments.
-      cxx = (ENV['CXX'] || RbConfig::CONFIG['CXX'] || 'g++').split(' ').first
+      cxx = (ENV['CXX'] || RbConfig::CONFIG['CXX'] || 'g++').split.first
       exit_failure "No C++ compiler found in ${ENV['PATH']}. See mkmf.log for details." unless find_executable(cxx)
     end
 
@@ -341,17 +320,17 @@ module RMagick
         Check the mkmf.log file for more detailed information.
       END_FAILURE
 
-      if RUBY_PLATFORM !~ /mswin|mingw/
+      if RUBY_PLATFORM.include?('mingw')
+        `#{magick_command} -version` =~ /Version: ImageMagick (\d+\.\d+\.\d+)-+\d+ /
+        $magick_version = Regexp.last_match(1)
+        exit_failure failure_message unless $magick_version
+      else
         unless PKGConfig.libs('MagickCore')[/\bl\s*(MagickCore|Magick)6?\b/]
           exit_failure failure_message
         end
 
         $magick_package = determine_imagemagick_package
         $magick_version = PKGConfig.modversion($magick_package)[/^(\d+\.\d+\.\d+)/]
-      else
-        `#{magick_command} -version` =~ /Version: ImageMagick (\d+\.\d+\.\d+)-+\d+ /
-        $magick_version = Regexp.last_match(1)
-        exit_failure failure_message unless $magick_version
       end
 
       # Ensure minimum ImageMagick version
@@ -365,8 +344,6 @@ module RMagick
 
     def create_header_file
       ruby_api = [
-        'rb_gc_adjust_memory_usage', # Ruby 2.4.0
-        'rb_gc_mark_movable', # Ruby 2.7.0
         'rb_io_path' # Ruby 3.2.0
       ]
       memory_api = %w[
@@ -386,15 +363,10 @@ module RMagick
         have_func(func, headers)
       end
 
-      unless have_header('malloc.h')
-        have_header('malloc/malloc.h')
-      end
-
       # Miscellaneous constants
       $defs.push("-DRUBY_VERSION_STRING=\"ruby #{RUBY_VERSION}\"")
       $defs.push("-DRMAGICK_VERSION_STRING=\"RMagick #{RMAGICK_VERS}\"")
 
-      $defs.push('-DIMAGEMAGICK_GREATER_THAN_EQUAL_6_8_9=1') if im_version_at_least?('6.8.9')
       $defs.push('-DIMAGEMAGICK_GREATER_THAN_EQUAL_6_9_0=1') if im_version_at_least?('6.9.0')
       $defs.push('-DIMAGEMAGICK_GREATER_THAN_EQUAL_6_9_10=1') if im_version_at_least?('6.9.10')
       $defs.push('-DIMAGEMAGICK_7=1') if im_version_at_least?('7.0.0')
@@ -415,7 +387,7 @@ module RMagick
     end
 
     def create_compile_flags_txt
-      cppflags = $CPPFLAGS.split(' ')
+      cppflags = $CPPFLAGS.split
       include_flags = cppflags.select { |flag| flag.start_with?('-I') }
       define_flags = cppflags.select { |flag| flag.start_with?('-D') } + $defs
 

data/ext/RMagick/rmagick.h

@@ -310,10 +310,8 @@ typedef enum _QuantumExpressionOperator
     AbsQuantumOperator, /**< abs */
     ExponentialQuantumOperator, /**< exponential */
     MedianQuantumOperator, /**< median */
-    SumQuantumOperator /**< sum */
-#if defined(IMAGEMAGICK_GREATER_THAN_EQUAL_6_8_9)
-    , RootMeanSquareQuantumOperator /** root mean square */
-#endif
+    SumQuantumOperator, /**< sum */
+    RootMeanSquareQuantumOperator /** root mean square */
 } QuantumExpressionOperator ;
 
 
@@ -1227,13 +1225,8 @@ MagickExport void UnityAddKernelInfo(KernelInfo *kernel, const double scale);
 MagickExport void ScaleKernelInfo(KernelInfo *kernel, const double scaling_factor, const GeometryFlags normalize_flags);
 #endif
 
-#if (RUBY_VERSION_MAJOR == 2 && RUBY_VERSION_MINOR < 7)
-    #define RESCUE_FUNC(func)                   (VALUE (*)(ANYARGS))(func)
-    #define RESCUE_EXCEPTION_HANDLER_FUNC(func) (VALUE (*)(ANYARGS))(func)
-#else
-    #define RESCUE_FUNC(func)                   (VALUE(*)(VALUE))(func)
-    #define RESCUE_EXCEPTION_HANDLER_FUNC(func) (VALUE(*)(VALUE, VALUE))(func)
-#endif
+#define RESCUE_FUNC(func)                   (VALUE(*)(VALUE))(func)
+#define RESCUE_EXCEPTION_HANDLER_FUNC(func) (VALUE(*)(VALUE, VALUE))(func)
 
 } // extern "C"
 

data/ext/RMagick/rmdraw.cpp

@@ -13,9 +13,7 @@
 #include "rmagick.h"
 #include "float.h"
 
-#ifdef HAVE_RB_GC_MARK_MOVABLE
 static void Draw_compact(void *drawptr);
-#endif
 static void Draw_mark(void *);
 static void Draw_destroy(void *);
 static size_t Draw_memsize(const void *);
@@ -28,9 +26,7 @@ const rb_data_type_t rm_draw_data_type = {
         Draw_mark,
         Draw_destroy,
         Draw_memsize,
-#ifdef HAVE_RB_GC_MARK_MOVABLE
         Draw_compact,
-#endif
     },
     0, 0,
     RUBY_TYPED_FROZEN_SHAREABLE,
@@ -1277,7 +1273,6 @@ Draw_primitive(VALUE self, VALUE primitive)
     return self;
 }
 
-#ifdef HAVE_RB_GC_MARK_MOVABLE
 /**
  * Compact the objects.
  *
@@ -1295,7 +1290,6 @@ Draw_compact(void *drawptr)
         draw->primitives = rb_gc_location(draw->primitives);
     }
 }
-#endif
 
 /**
  * Mark referenced objects.
@@ -1311,11 +1305,7 @@ Draw_mark(void *drawptr)
 
     if (draw->primitives != (VALUE)0)
     {
-#ifdef HAVE_RB_GC_MARK_MOVABLE
         rb_gc_mark_movable(draw->primitives);
-#else
-        rb_gc_mark(draw->primitives);
-#endif
     }
 }
 

data/ext/RMagick/rmimage.cpp

@@ -288,17 +288,11 @@ DEFINE_GVL_STUB2(WriteImage, const ImageInfo *, Image *);
 DEFINE_GVL_STUB4(GetImageChannelEntropy, const Image *, const ChannelType, double *, ExceptionInfo *);
 #endif
 
-#if defined(IMAGEMAGICK_GREATER_THAN_EQUAL_6_8_9)
 DEFINE_GVL_STUB3(RotationalBlurImage, const Image *, const double, ExceptionInfo *);
-#else
-DEFINE_GVL_STUB3(RadialBlurImage, const Image *, const double, ExceptionInfo *);
-#endif
 
 #if defined(IMAGEMAGICK_7)
-#elif defined(IMAGEMAGICK_GREATER_THAN_EQUAL_6_8_9)
-DEFINE_GVL_STUB4(RotationalBlurImageChannel, const Image *, const ChannelType, const double, ExceptionInfo *);
 #else
-DEFINE_GVL_STUB4(RadialBlurImageChannel, const Image *, const ChannelType, const double, ExceptionInfo *);
+DEFINE_GVL_STUB4(RotationalBlurImageChannel, const Image *, const ChannelType, const double, ExceptionInfo *);
 #endif
 
 /**
@@ -2327,6 +2321,7 @@ Image_capture(int argc, VALUE *argv, VALUE self ATTRIBUTE_UNUSED)
     rm_ensure_result(new_image);
 
     rm_set_user_artifact(new_image, image_info);
+    rm_sync_image_options(new_image, image_info);
 
     RB_GC_GUARD(info_obj);
 
@@ -7307,6 +7302,7 @@ Image_from_blob(VALUE klass ATTRIBUTE_UNUSED, VALUE blob_arg)
 
     rm_ensure_result(images);
     rm_set_user_artifact(images, info);
+    rm_sync_image_options(images, info);
 
     RB_GC_GUARD(info_obj);
 
@@ -7963,26 +7959,7 @@ Image_gray_q(VALUE self)
 #if defined(HAVE_SETIMAGEGRAY)
     return has_attribute(self, (MagickBooleanType (*)(const Image *, ExceptionInfo *))SetImageGray);
 #else
-#if defined(IMAGEMAGICK_GREATER_THAN_EQUAL_6_8_9)
     return has_attribute(self, IsGrayImage);
-#else
-    // For ImageMagick 6.7
-    Image *image;
-    ColorspaceType colorspace;
-    VALUE ret;
-
-    image = rm_check_destroyed(self);
-    colorspace = image->colorspace;
-    if (image->colorspace == sRGBColorspace || image->colorspace == TransparentColorspace) {
-        // Workaround
-        //   If image colorspace has non-RGBColorspace, IsGrayImage() always return false.
-        image->colorspace = RGBColorspace;
-    }
-
-    ret = has_attribute(self, IsGrayImage);
-    image->colorspace = colorspace;
-    return ret;
-#endif
 #endif
 }
 
@@ -9931,6 +9908,8 @@ Image_initialize(int argc, VALUE *argv, VALUE self)
     CALL_FUNC_WITHOUT_GVL(GVL_FUNC(SetImageExtent), &args);
 #endif
 
+    rm_sync_image_options(image, info);
+
     // If the caller did not supply a fill argument, call SetImageBackgroundColor
     // to fill the image using the background color. The background color can
     // be set by specifying it when creating the Info parm block.
@@ -10671,11 +10650,11 @@ Image_pixel_color(int argc, VALUE *argv, VALUE self)
         DestroyExceptionInfo(exception);
 
 #if defined(IMAGEMAGICK_7)
-        old_color.red   = GetPixelRed(image, old_pixel);
-        old_color.green = GetPixelGreen(image, old_pixel);
-        old_color.blue  = GetPixelBlue(image, old_pixel);
-        old_color.alpha = GetPixelAlpha(image, old_pixel);
-        old_color.black = GetPixelBlack(image, old_pixel);
+        old_color.red   = GetPixelRed(image, old_pixel) + 0.5;
+        old_color.green = GetPixelGreen(image, old_pixel) + 0.5;
+        old_color.blue  = GetPixelBlue(image, old_pixel) + 0.5;
+        old_color.alpha = GetPixelAlpha(image, old_pixel) + 0.5;
+        old_color.black = GetPixelBlack(image, old_pixel) + 0.5;
         return Pixel_from_PixelPacket(&old_color);
 #else
         old_color = *old_pixel;
@@ -10750,11 +10729,11 @@ Image_pixel_color(int argc, VALUE *argv, VALUE self)
     if (pixel)
     {
 #if defined(IMAGEMAGICK_7)
-        old_color.red   = GetPixelRed(image, pixel);
-        old_color.green = GetPixelGreen(image, pixel);
-        old_color.blue  = GetPixelBlue(image, pixel);
-        old_color.alpha = GetPixelAlpha(image, pixel);
-        old_color.black = GetPixelBlack(image, pixel);
+        old_color.red   = GetPixelRed(image, pixel) + 0.5;
+        old_color.green = GetPixelGreen(image, pixel) + 0.5;
+        old_color.blue  = GetPixelBlue(image, pixel) + 0.5;
+        old_color.alpha = GetPixelAlpha(image, pixel) + 0.5;
+        old_color.black = GetPixelBlack(image, pixel) + 0.5;
 
         SetPixelRed(image,   new_color.red,   pixel);
         SetPixelGreen(image, new_color.green, pixel);
@@ -11185,11 +11164,9 @@ Image_quantum_operator(int argc, VALUE *argv, VALUE self)
         case SumQuantumOperator:
             qop = SumEvaluateOperator;
             break;
-#if defined(IMAGEMAGICK_GREATER_THAN_EQUAL_6_8_9)
         case RootMeanSquareQuantumOperator:
             qop = RootMeanSquareEvaluateOperator;
             break;
-#endif
     }
 
     exception = AcquireExceptionInfo();
@@ -11307,15 +11284,9 @@ Image_radial_blur(VALUE self, VALUE angle_obj)
     image = rm_check_destroyed(self);
     exception = AcquireExceptionInfo();
 
-#if defined(IMAGEMAGICK_GREATER_THAN_EQUAL_6_8_9)
     GVL_STRUCT_TYPE(RotationalBlurImage) args = { image, angle, exception };
     void *ret = CALL_FUNC_WITHOUT_GVL(GVL_FUNC(RotationalBlurImage), &args);
     new_image = reinterpret_cast<decltype(new_image)>(ret);
-#else
-    GVL_STRUCT_TYPE(RadialBlurImage) args = { image, angle, exception };
-    void *ret = CALL_FUNC_WITHOUT_GVL(GVL_FUNC(RadialBlurImage), &args);
-    new_image = reinterpret_cast<decltype(new_image)>(ret);
-#endif
     rm_check_exception(exception, new_image, DestroyOnError);
     DestroyExceptionInfo(exception);
 
@@ -11366,12 +11337,9 @@ Image_radial_blur_channel(int argc, VALUE *argv, VALUE self)
     new_image = (Image *)CALL_FUNC_WITHOUT_GVL(GVL_FUNC(RotationalBlurImage), &args);
     CHANGE_RESULT_CHANNEL_MASK(new_image);
     END_CHANNEL_MASK(image);
-#elif defined(IMAGEMAGICK_GREATER_THAN_EQUAL_6_8_9)
+#else
     GVL_STRUCT_TYPE(RotationalBlurImageChannel) args = { image, channels, angle, exception };
     new_image = (Image *)CALL_FUNC_WITHOUT_GVL(GVL_FUNC(RotationalBlurImageChannel), &args);
-#else
-    GVL_STRUCT_TYPE(RadialBlurImageChannel) args = { image, channels, angle, exception };
-    new_image = (Image *)CALL_FUNC_WITHOUT_GVL(GVL_FUNC(RadialBlurImageChannel), &args);
 #endif
     rm_check_exception(exception, new_image, DestroyOnError);
     DestroyExceptionInfo(exception);
@@ -11630,9 +11598,11 @@ rd_image(VALUE klass ATTRIBUTE_UNUSED, VALUE file, gvl_function_t fp)
 #endif
 
     rm_check_exception(exception, images, DestroyOnError);
-    rm_set_user_artifact(images, info);
     DestroyExceptionInfo(exception);
 
+    rm_set_user_artifact(images, info);
+    rm_sync_image_options(images, info);
+
     RB_GC_GUARD(info_obj);
 
     return array_from_images(images);
@@ -11774,9 +11744,10 @@ Image_read_inline(VALUE self ATTRIBUTE_UNUSED, VALUE content)
     magick_free((void *)blob);
 
     rm_check_exception(exception, images, DestroyOnError);
-
     DestroyExceptionInfo(exception);
+
     rm_set_user_artifact(images, info);
+    rm_sync_image_options(images, info);
 
     RB_GC_GUARD(info_obj);