ffi-gdal 1.0.1 → 1.0.3

data/lib/ogr/extensions/layer/capability_methods.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'ogr/layer'
+require "ogr/layer"
 
 module OGR
   module LayerMixins
@@ -10,17 +10,17 @@ module OGR
       #   manner for this layer (as opposed to using #next_feature and
       #   #reset_reading).
       def can_random_read?
-        test_capability('RandomRead')
+        test_capability("RandomRead")
       end
 
       # @return [Boolean] +true+ if #create_feature is allowed on this layer.
       def can_sequential_write?
-        test_capability('SequentialWrite')
+        test_capability("SequentialWrite")
       end
 
       # @return [Boolean] +true+ if #feature= is allowed on this layer.
       def can_random_write?
-        test_capability('RandomWrite')
+        test_capability("RandomWrite")
       end
 
       # @return [Boolean] +true+ if this layer implements spatial filtering
@@ -28,78 +28,78 @@ module OGR
       #   it should build and maintain its own spatial index for features in
       #   this layer.
       def can_fast_spatial_filter?
-        test_capability('FastSpatialFilter')
+        test_capability("FastSpatialFilter")
       end
 
       # @return [Boolean] +true+ if this layer can return a feature count
       #   efficiently (i.e. without counting all of the features).
       def can_fast_feature_count?
-        test_capability('FastFeatureCount')
+        test_capability("FastFeatureCount")
       end
 
       # @return [Boolean] +true+ if this Layer can return its extent
       #   efficiently (i.e. without scanning all of the features.).
       def can_fast_get_extent?
-        test_capability('FastGetExtent')
+        test_capability("FastGetExtent")
       end
 
       # @return [Boolean] +true+ if this layer can perform the
       #   #next_feature_index= call efficiently.
       def can_fast_set_next_by_index?
-        test_capability('FastSetNextByIndex')
+        test_capability("FastSetNextByIndex")
       end
 
       # @return [Boolean] +true+ if new Fields can be created on this Layer.
       def can_create_field?
-        test_capability('CreateField')
+        test_capability("CreateField")
       end
 
       # @return [Boolean] +true+ if the Layer supports creating new geometry
       #   fields on the current layer.
       def can_create_geometry_field?
-        test_capability('CreateGeomField')
+        test_capability("CreateGeomField")
       end
 
       # @return [Boolean] +true+ if the Layer supports deleting existing fields
       #   on the current layer.
       def can_delete_field?
-        test_capability('DeleteField')
+        test_capability("DeleteField")
       end
 
       # @return [Boolean] +true+ if the Layer supports reording fields on the
       #   current layer.
       def can_reorder_fields?
-        test_capability('ReorderFields')
+        test_capability("ReorderFields")
       end
 
       # @return [Boolean] +true+ if the Layer supports altering the defintition
       #   of an existing field on the current layer.
       def can_alter_field_definition?
-        test_capability('AlterFieldDefn')
+        test_capability("AlterFieldDefn")
       end
 
       # @return [Boolean] +true+ if the Layer supports deleting Features.
       def can_delete_feature?
-        test_capability('DeleteFeature')
+        test_capability("DeleteFeature")
       end
 
       # @return [Boolean] +true+ if :OFTString fields are guaranteed to be in
       #   UTF-8.
       def strings_are_utf_8?
-        test_capability('StringsAsUTF8')
+        test_capability("StringsAsUTF8")
       end
 
       # @return [Boolean] +true+ if this Layer supports transactions. If not,
       #   #start_transaction, #commit_transaction, and #rollback_transaction
       #   will not work in a meaningful manner.
       def supports_transactions?
-        test_capability('Transactions')
+        test_capability("Transactions")
       end
 
       # @return [Boolean] +true+ if this Layer supports reading or writing
       #   curve geometries.
       def supports_curve_geometries?
-        test_capability('CurveGeometries')
+        test_capability("CurveGeometries")
       end
     end
   end

data/lib/ogr/extensions/layer/extensions.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'ogr/layer'
+require "ogr/layer"
 
 module OGR
   class Layer

data/lib/ogr/extensions/spatial_reference/extensions.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'ogr/spatial_reference'
+require "ogr/spatial_reference"
 
 module OGR
   class SpatialReference

data/lib/ogr/extensions/spatial_reference/initializers.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'ogr/spatial_reference'
+require "ogr/spatial_reference"
 
 module OGR
   module SpatialReferenceMixins
