Identical subexpressions in op_color_number

[ariccio]

I ran msvc code analysis while waiting for a heroku instance to deploy, and it found this code introduced in 95011cd in operators.cpp:

      if ((op == Sass_OP::DIV || op == Sass_OP::DIV) && rval == 0) {
        // comparison of Fixnum with Float failed?
        throw Exception::ZeroDivisionError(lhs, rhs);
      }

...that can't be right?

[ariccio]

Sidenote: there are a few other warnings that I cannot decide if they're valid or not:

Warning C26815 The pointer is dangling because it points at a temporary instance which was destroyed.	libsass ...\libsass-master\libsass-master\src\eval.cpp 1090
Warning C26815 The pointer is dangling because it points at a temporary instance which was destroyed.	libsass ...\libsass-master\libsass-master\src\eval.cpp 1050	

[mgreter]

I guess it should have been op == Sass_OP::DIV || op == Sass_OP::MOD.