RFR: 8375011: OldJTable.java - NullPointerException when columnData is null [v3]

[ANUPAM DEV]

On Mon, 19 Jan 2026 07:02:12 GMT, ANUPAM DEV <duke at openjdk.org> wrote:

>> Hi,
>> 
>> This is a fix for NPE in OldJTable.addColumn
>> 
>> The addColumn method in OldJTable.java was unconditionally calling columnData.toArray(), which caused a NullPointerException when columnData was null (e.g., when called from addColumn(Object, int)).
>> 
>> This commit adds a null check to safe-guard against this NPE, passing null to the underlying DefaultTableModel when columnData is missing.
>> 
>> Kindly review.
>> 
>> Regards,
>> Anupam
>
> ANUPAM DEV has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update full name

No, there is no reference of OldJTable in the demo being used anywhere. Moreover, the comment at the beginning says, OldJTable is an unsupported class. If we need to keep that class for future, the above fix should be fine, or we may consider removing the class altogether. Please let me know.

Regards,
Anupam

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

PR Comment: https://git.openjdk.org/jdk/pull/29226#issuecomment-3787921851