Merge pull request #38 from TeaSeaLancs/add-boolean

Add boolean script support

Author: bolandrm

lib/sassc/native/native_functions_api.rb

@@ -26,6 +26,9 @@ module Native
 
     # ADDAPI union Sass_Value* ADDCALL sass_make_map     (size_t len);
     attach_function :sass_make_map, [:size_t], :sass_value_ptr
+    
+    # ADDAPI union Sass_Value* ADDCALL sass_make_boolean (boolean val);
+    attach_function :sass_make_boolean, [:bool], :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
@@ -60,6 +63,9 @@ module Native
 
     # ADDAPI const char* ADDCALL sass_number_get_unit (const union Sass_Value* v);
     attach_function :sass_number_get_unit, [:sass_value_ptr], :string
+    
+    # ADDAPI const char* ADDCALL sass_boolean_get_value (const union Sass_Value* v);
+    attach_function :sass_boolean_get_value, [:sass_value_ptr], :bool
 
     def self.string_get_type(native_value)
       string_is_quoted(native_value) ? :string : :identifier

lib/sassc/script.rb

@@ -31,9 +31,13 @@ module Script
 require 'sass/script/value/base'
 require 'sass/script/value/string'
 require 'sass/script/value/color'
+require 'sass/script/value/bool'
 
 SassC::Script::String = Sass::Script::Value::String
 SassC::Script::Value::String = Sass::Script::Value::String
 
 SassC::Script::Color = Sass::Script::Value::Color
 SassC::Script::Value::Color = Sass::Script::Value::Color
+
+SassC::Script::Bool = Sass::Script::Value::Bool
+SassC::Script::Value::Bool = Sass::Script::Value::Bool

lib/sassc/script/value_conversion.rb

@@ -10,6 +10,11 @@ def self.from_native(native_value, options)
           type = Native.string_get_type(native_value)
           argument = Script::String.new(value, type)
 
+          argument
+        when :sass_boolean
+          value = Native.boolean_get_value(native_value)
+          argument = Script::Bool.new(value)
+          
           argument
         when :sass_number
           value = Native.number_get_value(native_value)
@@ -51,6 +56,8 @@ def self.to_native(value)
           Number.new(value).to_native
         when "Map"
           Map.new(value).to_native
+        when "Bool"
+          Bool.new(value).to_native
         else
           raise UnsupportedValue.new("Sass return type #{value_name} unsupported")
         end
@@ -64,3 +71,4 @@ def self.to_native(value)
 require_relative "value_conversion/number"
 require_relative "value_conversion/color"
 require_relative "value_conversion/map"
+require_relative "value_conversion/bool"

lib/sassc/script/value_conversion/bool.rb

@@ -0,0 +1,11 @@
+module SassC
+  module Script
+    module ValueConversion
+      class Bool < Base
+        def to_native
+          Native::make_boolean(@value.value)
+        end
+      end
+    end
+  end
+end

test/functions_test.rb

@@ -80,6 +80,22 @@ def test_function_that_takes_a_number
       CSS
     end
 
+    def test_function_that_returns_a_bool
+      assert_sass <<-SCSS, <<-CSS
+        div { width: returns-a-bool(); }
+      SCSS
+        div { width: true; }
+      CSS
+    end
+
+    def test_function_that_takes_a_bool
+      assert_sass <<-SCSS, <<-CSS
+        div { display: inspect-bool(true)}
+      SCSS
+        div { display: true; }
+      CSS
+    end
+
     def test_function_with_optional_arguments
       assert_sass <<-SCSS, <<-EXPECTED_CSS
         div {
@@ -199,6 +215,15 @@ def returns_a_number
         return Sass::Script::Value::Number.new(-312,'rem')
       end
 
+      def returns_a_bool
+        return Sass::Script::Value::Bool.new(true)
+      end
+
+      def inspect_bool ( argument )
+        raise StandardError.new "passed value is not a Sass::Script::Value::Bool" unless argument.is_a? Sass::Script::Value::Bool
+        return argument
+      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