Merge pull request #1914 from asottile/fix_charset_unsigned_char

Handle signed char when considering @charset
sass · Feb 9, 2016 · 3054e97 · 3054e97
2 parents 601be96 + e9f4310
commit 3054e97
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/output.cpp b/src/output.cpp
@@ -67,7 +67,8 @@ namespace Sass {
     // search for unicode char
     for(const char& chr : wbuf.buffer) {
       // skip all ascii chars
-      if (chr >= 0) continue;
+      // static cast to unsigned to handle `char` being signed / unsigned
+      if (static_cast<unsigned>(chr) < 128) continue;
       // declare the charset
       if (output_style() != COMPRESSED)
         charset = "@charset \"UTF-8\";"