@@ -124,7 +124,7 @@ module OGR
           object = new(spatial_reference_or_wkt)
           ogr_err = yield object.c_pointer
 
-          OGR::ErrorHandling.handle_ogr_err('Unable to build SpatialReference') do
+          OGR::ErrorHandling.handle_ogr_err("Unable to build SpatialReference") do
             ogr_err
           end
 

data/lib/ogr/extensions/style_table/extensions.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'ogr/style_table'
+require "ogr/style_table"
 
 module OGR
   class StyleTable

data/lib/ogr/feature.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'date'
-require_relative '../ogr'
-require_relative '../gdal'
+require "date"
+require_relative "../ogr"
+require_relative "../gdal"
 
 module OGR
   class Feature
@@ -26,7 +26,7 @@ module OGR
                   fd_or_pointer.autorelease = false
                   fd_or_pointer
                 else
-                  raise OGR::InvalidFeature, 'OGR::Feature must be instantiated with valid feature'
+                  raise OGR::InvalidFeature, "OGR::Feature must be instantiated with valid feature"
                 end
 
       if !pointer.is_a?(FFI::Pointer) || pointer.null?
@@ -50,7 +50,7 @@ module OGR
     def clone
       # This new feature is owned by the caller and must be released accordingly.
       feature_ptr = FFI::OGR::API.OGR_F_Clone(@c_pointer)
-      raise OGR::Failure, 'Unable to clone feature' if feature_ptr.nil?
+      raise OGR::Failure, "Unable to clone feature" if feature_ptr.nil?
 
       OGR::Feature.new(feature_ptr)
     end
@@ -59,7 +59,7 @@ module OGR
     #
     # @param file_path [String]
     def dump_readable(file_path = nil)
-      file_ptr = file_path ? FFI::CPL::Conv.CPLOpenShared(file_path, 'w', false) : nil
+      file_ptr = file_path ? FFI::CPL::Conv.CPLOpenShared(file_path, "w", false) : nil
       FFI::OGR::API.OGR_F_DumpReadable(@c_pointer, file_ptr)
       FFI::CPL::Conv.CPLCloseShared(file_ptr) if file_ptr
     end
@@ -74,9 +74,9 @@ module OGR
     # @raise [OGR::Failure]
     # TODO: Implement +with_map+
     def set_from!(_other_feature, _be_forgiving: false, with_map: nil)
-      raise NotImplementedError, 'with_map: is not yet supported' if with_map
+      raise NotImplementedError, "with_map: is not yet supported" if with_map
 
-      OGR::ErrorHandling.handle_ogr_err('Unable to set from other feature') do
+      OGR::ErrorHandling.handle_ogr_err("Unable to set from other feature") do
         FFI::OGR::API.OGR_F_SetFrom(@c_pointer, other_feature_ptr)
       end
     end
@@ -159,7 +159,7 @@ module OGR
     # @param index [Integer]
     # @param value [String]
     def set_field_binary(index, value)
-      raise TypeError, 'value must be a binary string' unless value.is_a? String
+      raise TypeError, "value must be a binary string" unless value.is_a? String
 
       value_ptr = FFI::MemoryPointer.new(:uchar, value.length)
       value_ptr.put_bytes(0, value)
@@ -249,7 +249,7 @@ module OGR
     # @param new_geometry [OGR::Geometry]
     # @raise [OGR::Failure]
     def set_geometry(new_geometry) # rubocop:disable Naming/AccessorMethodName
-      OGR::ErrorHandling.handle_ogr_err('Unable to set geometry on feature') do
+      OGR::ErrorHandling.handle_ogr_err("Unable to set geometry on feature") do
         FFI::OGR::API.OGR_F_SetGeometry(@c_pointer, new_geometry.c_pointer)
       end
     end
@@ -261,7 +261,7 @@ module OGR
     def set_geometry_directly(new_geometry) # rubocop:disable Naming/AccessorMethodName
       new_geometry.c_pointer.autorelease = false
 
-      OGR::ErrorHandling.handle_ogr_err('Unable to set geometry directly on feature') do
+      OGR::ErrorHandling.handle_ogr_err("Unable to set geometry directly on feature") do
         FFI::OGR::API.OGR_F_SetGeometryDirectly(@c_pointer, new_geometry.c_pointer)
       end
     end
@@ -272,7 +272,7 @@ module OGR
     # @return [OGR::Geometry]
     def steal_geometry
       geometry_ptr = FFI::OGR::API.OGR_F_StealGeometry(@c_pointer)
-      raise OGR::Failure, 'Unable to steal geometry.' if geometry_ptr.nil? || geometry_ptr.null?
+      raise OGR::Failure, "Unable to steal geometry." if geometry_ptr.nil? || geometry_ptr.null?
 
       OGR::Geometry.factory(geometry_ptr)
     end
@@ -285,7 +285,7 @@ module OGR
     # @param new_fid [Integer]
     # @raise [OGR::Failure]
     def fid=(new_fid)
-      OGR::ErrorHandling.handle_ogr_err('Unable to set FID') do
+      OGR::ErrorHandling.handle_ogr_err("Unable to set FID") do
         FFI::OGR::API.OGR_F_SetFID(@c_pointer, new_fid)
       end
     end
@@ -443,9 +443,9 @@ module OGR
       binary_data_ptr.autorelease = false
 
       byte_count = byte_count_ptr.read_int
-      string = byte_count.positive? ? binary_data_ptr.read_bytes(byte_count) : ''
+      string = byte_count.positive? ? binary_data_ptr.read_bytes(byte_count) : ""
 
-      string.unpack('C*')
+      string.unpack("C*")
     end
 
     def field_as_date_time(index)

data/lib/ogr/feature_definition.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative '../gdal'
-require_relative '../ogr'
+require_relative "../gdal"
+require_relative "../ogr"
 
 module OGR
   class FeatureDefinition

data/lib/ogr/field.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require 'date'
-require_relative '../ogr'
-require_relative '../gdal'
+require "date"
+require_relative "../ogr"
+require_relative "../gdal"
 
 module OGR
   class Field
@@ -53,7 +53,7 @@ module OGR
 
     # TODO: This blows up when another value type has been set.
     def string
-      return '' if @c_struct[:string]&.null?
+      return "" if @c_struct[:string]&.null?
 
       @c_struct[:string].read_string
     end
@@ -150,7 +150,7 @@ module OGR
     def binary
       b = @c_struct[:binary]
 
-      b[:count].positive? ? b[:data].read_bytes(b[:count]) : ''
+      b[:count].positive? ? b[:data].read_bytes(b[:count]) : ""
     end
 
     # @param new_binary [String] Binary string of 8-bit, unsigned data (uchar).

data/lib/ogr/field_definition.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative '../ogr'
+require_relative "../ogr"
 
 module OGR
   class FieldDefinition

data/lib/ogr/geocoder.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative '../ogr'
+require_relative "../ogr"
 
 module OGR
   # Geocode things!  http://www.gdal.org/ogr__geocoding_8h.html

data/lib/ogr/geometries/geometry_collection.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative '../geometry_types/container'
-require_relative '../geometry_types/surface'
+require_relative "../geometry_types/container"
+require_relative "../geometry_types/surface"
 
 module OGR
   class GeometryCollection

data/lib/ogr/geometries/geometry_collection_25d.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'geometry_collection'
+require_relative "geometry_collection"
 
 module OGR
   # NOTE: {{#type}} will return :wkbGeometryCollection (read: 2D instead of

data/lib/ogr/geometries/line_string.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative '../geometry_types/curve'
+require_relative "../geometry_types/curve"
 
 module OGR
   class LineString

data/lib/ogr/geometries/line_string_25d.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'line_string'
+require_relative "line_string"
 
 module OGR
   # NOTE: {{#type}} will return :wkbLineString (read: 2D instead of 2.5D) until

data/lib/ogr/geometries/linear_ring.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'line_string'
+require_relative "line_string"
 
 module OGR
   class LinearRing < LineString
@@ -14,7 +14,7 @@ module OGR
     def to_line_string
       line_string = OGR::LineString.new
       line_string.spatial_reference = spatial_reference if spatial_reference
-      line_string.import_from_wkt(to_wkt.sub('LINEARRING', 'LINESTRING'))
+      line_string.import_from_wkt(to_wkt.sub("LINEARRING", "LINESTRING"))
 
       line_string
     end

data/lib/ogr/geometries/multi_line_string.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative '../geometry_types/curve'
-require_relative '../geometry_types/container'
+require_relative "../geometry_types/curve"
+require_relative "../geometry_types/container"
 
 module OGR
   class MultiLineString

data/lib/ogr/geometries/multi_line_string_25d.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'multi_line_string'
+require_relative "multi_line_string"
 
 module OGR
   # NOTE: {{#type}} will return :wkbMultiLineString (read: 2D instead of 2.5D)

