search

qwertie / ecsharp

Home of LoycCore, the LES language of Loyc trees, the Enhanced C# parser, the LeMP macro preprocessor, and the LLLPG parser generator.
http://ecsharp.net
Other
177 stars 25 forks source link

Add parentheses to FixedPoint.{cs,tt} comparisons #49

Closed jonathanvdc closed 7 years ago

jonathanvdc commented 7 years ago

I ran into an issue that prevented me from compiling Loyc. This PR fixes that issue.

Specifically, mcs thought value.N < Int32.MinValue >> 8 was a generic instance, which resulted in error messages from the compiler.

I added some parentheses to resolve the issue. Also, they should make the code easier to read IMO.

qwertie commented 7 years ago

That's really interesting! The EC# parser does the same thing as mcs.