RFR: 8313348: Fix typo in JFormattedTextField: 'it self'

Alexey Ivanov aivanov at openjdk.org
Wed Aug 2 14:19:52 UTC 2023


On Tue, 1 Aug 2023 21:14:58 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

> we have similar typo in the BasicTreeUI.java as well.

Yes, it's there:

https://github.com/openjdk/jdk/blob/b093880acd89d8d0bccd4b8b260b721f4dcfc161/src/java.desktop/share/classes/javax/swing/plaf/basic/BasicTreeUI.java#L4084-L4088

Yet it's not visible *externally*, it's visible only to those who read the code of the `BasicTreeUI.Handler.handleSelection` method whereas the typo I'm addressing here is in the published API documentation for [`JFormattedTextField`](https://docs.oracle.com/en/java/javase/17/docs/api/java.desktop/javax/swing/JFormattedTextField.html).

It wasn't my goal to amend [**all** the instances](https://github.com/search?q=repo%3Aopenjdk%2Fjdk%20%22it%20self%22&type=code)¹ where *‘it self’* is present in the code base — it takes much more time and effort².

It was supposed to be a quick and trivial bug report and fix which addresses *this particular typo* in the single file, `JFormattedTextField.java`. A single file with ~~a single change~~ with four changed lines should've been reviewed quickly and smoothly.

For what it's worth, we have spent much more time discussing it…

¹ There are quite a few; yet they're mostly in hotspot code comments. Eventually, all could be amended.  
² I've been on this journey with [*‘a the’*](https://bugs.openjdk.org/browse/JDK-8284189) in various combinations, it took about two months since its inception to its completion.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15087#discussion_r1281973937



More information about the client-libs-dev mailing list