The fix looks fine, but Sergey B. is right, a new bug must be filed to
fix the initial deadlock (7146146).
Thanks,
Artem
On 12/17/2013 6:36 PM, Sergey Bylokhov wrote:
Hi, Sergey.
The fix looks good.
Can you submit additional CR for initial deadlock. It would be good to
look at it.
On 17.12.2013 17:58, sergey malenkov wrote:
Hello,
Could you please review the following fix:
fix:http://cr.openjdk.java.net/~malenkov/8030118.9.0/
bug:https://bugs.openjdk.java.net/browse/JDK-8030118
We must revert the 7146146 fix that contradicts the specification:
"The code path for any DocumentListener implementation and any
UndoListener implementation must be threadsafe, and not access the
component lock if trying to be safe from deadlocks".
Thanks,
SAM
