On Fri, 10 Jul 2026 13:50:35 GMT, Matthias Baesken <[email protected]> wrote:

>> The property name itself, `NativeDialogWndProcProp`, could be removed if 
>> it's not used after this PR, which should be the case.
>
> We still use it below at line 202
> ` LPOPENFILENAME lpofn = (LPOPENFILENAME)GetProp(parent, OpenFileNameProp);`
> 
> so I am not sure about this whole  OpenFileNameProp  removal ?

Where did I say we need to remove `OpenFileNameProp`?

I said to remove **`NativeDialogWndProcProp`**. It used to store the original 
`WndProc` before subclassing. And it seems the value of the property has always 
been `NULL` for a while now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31814#discussion_r3559430526

Reply via email to