On Sat, Nov 9, 2019 at 9:08 AM Michael Van Canneyt <mich...@freepascal.org> wrote:
> That is why I decided to keep it: the mode of sysutils is known and will > never change. A user is supposed to take this into account. OK. > This error was confirmed as a compiler bug. It also disappears if you remove > the inline from the methods. > > Since there is no run-time error for the same value, I kept the code as-is. > I consider this a remote corner case. [snip] > If you send me a patch to rework the bit manipulations, I will check & apply > it. If this is considered a compiler bug, then my patch (which would consist of typecating to unsigned) is a workaround tha simply hides the compiler error. So, it should not be applied then, as you already decided, Thank you for explaining. B.t.w. I really enjoyed doing the extensive testing. Apart from the cornercase compiler-time issues, all test are ok for all modes on 32 and 64 bit. So, I'm happy. Bart -- Bart _______________________________________________ fpc-devel maillist - fpc-devel@lists.freepascal.org https://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel
