Roman Kennke wrote:
This removes a not used constant in BasicTreeUI. This was creating a
synthetic MouseEvent, which wouldn't work because the MouseEvent tries
to get the screen location of the source component, which fails because
the artificial component is not showing.
The coordinates of that mouse event were never used. Apart constructor,
the only other operation where it was involved was the direct comparison
(==) - to check if a mouse event under consideration is THE EVENT which
must be handled in a specific way. Anyway, this is seems not important
as the code is now rewritten differently and does not need this
singleton instance.
Alas, the constant isn't used
anyway, so I remove it. This should fix the Mauve regression and should
also help with FindBugs, AFAICS.
Which exactly Mauve regression is fixed by removing that unused piece of
code?
Audrius