On Thu, 20 Jun 2024 08:29:39 GMT, Julian Waters <jwat...@openjdk.org> wrote:
> In [JDK-8302671](https://bugs.openjdk.org/browse/JDK-8302671) I fixed a > memmove decay bug by rewriting a sizeof on an array to an explicit size of > 256, but this is a bit of a band aid fix. It's come to my attention that in > C++, one can pass an array by reference, which causes sizeof to work > correctly on an array and has the added bonus of enforcing an array of that > size on the arguments passed to that method. I've reverted my change from > 8302671 and instead explicitly made kstate an array reference so that sizeof > works on the array as expected, and that the array size can be explicitly set > in the array brackets > > Verification: https://godbolt.org/z/Ezj76eWWY and GitHub Actions This pull request has now been integrated. Changeset: dabdae6b Author: Julian Waters <jwat...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/dabdae6b355b3e7a2795f6b01ba475652d483288 Stats: 3 lines in 1 file changed: 0 ins; 0 del; 3 mod 8334599: Improve code from JDK-8302671 Reviewed-by: aivanov, stuefe, prr ------------- PR: https://git.openjdk.org/jdk/pull/19798
