sassc 1.8.2 → 1.8.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8d364a5c0cc8d7b4890657139a0970c0bf39385e
-  data.tar.gz: 095ebc9195614341559f74e687e59ee951c75dcf
+  metadata.gz: 4711ad9a2cb9091ae22ff8fb0c373f767212dbec
+  data.tar.gz: 5d51cbb258d5855a729028c1db0ce35871e991cb
 SHA512:
-  metadata.gz: 9e97b7a55b207c857d6f2017e7877e6f7df3385e6c366ccc5412ab2fdddf73578ffd6be0279427b6b51df81d886bbd368000a267c8039c9411e7c3c9c7ee4759
-  data.tar.gz: 40a4b7bcca1a22180400841ce114f5b67c72e8e88e30315dacd265a9c3470994df40695f8d65a23c58074d141c328e7cdaa46f792ae4c49b8b9da392d877b5be
+  metadata.gz: ea92abeccd8eda5bd3c7c7dfa7e64bd8bab8182891ff6bb9fcb11f995ab2b6e45c6bb5a0a2260c5eb55131e4ef409bf9754aae6ef50524e2c3f0ea3900bd18a0
+  data.tar.gz: 4d3defd176979708077f1f80fb549643e471092097f201f86e0cf702ec23e90407173bafde4abee9d697ee9ba5f42cf33cf17e1b2f9dc0a9a3138d4395540684

data/.gitignore

@@ -1,5 +1,6 @@
 /.bundle/
 /.yardoc
+/.rvmrc
 /Gemfile.lock
 /_yardoc/
 /coverage/

data/README.md

@@ -2,7 +2,7 @@
 
 Use `libsass` with Ruby!
 
