On Wed, 20 Sep 2023 14:42:40 GMT, Jorn Vernee <jver...@openjdk.org> wrote:

>> src/java.base/windows/native/libjava/java_props_md.c line 214:
>> 
>>> 212:         HRESULT hr;
>>> 213:         WCHAR *tmpPath = NULL;
>>> 214:         hr = SHGetKnownFolderPath(&FOLDERID_Profile, 
>>> KF_FLAG_DONT_VERIFY, NULL, &tmpPath);
>> 
>> I'd say inline the variable declaration here as well
>> Suggestion:
>> 
>>         HRESULT hr = SHGetKnownFolderPath(&FOLDERID_Profile, 
>> KF_FLAG_DONT_VERIFY, NULL, &tmpPath);
>
> (And you will have to remove the declaration from the line above)

Done

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/15789#discussion_r1332620350

Reply via email to