sass 3.3.0.alpha.181 → 3.3.0.alpha.184

data/REVISION

@@ -1 +1 @@
-dca1498acda9d3549cc10e87045f75516e494284
+9c8ae2980333f2715678c6737448ef738e0bb0cb

data/VERSION

@@ -1 +1 @@
-3.3.0.alpha.181
+3.3.0.alpha.184

data/VERSION_DATE

@@ -1 +1 @@
-17 June 2013 21:59:02 GMT
+11 June 2013 21:31:46 GMT

data/lib/sass/script/functions.rb

@@ -1481,30 +1481,28 @@ module Sass::Script
     # Gets the nth item in a list.
     #
     # Note that unlike some languages, the first item in a Sass list is number 1,
-    # the second number 2, and so forth. You can also use negative numbers to
-    # count from the end of the list. So -1 is the last item, -2 is the
-    # second-to-last item, etc.
+    # the second number 2, and so forth.
     #
     # @example
     #   nth(10px 20px 30px, 1) => 10px
     #   nth((Helvetica, Arial, sans-serif), 3) => sans-serif
-    #   nth((red, green, blue), -2) => green
     # @param list [Value] The list
     # @param n [Sass::Script::Value::Number] The index into the list
     # @return [Sass::Script::Value::Base] The nth item in the list
-    # @raise [ArgumentError] If `n` isn't an integer whose absolute value is between 1 and the list's length.
+    # @raise [ArgumentError] If `n` isn't an integer between 1 and the list's length.
     def nth(list, n)
       assert_type n, :Number
-      if !n.int? || n.to_i == 0
-        raise ArgumentError.new("List index #{n} must be a non-zero integer")
+      if !n.int?
+        raise ArgumentError.new("List index #{n} must be an integer")
+      elsif n.to_i < 1
+        raise ArgumentError.new("List index #{n} must be greater than or equal to 1")
       elsif list.to_a.size == 0
         raise ArgumentError.new("List index is #{n} but list has no items")
-      elsif n.to_i.abs > (size = list.to_a.size)
+      elsif n.to_i > (size = list.to_a.size)
         raise ArgumentError.new("List index is #{n} but list is only #{size} item#{'s' if size != 1} long")
       end
 
-      index = n.to_i > 0 ? n.to_i - 1 : n.to_i
-      list.to_a[index]
+      list.to_a[n.to_i - 1]
     end
     declare :nth, [:list, :n]
 

data/test/sass/functions_test.rb

@@ -1017,14 +1017,12 @@ MSG
   def test_nth
     assert_equal("1", evaluate("nth(1 2 3, 1)"))
     assert_equal("2", evaluate("nth(1 2 3, 2)"))
-    assert_equal("3", evaluate("nth(1 2 3, -1)"))
-    assert_equal("1", evaluate("nth(1 2 3, -3)"))
     assert_equal("3", evaluate("nth((1, 2, 3), 3)"))
     assert_equal("foo", evaluate("nth(foo, 1)"))
     assert_equal("bar baz", evaluate("nth(foo (bar baz) bang, 2)"))
-    assert_error_message("List index 0 must be a non-zero integer for `nth'", "nth(foo, 0)")
-    assert_error_message("List index is -10 but list is only 1 item long for `nth'", "nth(foo, -10)")
-    assert_error_message("List index 1.5 must be a non-zero integer for `nth'", "nth(foo, 1.5)")
+    assert_error_message("List index 0 must be greater than or equal to 1 for `nth'", "nth(foo, 0)")
+    assert_error_message("List index -10 must be greater than or equal to 1 for `nth'", "nth(foo, -10)")
+    assert_error_message("List index 1.5 must be an integer for `nth'", "nth(foo, 1.5)")
     assert_error_message("List index is 5 but list is only 4 items long for `nth'", "nth(1 2 3 4, 5)")
     assert_error_message("List index is 2 but list is only 1 item long for `nth'", "nth(foo, 2)")
     assert_error_message("List index is 1 but list has no items for `nth'", "nth((), 1)")

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification 
 name: sass
 version: !ruby/object:Gem::Version 
-  hash: 592302695
+  hash: 592302717
   prerelease: 6
   segments: 
   - 3
   - 3
   - 0
   - alpha
-  - 181
-  version: 3.3.0.alpha.181
+  - 184
+  version: 3.3.0.alpha.184
 platform: ruby
 authors: 
 - Nathan Weizenbaum
@@ -19,7 +19,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2013-06-17 00:00:00 -04:00
+date: 2013-06-11 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency