On Fri, 28 Jan 2022 21:32:16 GMT, Phil Race <p...@openjdk.org> wrote:
>> Updated JTable's tableChanged() method docs to explicitly mention the >> coordinate system of the rows and columns. > >> I wasn't able to locate the code that does model-to-view coordinate mapping >> for rows from here. > > That's worrying. So maybe it doesn't exist ? > > When I look at TableModelEvent docs it also says at the VERY BEGINNING of the > class docs .. > /** > * TableModelEvent is used to notify listeners that a table model > * has changed. The model event describes changes to a TableModel > * and all references to rows and columns are in the co-ordinate > * system of the model. > > So at the very least then these docs would be wrong too !?!? > > I think look at the code that does "new TableModelEvent(...") and it is in > methods caller fireTableCellUpdated() which is defined on the TableModel. > > So it goes back to what I said to you earlier (somewhere) - don't just trust > what is written in a bug report - verify it. > > So I think you should do some more looking at code and actual testing to > verify. > Using RowSorter and/or dragging rows around ... @prrace Thank you. Will look into the code and verify with actual testing as well. ------------- PR: https://git.openjdk.java.net/jdk/pull/7253