data/lib/ogr/geometries/multi_point.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative '../geometry_types/container'
+require_relative "../geometry_types/container"
 
 module OGR
   class MultiPoint

data/lib/ogr/geometries/multi_point_25d.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'multi_point'
+require_relative "multi_point"
 
 module OGR
   # NOTE: {{#type}} will return :wkbMultiPoint (read: 2D instead of 2.5D) until

data/lib/ogr/geometries/multi_polygon.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative '../geometry_types/container'
-require_relative '../geometry_types/surface'
+require_relative "../geometry_types/container"
+require_relative "../geometry_types/surface"
 
 module OGR
   class MultiPolygon

data/lib/ogr/geometries/multi_polygon_25d.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'multi_polygon'
+require_relative "multi_polygon"
 
 module OGR
   # NOTE: {{#type}} will return :wkbMultiPolygon (read: 2D instead of 2.5D)

data/lib/ogr/geometries/point_25d.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'point'
+require_relative "point"
 
 module OGR
   # NOTE: {{#type}} will return :wkbPoint (read: 2D instead of 2.5D) until a Z

data/lib/ogr/geometries/polygon.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative '../geometry_types/container'
-require_relative '../geometry_types/surface'
+require_relative "../geometry_types/container"
+require_relative "../geometry_types/surface"
 
 module OGR
   class Polygon

data/lib/ogr/geometries/polygon_25d.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'polygon'
+require_relative "polygon"
 
 module OGR
   # NOTE: {{#type}} will return :wkbPolygon (read: 2D instead of 2.5D) until a Z

data/lib/ogr/geometry.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative '../ogr'
-require_relative '../gdal'
+require_relative "../ogr"
+require_relative "../gdal"
 
 module OGR
   module Geometry
@@ -213,7 +213,7 @@ module OGR
         FFI::OGR::API.OGR_G_GetEnvelope3D(@c_pointer, envelope)
       when 0 then return nil
       else
-        raise 'Unknown envelope dimension.'
+        raise "Unknown envelope dimension."
       end
 
       return if envelope.null?
@@ -268,7 +268,7 @@ module OGR
     # @param file_path [String] The text file to write to.
     # @param prefix [String] The prefix to put on each line of output.
     def dump_readable(file_path = nil, prefix: nil)
-      file_ptr = file_path ? FFI::CPL::Conv.CPLOpenShared(file_path, 'w', false) : nil
+      file_ptr = file_path ? FFI::CPL::Conv.CPLOpenShared(file_path, "w", false) : nil
       FFI::OGR::API.OGR_G_DumpReadable(@c_pointer, file_ptr, prefix)
       FFI::CPL::Conv.CPLCloseShared(file_ptr) if file_ptr
     end
@@ -363,7 +363,7 @@ module OGR
     def ring?
       FFI::OGR::API.OGR_G_IsRing(@c_pointer)
     rescue GDAL::Error => e
-      return false if e.message.include? 'IllegalArgumentException'
+      return false if e.message.include? "IllegalArgumentException"
 
       raise
     end
@@ -467,7 +467,7 @@ module OGR
 
       return if coord_trans_ptr.nil? || coord_trans_ptr.null?
 
-      OGR::ErrorHandling.handle_ogr_err('Unable to transform geometry') do
+      OGR::ErrorHandling.handle_ogr_err("Unable to transform geometry") do
         FFI::OGR::API.OGR_G_Transform(@c_pointer, coord_trans_ptr)
       end
     end
@@ -489,7 +489,7 @@ module OGR
       new_spatial_ref_ptr = GDAL._pointer(OGR::SpatialReference, new_spatial_ref, autorelease: false)
       return if new_spatial_ref_ptr.null?
 
-      OGR::ErrorHandling.handle_ogr_err('Unable to transform geometry') do
+      OGR::ErrorHandling.handle_ogr_err("Unable to transform geometry") do
         FFI::OGR::API.OGR_G_TransformTo(@c_pointer, new_spatial_ref_ptr)
       end
     end
@@ -552,7 +552,7 @@ module OGR
     # @param wkb_data [String] Binary WKB data.
     # @raise [OGR::Failure]
     def import_from_wkb(wkb_data)
-      OGR::ErrorHandling.handle_ogr_err('Unable to import geometry from WKB') do
+      OGR::ErrorHandling.handle_ogr_err("Unable to import geometry from WKB") do
         FFI::OGR::API.OGR_G_ImportFromWkb(@c_pointer, wkb_data, wkb_data.length)
       end
     end
