RFR: 8283059: Uninitialized warning in check_code.c with GCC 11.2
Mikael Vidstedt
mikael at openjdk.java.net
Thu Mar 17 20:23:27 UTC 2022
On Thu, 17 Mar 2022 08:44:57 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> src/java.base/share/native/libverify/check_code.c line 472:
>>
>>> 470:
>>> 471: static void check_and_push(context_type *context, void *ptr, int kind);
>>> 472: static void check_and_push_const(context_type *context, const void *ptr, int kind);
>>
>> This really needs a comment explaining why two functions are needed.
>
> I think it would be clearer to drop the kind argument and have two functions
> check_and_push_vm_string(context_type*, const char*)
> check_and_push_malloc_block(context_type*, void*)
> with a shared helper that is given the appropriate kind.
I had that at some point and then dropped it. I don't know why and it does seem like the better option so I'll try it again.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7794
More information about the core-libs-dev
mailing list