-This gem combines the speed of `libsass`, the [Sass C implementation](https://github.com/sass/libsass), with the easy of use of the original [Ruby Sass](https://github.com/sass/sass) library.
+This gem combines the speed of `libsass`, the [Sass C implementation](https://github.com/sass/libsass), with the ease of use of the original [Ruby Sass](https://github.com/sass/sass) library.
 
 ### libsass Version
 
@@ -25,6 +25,8 @@ and [awesome contributors](https://github.com/bolandrm/sassc-ruby/graphs/contrib
 
 ## Changelog
 
+- **1.8.3**
+  - [Passing empty string into engine does not raise error](https://github.com/sass/sassc-ruby/pull/31)
 - **1.8.2**
   - Update to Libsass 3.3.2
 - **1.8.1**

data/lib/sassc/engine.rb

@@ -17,6 +17,8 @@ module SassC
     end
 
     def render
+      return @template.dup if @template.empty?
+
       data_context = Native.make_data_context(@template)
       context = Native.data_context_get_context(data_context)
       native_options = Native.context_get_options(context)

data/lib/sassc/import_handler.rb

@@ -24,7 +24,7 @@ module SassC
     def import_function
       @import_function ||= FFI::Function.new(:pointer, [:string, :pointer, :pointer]) do |path, importer_entry, compiler|
         last_import = Native::compiler_get_last_import(compiler)
-        parent_path = Native::import_get_imp_path(last_import)
+        parent_path = Native::import_get_abs_path(last_import)
 
         imports = [*@importer.imports(path, parent_path)]
         imports_to_native(imports)

data/lib/sassc/native/native_functions_api.rb

@@ -24,6 +24,24 @@ module SassC
     # ADDAPI union Sass_Value* ADDCALL sass_make_color   (double r, double g, double b, double a);
     attach_function :sass_make_color, [:double, :double, :double, :double], :sass_value_ptr
 
+    # ADDAPI union Sass_Value* ADDCALL sass_make_map     (size_t len);
+    attach_function :sass_make_map, [:size_t], :sass_value_ptr
+
+    # ADDAPI void ADDCALL sass_map_set_key (union Sass_Value* v, size_t i, union Sass_Value*);
+    attach_function :sass_map_set_key, [:sass_value_ptr, :size_t, :sass_value_ptr], :void
+
+    # ADDAPI union Sass_Value* ADDCALL sass_map_get_key (const union Sass_Value* v, size_t i);
+    attach_function :sass_map_get_key, [:sass_value_ptr, :size_t], :sass_value_ptr
+
+    # ADDAPI void ADDCALL sass_map_set_value (union Sass_Value* v, size_t i, union Sass_Value*);
+    attach_function :sass_map_set_value, [:sass_value_ptr, :size_t, :sass_value_ptr], :void
+
+    # ADDAPI union Sass_Value* ADDCALL sass_map_get_value (const union Sass_Value* v, size_t i);
+    attach_function :sass_map_get_value, [:sass_value_ptr, :size_t], :sass_value_ptr
+
+    # ADDAPI size_t ADDCALL sass_map_get_length (const union Sass_Value* v);
+    attach_function :sass_map_get_length, [:sass_value_ptr], :size_t
+
     # ADDAPI union Sass_Value* ADDCALL sass_make_error   (const char* msg);
     attach_function :sass_make_error, [:string], :sass_value_ptr
 
@@ -37,6 +55,12 @@ module SassC
     # ADDAPI bool ADDCALL sass_string_is_quoted(const union Sass_Value* v);
     attach_function :sass_string_is_quoted, [:sass_value_ptr], :bool
 
+    # ADDAPI const char* ADDCALL sass_number_get_value (const union Sass_Value* v);
+    attach_function :sass_number_get_value, [:sass_value_ptr], :double
+
+    # ADDAPI const char* ADDCALL sass_number_get_unit (const union Sass_Value* v);
+    attach_function :sass_number_get_unit, [:sass_value_ptr], :string
+
     def self.string_get_type(native_value)
       string_is_quoted(native_value) ? :string : :identifier
     end
@@ -103,6 +127,7 @@ module SassC
     # ADDAPI const char* ADDCALL sass_import_get_imp_path (struct Sass_Import*);
     attach_function :sass_import_get_imp_path, [:sass_import_ptr], :string
     # ADDAPI const char* ADDCALL sass_import_get_abs_path (struct Sass_Import*);
+    attach_function :sass_import_get_abs_path, [:sass_import_ptr], :string
     # ADDAPI const char* ADDCALL sass_import_get_source (struct Sass_Import*);
     attach_function :sass_import_get_source, [:sass_import_ptr], :string
     # ADDAPI const char* ADDCALL sass_import_get_srcmap (struct Sass_Import*);

data/lib/sassc/script/value_conversion.rb

@@ -10,6 +10,12 @@ module SassC
           type = Native.string_get_type(native_value)
           argument = Script::String.new(value, type)
 
+          argument
+        when :sass_number
+          value = Native.number_get_value(native_value)
+          unit = Native.number_get_unit(native_value)
+          argument = Sass::Script::Value::Number.new(value, unit)
+
           argument
         when :sass_color
           red, green, blue, alpha = Native.color_get_r(native_value), Native.color_get_g(native_value), Native.color_get_b(native_value), Native.color_get_a(native_value)
@@ -17,6 +23,18 @@ module SassC
           argument = Script::Color.new([red, green, blue, alpha])
           argument.options = options
 
+          argument
+        when :sass_map
+          values = {}
+          length = Native::map_get_length native_value
+
+          (0..length-1).each do |index|
+            key = Native::map_get_key(native_value, index)
+            value = Native::map_get_value(native_value, index)
+            values[from_native(key, options)] = from_native(value, options)
+          end
+
+          argument = Sass::Script::Value::Map.new values
           argument
         else
           raise UnsupportedValue.new("Sass argument of type #{value_tag} unsupported")
@@ -29,6 +47,10 @@ module SassC
           String.new(value).to_native
         when "Color"
           Color.new(value).to_native
+        when "Number"
+          Number.new(value).to_native
+        when "Map"
+          Map.new(value).to_native
         else
           raise UnsupportedValue.new("Sass return type #{value_name} unsupported")
         end
@@ -39,4 +61,6 @@ end
 
 require_relative "value_conversion/base"
 require_relative "value_conversion/string"
+require_relative "value_conversion/number"
 require_relative "value_conversion/color"
+require_relative "value_conversion/map"

data/lib/sassc/script/value_conversion/map.rb

@@ -0,0 +1,19 @@
+module SassC
+  module Script
+    module ValueConversion
+      class Map < Base
+        def to_native
+          hash = @value.to_h
+          native_map = Native::make_map( hash.size )
+          hash.each_with_index do |(key, value), index|
+            key   = ValueConversion.to_native key
+            value = ValueConversion.to_native value
+            Native::map_set_key(   native_map, index, key )
+            Native::map_set_value( native_map, index, value )
+          end
+          return native_map
+        end
+      end
+    end
+  end
+end

data/lib/sassc/script/value_conversion/number.rb

@@ -0,0 +1,11 @@
+module SassC
+  module Script
+    module ValueConversion
+      class Number < Base
+        def to_native
+          Native::make_number(@value.value, @value.numerator_units.first)
+        end
+      end
+    end
+  end
+end

data/lib/sassc/version.rb

@@ -1,3 +1,3 @@
 module SassC
-  VERSION = "1.8.2"
+  VERSION = "1.8.3"
 end

data/test/engine_test.rb

@@ -189,5 +189,10 @@ CSS
       assert_match /sourceMappingURL/, output
       assert_match /.foo/, output
     end
+
+    def test_empty_template
+      output = Engine.new('').render
+      assert_equal '', output
+    end
   end
 end

data/test/functions_test.rb

@@ -64,6 +64,22 @@ module SassC
       CSS
     end
 
+    def test_function_that_returns_a_number
+      assert_sass <<-SCSS, <<-CSS
+        div { width: returns-a-number(); }
+      SCSS
+        div { width: -312rem; }
+      CSS
+    end
+
+    def test_function_that_takes_a_number
+      assert_sass <<-SCSS, <<-CSS
+        div { display: inspect-number(42.1px); }
+      SCSS
+        div { display: 42.1px; }
+      CSS
+    end
+
     def test_function_with_optional_arguments
       assert_sass <<-SCSS, <<-EXPECTED_CSS
         div {
@@ -87,14 +103,14 @@ module SassC
     end
 
     def test_function_with_unsupported_tag
-      engine = Engine.new("div {url: function_with_unsupported_tag(1);}")
+      engine = Engine.new("div {url: function_with_unsupported_tag(());}")
 
       exception = assert_raises(SassC::SyntaxError) do
         engine.render
       end
 
-      assert_match /Sass argument of type sass_number unsupported/, exception.message
-      assert_equal "[SassC::FunctionsHandler] Sass argument of type sass_number unsupported", stderr_output
+      assert_match /Sass argument of type sass_list unsupported/, exception.message
+      assert_equal "[SassC::FunctionsHandler] Sass argument of type sass_list unsupported", stderr_output
     end
 
     def test_function_with_error
@@ -117,6 +133,23 @@ module SassC
       CSS
     end
 
+    def test_function_that_returns_a_sass_map
+      assert_sass <<-SCSS, <<-CSS
+        $my-map: returns-sass-map();
+        div { background: map-get( $my-map, color ); }
+      SCSS
+        div { background: black; }
+      CSS
+    end
+
+    def test_function_that_takes_a_sass_map
+      assert_sass <<-SCSS, <<-CSS
+        div { background-color: map-get( inspect-map(( color: black, number: 1.23px, string: "abc", map: ( x: 'y' ))), color ); }
+      SCSS
+        div { background-color: black; }
+      CSS
+    end
+
     private
 
     def assert_sass(sass, expected_css)
@@ -151,7 +184,7 @@ module SassC
         raise StandardError, "Intentional wrong thing happened somewhere inside the custom function"
       end
 
-      def function_with_unsupported_tag(number)
+      def function_with_unsupported_tag(value)
       end
 
       def nice_color_argument(color)
@@ -162,10 +195,49 @@ module SassC
         return Script::Color.new(red: 0, green: 0, blue: 0)
       end
 
+      def returns_a_number
+        return Sass::Script::Value::Number.new(-312,'rem')
+      end
+
+      def inspect_number ( argument )
+        raise StandardError.new "passed value is not a Sass::Script::Value::Number" unless argument.is_a? Sass::Script::Value::Number
+        return argument
+      end
+
+      def inspect_map ( argument )
+        argument.to_h.each_pair do |key, value|
+          raise StandardError.new "key #{key.inspect} is not a string" unless key.is_a? Sass::Script::Value::String
+
+          valueClass = case key.value
+                         when 'string'
+                           Sass::Script::Value::String
+                         when 'number'
+                           Sass::Script::Value::Number
+                         when 'color'
+                           Sass::Script::Value::Color
+                         when 'map'
+                           Sass::Script::Value::Map
+                       end
+
+          raise StandardError.new "unknown key #{key.inspect}" unless valueClass
+          raise StandardError.new "value for #{key.inspect} is not a #{valueClass}" unless value.is_a? valueClass
+        end
+        return argument
+      end
+
       def returns_sass_value
         return Sass::Script::Value::Color.new(red: 0, green: 0, blue: 0)
       end
 
+      def returns_sass_map
+        key = Script::String.new("color", "string")
+        value = Sass::Script::Value::Color.new(red: 0, green: 0, blue: 0)
+        values = {}
+        values[key] = value
+        map = Sass::Script::Value::Map.new values
+        return map
+      end
+
       module Compass
         def stylesheet_path(path)
           Script::String.new("/css/#{path.value}", :identifier)

data/test/native_test.rb

@@ -96,6 +96,10 @@ module SassC
         assert_equal :sass_string, Native.value_get_tag(string)
         assert_equal "hello", Native.string_get_value(string)
 
+        number = Native.make_number(123.4, "rem")
+        assert_equal 123.4, Native.number_get_value(number)
+        assert_equal "rem", Native.number_get_unit(number)
+
         Native.compile_data_context(data_context)
 
         css = Native.context_get_output_string(context)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sassc
 version: !ruby/object:Gem::Version
-  version: 1.8.2
+  version: 1.8.3
 platform: ruby
 authors:
 - Ryan Boland
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-14 00:00:00.000000000 Z
+date: 2016-01-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -347,6 +347,8 @@ files:
 - lib/sassc/script/value_conversion.rb
 - lib/sassc/script/value_conversion/base.rb
 - lib/sassc/script/value_conversion/color.rb
+- lib/sassc/script/value_conversion/map.rb
+- lib/sassc/script/value_conversion/number.rb
 - lib/sassc/script/value_conversion/string.rb
 - lib/sassc/version.rb
 - lib/tasks/libsass.rb