Merge pull request #3155 from mgreter/bugfix/issue-3148

mgreter · web-flow · commit f158280c2143 · 2021-05-21T02:32:56.000+02:00
Fix obvious identical subexpressions in `op_color_number`
diff --git a/src/operators.cpp b/src/operators.cpp
@@ -247,7 +247,7 @@ namespace Sass {
     {
       double rval = rhs.value();
 
-      if ((op == Sass_OP::DIV || op == Sass_OP::DIV) && rval == 0) {
+      if ((op == Sass_OP::DIV || op == Sass_OP::MOD) && rval == 0) {
         // comparison of Fixnum with Float failed?
         throw Exception::ZeroDivisionError(lhs, rhs);
       }

Original file line number	Diff line number	Diff line change
`@@ -247,7 +247,7 @@ namespace Sass {`
`247`	`247`	`{`
`248`	`248`	`double rval = rhs.value();`
`249`	`249`
`250`		`- if ((op == Sass_OP::DIV \|\| op == Sass_OP::DIV) && rval == 0) {`
	`250`	`+ if ((op == Sass_OP::DIV \|\| op == Sass_OP::MOD) && rval == 0) {`
`251`	`251`	`// comparison of Fixnum with Float failed?`
`252`	`252`	`throw Exception::ZeroDivisionError(lhs, rhs);`
`253`	`253`	`}`