Re: [cp-patches] Patch for AbstractTableModel/DefaultTableModel - OK?

[Roman Kennke]

David Gilbert wrote:

> This patch fixes bug report 13422 and all (six) failing Mauve checks
> (see before.txt and after.txt) for AbstractTableModel and DefaultTableModel:
>
> 2005-06-23  Dave Gilbert  <address@hidden>
>
>         * javax/swing/table/AbstractTableModel.java:
>         (getColumnName): eliminated temp index to fix very minor bug;
>         (findColumn): throw NullPointerException for null argument;
>         (fireTableDataChanged): set correct row indices.
>         * javax/swing/table/DefaultTableModel.java:
>        (setDataVector): replace null argument with empty vector then call
>         setColumnIdentifiers() to set up column names;
>         (getColumnName): for a column index beyond the number of columns in
>         the model, pass control to the superclass.
>
> OK to commit?
>  

Looks ok. Please synchronize with my fixlet in DefaultTableModel and commit.

/Roman