Fixes an edge case when buffer is empty

mgreter · mgreter · commit 6d4af0e943d2 · 2014-10-15T23:28:34.000+02:00
sass@e5a8ea7
diff --git a/output_compressed.cpp b/output_compressed.cpp
@@ -61,7 +61,7 @@ namespace Sass {
         }
       }
       size_t l = buffer.length();
-      buffer.erase(l-1);
+      if (l > 0 && buffer.at(l - 1) == ';') buffer.erase(l - 1);
       append_singleline_part_to_buffer("}");
     }
 
@@ -90,7 +90,7 @@ namespace Sass {
       }
       return;
     }
-    
+
     ctx->source_map.add_mapping(m);
     append_singleline_part_to_buffer("@media ");
     q->perform(this);
@@ -108,9 +108,9 @@ namespace Sass {
           stm->perform(this);
         }
       }
-      
+
       append_singleline_part_to_buffer("}");
-      
+
       for (size_t i = 0, L = b->length(); i < L; ++i) {
         Statement* stm = (*b)[i];
         if (stm->is_hoistable()) {
@@ -328,7 +328,7 @@ namespace Sass {
     {
       tail->perform(this);
       return;
-    } 
+    }
     if (head && !head->is_empty_reference()) head->perform(this);
     switch (comb) {
       case Complex_Selector::ANCESTOR_OF:

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ namespace Sass {`
`61`	`61`	`}`
`62`	`62`	`}`
`63`	`63`	`size_t l = buffer.length();`
`64`		`- buffer.erase(l-1);`
	`64`	`+ if (l > 0 && buffer.at(l - 1) == ';') buffer.erase(l - 1);`
`65`	`65`	`append_singleline_part_to_buffer("}");`
`66`	`66`	`}`
`67`	`67`
`@@ -90,7 +90,7 @@ namespace Sass {`
`90`	`90`	`}`
`91`	`91`	`return;`
`92`	`92`	`}`
`93`		`-`
	`93`	`+`
`94`	`94`	`ctx->source_map.add_mapping(m);`
`95`	`95`	`append_singleline_part_to_buffer("@media ");`
`96`	`96`	`q->perform(this);`
`@@ -108,9 +108,9 @@ namespace Sass {`
`108`	`108`	`stm->perform(this);`
`109`	`109`	`}`
`110`	`110`	`}`
`111`		`-`
	`111`	`+`
`112`	`112`	`append_singleline_part_to_buffer("}");`
`113`		`-`
	`113`	`+`
`114`	`114`	`for (size_t i = 0, L = b->length(); i < L; ++i) {`
`115`	`115`	`Statement* stm = (*b)[i];`
`116`	`116`	`if (stm->is_hoistable()) {`
`@@ -328,7 +328,7 @@ namespace Sass {`
`328`	`328`	`{`
`329`	`329`	`tail->perform(this);`
`330`	`330`	`return;`
`331`		`- }`
	`331`	`+ }`
`332`	`332`	`if (head && !head->is_empty_reference()) head->perform(this);`
`333`	`333`	`switch (comb) {`
`334`	`334`	`case Complex_Selector::ANCESTOR_OF:`