Allow unicode chars in unquoted urls

Fixes sass#2120
mgreter · Jul 19, 2016 · db1d56b · db1d56b
1 parent 2fde7e6
commit db1d56b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/src/lexer.cpp b/src/lexer.cpp
@@ -88,11 +88,12 @@ namespace Sass {
     }
 
     // check if char is within a reduced ascii range
-    // valid in a uri (copied from Ruby Sass)
+    // valid in a uri (and also unicode octets)
     bool is_uri_character(const char& chr)
     {
-      return (unsigned(chr) > 41 && unsigned(chr) < 127) ||
-             unsigned(chr) == ':' || unsigned(chr) == '/';
+      return unsigned(chr) > 41 ||
+             unsigned(chr) == ':' ||
+             unsigned(chr) == '/';
     }
 
     // check if char is within a reduced ascii range

diff --git a/src/parser.cpp b/src/parser.cpp
@@ -316,10 +316,6 @@ namespace Sass {
         else if (String* the_url = parse_url_function_argument()) {
           *args << SASS_MEMORY_NEW(ctx.mem, Argument, the_url->pstate(), the_url);
         }
-        else if (peek < skip_over_scopes < exactly < '(' >, exactly < ')' > > >(position)) {
-          Expression* the_url = parse_list(); // parse_interpolated_chunk(lexed);
-          *args << SASS_MEMORY_NEW(ctx.mem, Argument, the_url->pstate(), the_url);
-        }
         else {
           error("malformed URL", pstate);
         }