Ah, my apologies, Michael.

I can see the issue of it being a convenience thing, but given that many programmers have fallen foul of the lack of range checking in case blocks, the typecasting necessary to avoid the problem is cumbersome, adds a performance penalty and, ultimately, implies you should just not use enumerations at all in that instance.

When an issue is raised from many different sources, it usually means something is counterintuitive.  We only know about it because we've done a lot of compiler development.

Gareth aka. Kit


On 05/07/2019 01:17, Michael Van Canneyt wrote:


On Fri, 5 Jul 2019, J. Gareth Moreton wrote:

It's not so much convenience... it's more the fact that the compiler can and will optimise out "if (MyValue>=ord(Low(TMyType))) and (MyValue<=Ord(High(TMyType))) then" because it has every reason to assume that MyValue cannot possibly take on a value that's less than Low(TMyType) or greater than High(TMyType), hence it gets shortcut and assumed to always be True.

No. MyValue is an integer. Hence the ord().

Obviously the check must be done before the typecast, not after...

Let me make that more explicit:

if MyIntValue is TMyEnum then
  MyValue:=TMyEnum(MyIntValue);
or
MyValye:=MyIntValue as TMyEnum;

That's why I said it it is not at odds with the fact that the compiler can
assume MyValue is always valid.

Michael.
_______________________________________________
fpc-devel maillist  -  fpc-devel@lists.freepascal.org
https://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel



---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus

_______________________________________________
fpc-devel maillist  -  fpc-devel@lists.freepascal.org
https://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel

Reply via email to