exif 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d039e9781e3484977eaa1f57d34d5fdeb3c7422a
-  data.tar.gz: 36d459ec69a08973f845931d827bf690dd494ae0
+  metadata.gz: 62ee375b5a2f547b733036b6d45e9f07f9157a7f
+  data.tar.gz: 1777637e0b478a47d4008a0953addbbc1b24f73b
 SHA512:
-  metadata.gz: 6b5c9f0d2a64c4d345512dd50476d32231be8fa1cb4aaed0e7f9c350b26f0c4c1d38a2ad172f25644243cb4ae59e6051aa31b1f0b682e983722456afe8343eae
-  data.tar.gz: 0da8b96a388e365dfeee2d9d0bca6cd24f761a3018a6d91299b957f6831281c1acaa367df87570821f815709fdab0ac3ec7dc31f29578d61b1928b6924e1347e
+  metadata.gz: 47983ae0fb7a3ad48c5b01d973da975c0d74bd1e50abb1db4ff310a4296b29c9ee435286c80a5280f45f804524f82e2f118bce0e8b0219f5674913206e09741d
+  data.tar.gz: f026ead2e3e8c67a13cec1093bba58c7c162d7c67e347c967362ee8c19fe992eba2a077ea91ccd3130004451a1b97d72742a05bc3dcc8a73861a762d91aec272

data/.travis.yml

@@ -0,0 +1,6 @@
+language: ruby
+rvm:
+  - 2.2.0
+  - 2.1.5
+  - 2.0.0
+  - 1.9.3

data/Gemfile

@@ -1,4 +1,2 @@
 source 'https://rubygems.org'
-
-# Specify your gem's dependencies in exif.gemspec
-gemspec
+gemspec

data/README.md

