Enum-of-byte is unnecessarily cast to int

[TobiasMarklund]

Enum-of-byte is unnecessarily cast to int

[JonathanMagnan]

Hello @TobiasMarklund ,

Thank you for reporting.

We tried the same Fiddle without our library and as expected, there is no CAST.

We will look at why this cast happen in the library

Best Regards,

Jonathan

---

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations • Dapper Plus

Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

[JonathanMagnan]

Hello @TobiasMarklund ,

We looked at it this weekend, however, we didn't succeed to find a solution.

We tried to move some code but unfortunately, the cast was always required at the end when comparing both DbExpression.

So unless you have found something on your side that might worth to investigate, I believe we will close this issue since we have not found an alternative way to make it works without the cast.

[TobiasMarklund]

Seems like it is the C# compiler which adds that cast for unknown reasons.
I have been experimenting with modifying Binary equality expressions in the LambdaToDbExpressionVisitor where I check if one operand is a converted byte enum and if the other operand is also possible to convert to byte to make a new BinaryExpression with byte operands.

I can make a pull request with my attempt.

[TobiasMarklund]

Why did you close this?

[JonathanMagnan]

Oops! Sorry,

The issue was marked as to be "closed" but the status was not updated with your last answer. So we closed it by mistake ;(

Now opened ;)

[JonathanMagnan]

Hello @TobiasMarklund ,

Thank you for your contribution.

The v3.1.1 has been released. It includes your both pull requests.