On Thu, 12 Mar 2026 16:01:31 GMT, Marius Hanl <[email protected]> wrote:

>> Dmitry Markov has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Added Oxford comma
>
> modules/javafx.graphics/src/main/java/com/sun/glass/ui/CommonDialogs.java 
> line 235:
> 
>> 233:                 return folder.getCanonicalPath();
>> 234:             } catch (IOException e) {
>> 235:                 // Do not propagate the exception and let the platfrom 
>> decide
> 
> Minor: The comment can probably be put in the same line

Done

> modules/javafx.graphics/src/main/java/com/sun/glass/ui/CommonDialogs.java 
> line 236:
> 
>> 234:             } catch (IOException e) {
>> 235:                 // Do not propagate the exception and let the platfrom 
>> decide
>> 236:                 // what folder to use as initial dirictory.
> 
> typo: `directory`

Good catch! Corrected

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2101#discussion_r2925802166
PR Review Comment: https://git.openjdk.org/jfx/pull/2101#discussion_r2925801393

Reply via email to