@@ -592,7 +592,7 @@ module OGR
     # @raise [OGR::Failure]
     def to_wkt
       GDAL._cpl_read_and_free_string do |output_ptr|
-        OGR::ErrorHandling.handle_ogr_err('Unable to export to WKT') do
+        OGR::ErrorHandling.handle_ogr_err("Unable to export to WKT") do
           FFI::OGR::API.OGR_G_ExportToWkt(@c_pointer, output_ptr)
         end
       end
@@ -602,7 +602,7 @@ module OGR
     # @raise [OGR::Failure]
     def to_iso_wkt
       GDAL._cpl_read_and_free_string do |output_ptr|
-        OGR::ErrorHandling.handle_ogr_err('Unable to export to WKT') do
+        OGR::ErrorHandling.handle_ogr_err("Unable to export to WKT") do
           FFI::OGR::API.OGR_G_ExportToIsoWkt(@c_pointer, output_ptr)
         end
       end
@@ -683,7 +683,7 @@ module OGR
 
       linear_ring.spatial_reference = line_string.spatial_reference.clone if line_string.spatial_reference
 
-      linear_ring.import_from_wkt(line_string.to_wkt.tr('LINESTRING', 'LINEARRING'))
+      linear_ring.import_from_wkt(line_string.to_wkt.tr("LINESTRING", "LINEARRING"))
       linear_ring.close_rings! if close_rings
 
       linear_ring

data/lib/ogr/geometry_types/container.rb

@@ -71,7 +71,7 @@ module OGR
                                                                   tolerance,
                                                                   ogrerr_ptr)
 
-        OGR::ErrorHandling.handle_ogr_err('Unable to create polygon from edges') do
+        OGR::ErrorHandling.handle_ogr_err("Unable to create polygon from edges") do
           FFI::OGR::Core::Err[ogrerr_ptr.read_int]
         end
 

data/lib/ogr/geometry_types/curve.rb

@@ -75,7 +75,7 @@ module OGR
                                                    x_buffer, x_stride, y_buffer,
                                                    y_stride, z_buffer, z_stride)
 
-        log 'Got different number of points than point_count in #point_values' unless num_points == point_count
+        log "Got different number of points than point_count in #point_values" unless num_points == point_count
 
         x_array = x_buffer.read_array_of_double(buffer_size)
         y_array = y_buffer.read_array_of_double(buffer_size)

data/lib/ogr/internal_helpers.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'time'
+require "time"
 
 module OGR
   module InternalHelpers
@@ -15,8 +15,8 @@ module OGR
       # @param flag [String] 'w' for writing, 'r' for reading.
       def _boolean_access_flag(flag)
         case flag
-        when 'w' then true
-        when 'r' then false
+        when "w" then true
+        when "r" then false
         else raise "Invalid access_flag '#{flag}'.  Use 'r' or 'w'."
         end
       end
@@ -33,7 +33,7 @@ module OGR
         case time_zone
         when 0 then nil
         when 1 then (Time.now.getlocal.utc_offset / 3600).to_s
-        when 100 then '+0'
+        when 100 then "+0"
         else raise "Unable to process time zone: #{time_zone}"
         end
       end

data/lib/ogr/layer.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
-require_relative '../ogr'
-require_relative '../gdal'
-require_relative 'layer_mixins/ogr_feature_methods'
-require_relative 'layer_mixins/ogr_field_methods'
-require_relative 'layer_mixins/ogr_layer_method_methods'
-require_relative 'layer_mixins/ogr_query_filter_methods'
-require_relative 'layer_mixins/ogr_sql_methods'
+require_relative "../ogr"
+require_relative "../gdal"
+require_relative "layer_mixins/ogr_feature_methods"
+require_relative "layer_mixins/ogr_field_methods"
+require_relative "layer_mixins/ogr_layer_method_methods"
+require_relative "layer_mixins/ogr_query_filter_methods"
+require_relative "layer_mixins/ogr_sql_methods"
 
 module OGR
   class Layer
@@ -42,7 +42,7 @@ module OGR
     # TODO: This seems to occasionally lead to: 28352 illegal hardware
     #   instruction, and sometimes full crashes.
     def sync_to_disk
-      OGR::ErrorHandling.handle_ogr_err('Unable to sync layer to disk') do
+      OGR::ErrorHandling.handle_ogr_err("Unable to sync layer to disk") do
         FFI::OGR::API.OGR_L_SyncToDisk(@c_pointer)
       end
     end