Issue with Double.CompareTo()

TinyCLR OS
1

Hello,

The method always returns 0, which is obviously not expected.

Issue #565 created.

Regards,
Christophe

2

Well…

No luck either with Decimals :frowning:

private static void TestDecimal()
        {
            Decimal a = 20.1M;
            Decimal b = 15.1M;

            var comp = Decimal.Equals(a, b);
            Debug.WriteLine($"Compare {a} to {b} : {comp}");

            a = 20.0M;
            b = 20.0M;
            comp = Decimal.Equals(a, b);
            Debug.WriteLine($"Compare {a} to {b} : {comp}");

            a = -4.7M;
            b = 20.0M;
            comp = Decimal.Equals(a, b);
            Debug.WriteLine($"Compare {a} to {b} : {comp}");
        }

Equals always returns true.

3

Single.Equals() seems ok, however.

4

double works as well
but I got this in VS with your example

image

5

Thanks. We are looking into it.

6

@philippeb : it looks like an issue with TinyCLR application. The same application in a .Net Windows form does not show this message.
Btw, 20.0M is a valid decimal, so I don’t understand why it displays this message ?!

However, you say that doubles do work ? That’s not what I see. Could you tell more, please ?

7

Decimal on embedded device?

8

I didn’t think decimal is supported but I was curious to investigate further and comment the answer on docs, not here.

9

Yes, I tried first in a .net winform too and it works as expected
Here is the screen with 2 double values instead of decimal

image
image798Ă—339 15.3 KB

note that generally, I only use decimal when I deal with some oracle database as the oracle datatype to Net is sometimes tricky

10

Please confirm that this is full .Net code. Because Double.Equals() is not currently implemented in TinyCLR 2.0 : Issue #502

11

test done in a TinyClr app

12

var eq = Equals(a, b); does work.
var eq2 = a.Equals(b); throws CLR_E_UNSUPPORTED_INSTRUCTION exception.

We are both right :smiley:

1 Like