Properly hydrate numbers after operations in variable assignments

[xzyfer]

This PR fixes an issue where a number variable's units would be mangled after a binary operation i.e. *, /.

I've also updated the number validity check and error message to be more inline with Ruby sass.

Fixes #783. Specs added sass/sass-spec#216.

[am11]

👍

@xzyfer, (just curious) what about those commented lines in the test? Are they part of the spec?

[xzyfer]

Those lines cause errors which we cannot assert yet. I've left them in with
the appropriate error message for when we can.
On 2 Jan 2015 06:33, "Adeel Mujahid" notifications@github.com wrote:

[image: 👍]

@xzyfer https://github.com/xzyfer, (just curious) what about those
commented lines in the test? Are they part of the spec?

—
Reply to this email directly or view it on GitHub
#797 (comment).

[am11]

Awesome! Apparently, number theory is (also) becoming a strong suit of libsass. 😎

On a separate note, would be nice if you guys can setup gitter (like node-sass channel there: https://gitter.im/sass/node-sass), so peeps can throw in questions, which you guys can answer later. IRC is another way to go. :)

PS: We have a badge for gitter in README as well: https://github.com/sass/node-sass/blob/master/README.md

[coveralls]

Coverage increased (+0.0%) when pulling 8b399d1 on xzyfer:fix/op-number-units into 9087a77 on sass:master.