On Wed, 18 Mar 2026 09:30:50 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

>> Support for the new DateTimeFormatter in JFormattedTextField is added
>
> Prasanta Sadhukhan has updated the pull request incrementally with two 
> additional commits since the last revision:
> 
>  - Fix
>  - Fix

src/java.desktop/share/classes/javax/swing/JFormattedTextField.java line 900:

> 898:             };
> 899:             return new DefaultFormatterFactory(abstractFormatter);
> 900:         }

I noticed each other instance type has its own formatter class (ex: Date & 
DateFormatter). Should `stringToValue` and `valueToString` be elsewhere? I 
suppose it's functionally fine here, but it doesn't seem to follow the rest of 
the types.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30292#discussion_r2962970779

Reply via email to