@@ -1,4 +1,4 @@
-Ruby EXIF reader written in C extension.
+Ruby EXIF reader written in C extension. [![Build Status](https://travis-ci.org/tonytonyjan/exif.svg?branch=master)](https://travis-ci.org/tonytonyjan/exif)
 
 # Installation
 

data/Rakefile

@@ -1,7 +1,11 @@
 require "bundler/gem_tasks"
-
 require "rake/extensiontask"
+require 'rspec/core/rake_task'
 
 Rake::ExtensionTask.new("exif") do |ext|
   ext.lib_dir = "lib/exif"
 end
+
+RSpec::Core::RakeTask.new(:spec)
+
+task default: [:compile, :spec]

data/exif.gemspec

@@ -22,4 +22,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 1.7"
   spec.add_development_dependency "rake", "~> 10.0"
   spec.add_development_dependency "rake-compiler"
+  spec.add_development_dependency "rspec"
 end

data/ext/exif/data.c

@@ -22,7 +22,7 @@ void init_data(){
   rb_define_method(rb_cData, "[]", rb_value, 1);
 }
 
-static VALUE rb_new(VALUE self, VALUE file_path){
+VALUE rb_new(VALUE self, VALUE file_path){
   Check_Type(file_path, T_STRING);
   ExifData *ed = exif_data_new_from_file(StringValueCStr(file_path));
   if(!ed) rb_raise(rb_eRuntimeError, "File not readable or no EXIF data in file.");
@@ -32,19 +32,19 @@ static VALUE rb_new(VALUE self, VALUE file_path){
   return rb_data;
 }
 
-static VALUE rb_dump(VALUE self){
+VALUE rb_dump(VALUE self){
   ExifData *ed;
   Data_Get_Struct(self, ExifData, ed);
   exif_data_dump(ed);
   return Qnil;
 }
 
-static VALUE rb_value(VALUE self, VALUE key){
+VALUE rb_value(VALUE self, VALUE key){
   VALUE rb_contents = rb_iv_get(self, "@contents");
   return rb_hash_aref(rb_contents, key);
 }
 
-static void each_content(ExifContent *ec, void *self_ptr){
+void each_content(ExifContent *ec, void *self_ptr){
   VALUE *self = (VALUE*)self_ptr;
   VALUE rb_contents = rb_iv_get(*self, "@contents");
   Check_Type(rb_contents, T_HASH);
@@ -55,11 +55,11 @@ static void each_content(ExifContent *ec, void *self_ptr){
   exif_content_foreach_entry(ec, each_entry, self);
 }
 
-static void each_entry(ExifEntry *ee, void *self_ptr){
+void each_entry(ExifEntry *ee, void *self_ptr){
   VALUE *self = (VALUE*)self_ptr;
   VALUE rb_contents = rb_iv_get(*self, "@contents");
   ExifIfd ifd = exif_entry_get_ifd(ee);
-  char *attr_name = attr_string(ifd, ee->tag);
+  const char *attr_name = attr_string(ifd, ee->tag);
   VALUE tag_name = ID2SYM(rb_intern(attr_name + 1));
   char buf[500];
   exif_entry_get_value(ee, buf, sizeof(buf));
@@ -101,7 +101,7 @@ static void each_entry(ExifEntry *ee, void *self_ptr){
   rb_iv_set(*self, attr_name, value);
 }
 
-static VALUE process_value(VALUE *self_ptr, ExifIfd ifd, ExifTag tag, char *buf){
+VALUE process_value(VALUE *self_ptr, ExifIfd ifd, ExifTag tag, char *buf){
   ExifData *ed;
   Data_Get_Struct(*self_ptr, ExifData, ed);
   switch((int)tag){
@@ -146,7 +146,7 @@ static VALUE process_value(VALUE *self_ptr, ExifIfd ifd, ExifTag tag, char *buf)
   return rb_str_new_cstr(buf);
 }
 
-static char* attr_string(ExifIfd ifd, ExifTag tag){
+const char* attr_string(ExifIfd ifd, ExifTag tag){
   switch((int)tag){
   case EXIF_TAG_INTEROPERABILITY_INDEX: /* EXIF_TAG_GPS_LATITUDE_REF */
     return ifd == EXIF_IFD_GPS ? "@gps_latitude_ref" : "@interoperability_index";

data/ext/exif/data.h

@@ -4,17 +4,17 @@
 #include <libexif/exif-data.h>
 #include "ruby.h"
 
-static char *attrs[] = {"aperture_value", "artist", "battery_level", "bits_per_sample", "brightness_value", "cfa_pattern", "cfa_repeat_pattern_dim", "color_space", "components_configuration", "compressed_bits_per_pixel", "compression", "contrast", "copyright", "custom_rendered", "date_time", "date_time_digitized", "date_time_original", "device_setting_description", "digital_zoom_ratio", "document_name", "exif_ifd_pointer", "exif_version", "exposure_bias_value", "exposure_index", "exposure_mode", "exposure_program", "exposure_time", "file_source", "fill_order", "flash", "flash_energy", "flash_pix_version", "fnumber", "focal_length", "focal_length_in_35mm_film", "focal_plane_resolution_unit", "focal_plane_x_resolution", "focal_plane_y_resolution", "gain_control", "gamma", "gps_altitude", "gps_altitude_ref", "gps_area_information", "gps_date_stamp", "gps_dest_bearing", "gps_dest_bearing_ref", "gps_dest_distance", "gps_dest_distance_ref", "gps_dest_latitude", "gps_dest_latitude_ref", "gps_dest_longitude", "gps_dest_longitude_ref", "gps_differential", "gps_dop", "gps_img_direction", "gps_img_direction_ref", "gps_info_ifd_pointer", "gps_latitude", "gps_latitude_ref", "gps_longitude", "gps_longitude_ref", "gps_map_datum", "gps_measure_mode", "gps_processing_method", "gps_satellites", "gps_speed", "gps_speed_ref", "gps_status", "gps_time_stamp", "gps_track", "gps_track_ref", "gps_version_id", "image_description", "image_length", "image_resources", "image_unique_id", "image_width", "inter_color_profile", "interoperability_ifd_pointer", "interoperability_index", "interoperability_version", "iptc_naa", "iso_speed_ratings", "jpeg_interchange_format", "jpeg_interchange_format_length", "jpeg_proc", "light_source", "make", "maker_note", "max_aperture_value", "metering_mode", "model", "new_cfa_pattern", "new_subfile_type", "oecf", "orientation", "padding", "photometric_interpretation", "pixel_x_dimension", "pixel_y_dimension", "planar_configuration", "primary_chromaticities", "print_image_matching", "reference_black_white", "related_image_file_format", "related_image_length", "related_image_width", "related_sound_file", "resolution_unit", "rows_per_strip", "samples_per_pixel", "saturation", "scene_capture_type", "scene_type", "sensing_method", "sharpness", "shutter_speed_value", "software", "spatial_frequency_response", "spectral_sensitivity", "strip_byte_counts", "strip_offsets", "sub_ifds", "sub_sec_time", "sub_sec_time_digitized", "sub_sec_time_original", "subject_area", "subject_distance", "subject_distance_range", "subject_location", "tiff_ep_standard_id", "time_zone_offset", "transfer_function", "transfer_range", "user_comment", "white_balance", "white_point", "x_resolution", "xml_packet", "xp_author", "xp_comment", "xp_keywords", "xp_subject", "xp_title", "y_resolution", "ycbcr_coefficients", "ycbcr_positioning", "ycbcr_sub_sampling"};
+static const char *attrs[] = {"aperture_value", "artist", "battery_level", "bits_per_sample", "brightness_value", "cfa_pattern", "cfa_repeat_pattern_dim", "color_space", "components_configuration", "compressed_bits_per_pixel", "compression", "contrast", "copyright", "custom_rendered", "date_time", "date_time_digitized", "date_time_original", "device_setting_description", "digital_zoom_ratio", "document_name", "exif_ifd_pointer", "exif_version", "exposure_bias_value", "exposure_index", "exposure_mode", "exposure_program", "exposure_time", "file_source", "fill_order", "flash", "flash_energy", "flash_pix_version", "fnumber", "focal_length", "focal_length_in_35mm_film", "focal_plane_resolution_unit", "focal_plane_x_resolution", "focal_plane_y_resolution", "gain_control", "gamma", "gps_altitude", "gps_altitude_ref", "gps_area_information", "gps_date_stamp", "gps_dest_bearing", "gps_dest_bearing_ref", "gps_dest_distance", "gps_dest_distance_ref", "gps_dest_latitude", "gps_dest_latitude_ref", "gps_dest_longitude", "gps_dest_longitude_ref", "gps_differential", "gps_dop", "gps_img_direction", "gps_img_direction_ref", "gps_info_ifd_pointer", "gps_latitude", "gps_latitude_ref", "gps_longitude", "gps_longitude_ref", "gps_map_datum", "gps_measure_mode", "gps_processing_method", "gps_satellites", "gps_speed", "gps_speed_ref", "gps_status", "gps_time_stamp", "gps_track", "gps_track_ref", "gps_version_id", "image_description", "image_length", "image_resources", "image_unique_id", "image_width", "inter_color_profile", "interoperability_ifd_pointer", "interoperability_index", "interoperability_version", "iptc_naa", "iso_speed_ratings", "jpeg_interchange_format", "jpeg_interchange_format_length", "jpeg_proc", "light_source", "make", "maker_note", "max_aperture_value", "metering_mode", "model", "new_cfa_pattern", "new_subfile_type", "oecf", "orientation", "padding", "photometric_interpretation", "pixel_x_dimension", "pixel_y_dimension", "planar_configuration", "primary_chromaticities", "print_image_matching", "reference_black_white", "related_image_file_format", "related_image_length", "related_image_width", "related_sound_file", "resolution_unit", "rows_per_strip", "samples_per_pixel", "saturation", "scene_capture_type", "scene_type", "sensing_method", "sharpness", "shutter_speed_value", "software", "spatial_frequency_response", "spectral_sensitivity", "strip_byte_counts", "strip_offsets", "sub_ifds", "sub_sec_time", "sub_sec_time_digitized", "sub_sec_time_original", "subject_area", "subject_distance", "subject_distance_range", "subject_location", "tiff_ep_standard_id", "time_zone_offset", "transfer_function", "transfer_range", "user_comment", "white_balance", "white_point", "x_resolution", "xml_packet", "xp_author", "xp_comment", "xp_keywords", "xp_subject", "xp_title", "y_resolution", "ycbcr_coefficients", "ycbcr_positioning", "ycbcr_sub_sampling"};
 
 void init_data();
 
-static VALUE rb_new(VALUE self, VALUE file_path);
-static VALUE rb_dump(VALUE self);
-static VALUE rb_value(VALUE self, VALUE key);
+VALUE rb_new(VALUE self, VALUE file_path);
+VALUE rb_dump(VALUE self);
+VALUE rb_value(VALUE self, VALUE key);
 
-static void each_content(ExifContent *ec, void *user_data);
-static void each_entry(ExifEntry *, void *user_data);
-static VALUE process_value(VALUE *self_ptr, ExifIfd ifd, ExifTag tag, char *buf);
-static char* attr_string(ExifIfd ifd, ExifTag tag);
+void each_content(ExifContent *ec, void *user_data);
+void each_entry(ExifEntry *, void *user_data);
+VALUE process_value(VALUE *self_ptr, ExifIfd ifd, ExifTag tag, char *buf);
+const char* attr_string(ExifIfd ifd, ExifTag tag);
 
 #endif /* DATA_H */

data/lib/exif/version.rb

@@ -1,3 +1,3 @@
 module Exif
-  VERSION = "1.0.0"
+  VERSION = "1.0.1"
 end

data/spec/exif_spec.rb

@@ -5,10 +5,6 @@ describe Exif do
     @data = Exif::Data.new(File.expand_path('../sample.jpg', __FILE__))
   end
 
-  # it '#dump' do
-  #   expect { @data.dump }.not_to raise_error
-  # end
-
   it 'works' do
     expect(@data.model).to eq 'NIKON D600'
     expect(@data.image_width).to eq 4000

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: exif
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Jian Weihang
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-02 00:00:00.000000000 Z
+date: 2015-01-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -52,6 +52,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Ruby EXIF reader written in C extension.
 email:
 - tonytonyjan@gmail.com
@@ -62,6 +76,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".travis.yml"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -98,7 +113,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.2
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Ruby EXIF reader written in C extension.
@@ -106,3 +121,4 @@ test_files:
 - spec/exif_spec.rb
 - spec/sample.jpg
 - spec/spec_helper.rb
+has_rdoc: