local-fastimage_resize 3.0.1 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 35d30f9c5c3d94e005b8fc135fc1dd05b7f6673d
-  data.tar.gz: e49f5802cd1dc2f9122a594659a1b018acb3ff3c
+  metadata.gz: ca87a23d6291dfd4c29c89cfcd301359a9c5f462
+  data.tar.gz: e23c3c7cb8d65db9a0d69975a6caade81a5daef6
 SHA512:
-  metadata.gz: f5e113bafa6aa697a41c0e27e8b0f3e19005643093dae7c31b59cfd5f4ce5e1597d3ef0d45680691f1a1abf25e8afa4b88a588156eafee578ae3b4c429101248
-  data.tar.gz: bb2dfa53b43dd1203a56d44fe6bec054110b44007e9cfaba432c58d0ad2ff8bc4ac52d5426b1d38b6151123c8227a1c1f9c655961deb6666cce7bc7607577d8d
+  metadata.gz: 3e8089f8e0dbcadb8a7d0a724a527a01c3e07216b671b5cafbf769b2bc5c646c15cfd94d16d8d85c90d21737ecf37a207934732039ab4512a2f4cc6cd8554d5d
+  data.tar.gz: 98958df3a3f1092189898370580be076ad2e0ca725dfe0570fad73d50a51e4c2c2afdc88ae8f17ceefd551d28a350a84644126b13450a3c8b00327bfcb45c067

data/.gitignore

@@ -1,2 +1,5 @@
 /Gemfile.lock
 /pkg/
+/tmp/
+/lib/fastimage/resize/native_resize.bundle
+/lib/fastimage/resize/native_resize.so

data/.travis.yml

@@ -0,0 +1,16 @@
+language: ruby
+rvm:
+  - 2.1.9
+  - 2.2.5
+  - 2.3.1
+
+before_install:
+  - gem install bundler
+
+cache: bundler
+sudo: false
+
+addons:
+  apt:
+    packages:
+    - libgd2-noxpm-dev

data/CHANGELOG.md

@@ -1,7 +1,11 @@
+# 3.1.0 - 2016-06-03
+
+Replacing RubyInline with native extension.
+
 # 3.0.1 - 2016-06-02
 
 Updating meta data in gemspec, no code changes.
 
 # 3.0.0 - 2016-06-02
 
-Removing remote image support
+Removing remote image support.

data/README.md

@@ -6,8 +6,12 @@ Resize](https://github.com/sdsykes/fastimage_resize) gem.
 It features the following differences:
 
 * Removal of all remote image handling code
+* Replacing RubyInline with native extension
 * Minor changes to code organization
 
+[![Build
+Status](https://travis-ci.org/planio-gmbh/local-fastimage_resize.svg?branch=master)](https://travis-ci.org/planio-gmbh/local-fastimage_resize)
+
 ## Installation
 
 Add this line to your application's Gemfile:
@@ -31,6 +35,19 @@ Or install it yourself as:
 Again, make sure to `require "fastimage/resize"`.
 
 
+### External dependencies
+
+`local-fastimage_resize`, just as `fastimage_resize`, depends on
+[libgd](http://www.libgd.org). Therefore you need to have the development
+headers installed on your system.
+
+* *Mac OS* (Homebrew): `brew install gd`
+* *Debian*: `apt-get install libgd-dev`
+* *Ubuntu*: `apt-get install libgd2-noxpm-dev`
+
+The Ubuntu package with XPM support will work just as well. It's just, that
+`fastimage_resize` will not make any use of it.
+
 
 ## Usage
 

data/Rakefile

@@ -1,4 +1,5 @@
 require "bundler/gem_tasks"
+
 require "rake/testtask"
 
 Rake::TestTask.new(:test) do |t|
@@ -7,4 +8,14 @@ Rake::TestTask.new(:test) do |t|
   t.test_files = FileList['test/**/*_test.rb']
 end
 
+require "rake/extensiontask"
+
+Rake::ExtensionTask.new do |ext|
+  ext.name = "native_resize"
+  ext.ext_dir = "ext/fastimage_resize"
+  ext.lib_dir = "lib/fastimage/resize"
+end
+
+Rake::Task[:test].prerequisites << :compile
+
 task :default => :test

data/bin/console

@@ -0,0 +1,7 @@
+#!/usr/bin/env ruby
+
+require "bundler/setup"
+require "fastimage/resize"
+
+require "irb"
+IRB.start

data/ext/fastimage_resize/extconf.rb

@@ -0,0 +1,6 @@
+require "mkmf"
+
+abort "missing libgd" unless have_library("gd")
+
+dir_config "fastimage_resize"
+create_makefile "native_resize"

data/ext/fastimage_resize/native_resize.c

@@ -0,0 +1,104 @@
+#include <ruby.h>
+#include <gd.h>
+
+// static void d(VALUE v) {
+//     ID sym_puts = rb_intern("puts");
+//     ID sym_inspect = rb_intern("inspect");
+//     rb_funcall(rb_mKernel, sym_puts, 1, rb_funcall(v, sym_inspect, 0));
+// }
+
+static VALUE native_resize(VALUE self, VALUE rb_in, VALUE rb_out, VALUE rb_w, VALUE rb_h, VALUE rb_image_type, VALUE rb_jpeg_quality) {
+  char *filename_in  = StringValuePtr(rb_in);
+  char *filename_out = StringValuePtr(rb_out);
+  int w              = NUM2INT(rb_w);
+  int h              = NUM2INT(rb_h);
+  int image_type     = NUM2INT(rb_image_type);
+  int jpeg_quality   = NUM2INT(rb_jpeg_quality);
+
+
+  gdImagePtr im_in, im_out;
+  FILE *in, *out;
+  int trans = 0, x = 0, y = 0, f = 0;
+
+  in = fopen(filename_in, "rb");
+  if (!in) return Qnil;
+
+  switch(image_type) {
+    case 0: im_in = gdImageCreateFromJpeg(in);
+            break;
+    case 1: im_in = gdImageCreateFromPng(in);
+            break;
+    case 2: im_in = gdImageCreateFromGif(in);
+            trans = gdImageGetTransparent(im_in);
+            /* find a transparent pixel, then turn off transparency
+               so that it copies correctly */
+            if (trans >= 0) {
+              for (x=0; x<gdImageSX(im_in); x++) {
+                for (y=0; y<gdImageSY(im_in); y++) {
+                  if (gdImageGetPixel(im_in, x, y) == trans) {
+                    f = 1;
+                    break;
+                  }
+                }
+                if (f) break;
+              }
+              gdImageColorTransparent(im_in, -1);
+              if (!f) trans = -1;  /* no transparent pixel found */
+            }
+            break;
+    default: return Qnil;
+  }
+
+  if (w == 0 || h == 0) {
+    int originalWidth  = gdImageSX(im_in);
+    int originalHeight = gdImageSY(im_in);
+    if (w == 0) {
+      w = (int)(h * originalWidth / originalHeight);
+    } else {
+      h = (int)(w * originalHeight / originalWidth);
+    }
+  }
+
+  im_out = gdImageCreateTrueColor(w, h);  /* must be truecolor */
+
+  if (image_type == 1) {
+    gdImageAlphaBlending(im_out, 0);  /* handle transparency correctly */
+    gdImageSaveAlpha(im_out, 1);
+  }
+
+  fclose(in);
+
+  /* Now copy the original */
+  gdImageCopyResampled(im_out, im_in, 0, 0, 0, 0,
+    gdImageSX(im_out), gdImageSY(im_out),
+    gdImageSX(im_in), gdImageSY(im_in));
+
+  out = fopen(filename_out, "wb");
+  if (out) {
+    switch(image_type) {
+      case 0: gdImageJpeg(im_out, out, jpeg_quality);
+              break;
+      case 1: gdImagePng(im_out, out);
+              break;
+      case 2: gdImageTrueColorToPalette(im_out, 0, 256);
+              if (trans >= 0) {
+                trans = gdImageGetPixel(im_out, x, y);  /* get the color index of our transparent pixel */
+                gdImageColorTransparent(im_out, trans); /* may not always work as hoped */
+              }
+              gdImageGif(im_out, out);
+              break;
+    }
+    fclose(out);
+  }
+  gdImageDestroy(im_in);
+  gdImageDestroy(im_out);
+  return Qnil;
+}
+
+void Init_native_resize(void) {
+  VALUE cFastImageResize;
+
+  cFastImageResize = rb_const_get(rb_cObject, rb_intern("FastImage"));
+
+  rb_define_singleton_method(cFastImageResize, "native_resize", native_resize, 6);
+}

data/lib/fastimage/resize.rb

@@ -19,7 +19,6 @@
 #
 # * http://blog.new-bamboo.co.uk/2007/12/3/super-f-simple-resizing
 
-require 'inline'
 require 'tempfile'
 require 'fastimage'
 
@@ -75,103 +74,13 @@ module FastImage::Resize
       temp_file = nil
     end
 
-    resize_image(path, file_out.to_s, width.to_i, height.to_i, type_index, jpeg_quality.to_i)
+    FastImage.native_resize(path, file_out.to_s, width.to_i, height.to_i, type_index, jpeg_quality.to_i)
 
     temp_file
   rescue RuntimeError => e
     raise ImageFetchFailure, e.message
   end
-
-  private
-
-  def resize_image(filename_in, filename_out, w, h, image_type, jpeg_quality); end
-
-  inline do |builder|
-    builder.include '"gd.h"'
-    builder.add_link_flags "-lgd"
-
-    builder.c <<-"END"
-      VALUE resize_image(char *filename_in, char *filename_out, int w, int h, int image_type, int jpeg_quality) {
-        gdImagePtr im_in, im_out;
-        FILE *in, *out;
-        int trans = 0, x = 0, y = 0, f = 0;
-
-        in = fopen(filename_in, "rb");
-        if (!in) return Qnil;
-
-        switch(image_type) {
-          case 0: im_in = gdImageCreateFromJpeg(in);
-                  break;
-          case 1: im_in = gdImageCreateFromPng(in);
-                  break;
-          case 2: im_in = gdImageCreateFromGif(in);
-                  trans = gdImageGetTransparent(im_in);
-                  /* find a transparent pixel, then turn off transparency
-                     so that it copies correctly */
-                  if (trans >= 0) {
-                    for (x=0; x<gdImageSX(im_in); x++) {
-                      for (y=0; y<gdImageSY(im_in); y++) {
-                        if (gdImageGetPixel(im_in, x, y) == trans) {
-                          f = 1;
-                          break;
-                        }
-                      }
-                      if (f) break;
-                    }
-                    gdImageColorTransparent(im_in, -1);
-                    if (!f) trans = -1;  /* no transparent pixel found */
-                  }
-                  break;
-          default: return Qnil;
-        }
-
-        if (w == 0 || h == 0) {
-          int originalWidth  = gdImageSX(im_in);
-          int originalHeight = gdImageSY(im_in);
-          if (w == 0) {
-            w = (int)(h * originalWidth / originalHeight);
-          } else {
-            h = (int)(w * originalHeight / originalWidth);
-          }
-        }
-
-        im_out = gdImageCreateTrueColor(w, h);  /* must be truecolor */
-
-        if (image_type == 1) {
-          gdImageAlphaBlending(im_out, 0);  /* handle transparency correctly */
-          gdImageSaveAlpha(im_out, 1);
-        }
-
-        fclose(in);
-
-        /* Now copy the original */
-        gdImageCopyResampled(im_out, im_in, 0, 0, 0, 0,
-          gdImageSX(im_out), gdImageSY(im_out),
-          gdImageSX(im_in), gdImageSY(im_in));
-
-        out = fopen(filename_out, "wb");
-        if (out) {
-          switch(image_type) {
-            case 0: gdImageJpeg(im_out, out, jpeg_quality);
-                    break;
-            case 1: gdImagePng(im_out, out);
-                    break;
-            case 2: gdImageTrueColorToPalette(im_out, 0, 256);
-                    if (trans >= 0) {
-                      trans = gdImageGetPixel(im_out, x, y);  /* get the color index of our transparent pixel */
-                      gdImageColorTransparent(im_out, trans); /* may not always work as hoped */
-                    }
-                    gdImageGif(im_out, out);
-                    break;
-          }
-          fclose(out);
-        }
-        gdImageDestroy(im_in);
-        gdImageDestroy(im_out);
-        return Qnil;
-      }
-    END
-  end
 end
 
+require 'fastimage/resize/native_resize'
 FastImage.send(:include, FastImage::Resize)

data/lib/fastimage/resize/version.rb

@@ -1,5 +1,5 @@
 class FastImage
   module Resize
-    VERSION = "3.0.1"
+    VERSION = "3.1.0"
   end
 end

data/local-fastimage_resize.gemspec

@@ -20,10 +20,12 @@ Gem::Specification.new do |s|
   s.files = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   s.require_paths = ["lib"]
 
-  s.add_dependency "RubyInline", ">= 3.8.2"
+  s.extensions = "ext/fastimage_resize/extconf.rb"
+
   s.add_dependency "local-fastimage", "~> 3.0"
 
   s.add_development_dependency "bundler", "~> 1.12"
   s.add_development_dependency "rake", "~> 10.0"
   s.add_development_dependency "minitest", "~> 5.0"
+  s.add_development_dependency "rake-compiler", "~> 0.9.9"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: local-fastimage_resize
 version: !ruby/object:Gem::Version
-  version: 3.0.1
+  version: 3.1.0
 platform: ruby
 authors:
 - Stephen Sykes
@@ -9,22 +9,8 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-02 00:00:00.000000000 Z
+date: 2016-06-03 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: RubyInline
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 3.8.2
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 3.8.2
 - !ruby/object:Gem::Dependency
   name: local-fastimage
   requirement: !ruby/object:Gem::Requirement
@@ -81,6 +67,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '5.0'
+- !ruby/object:Gem::Dependency
+  name: rake-compiler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.9.9
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.9.9
 description: Local FastImage Resize is an extremely light solution for resizing images
   in ruby by using libgd.
 email:
@@ -88,10 +88,12 @@ email:
 - gregor@plan.io
 - support@plan.io
 executables: []
-extensions: []
+extensions:
+- ext/fastimage_resize/extconf.rb
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - MIT_LICENCE
@@ -100,6 +102,9 @@ files:
 - README.textile
 - Rakefile
 - VERSION.yml
+- bin/console
+- ext/fastimage_resize/extconf.rb
+- ext/fastimage_resize/native_resize.c
 - lib/fastimage/resize.rb
 - lib/fastimage/resize/version.rb
 - local-fastimage_resize.gemspec