On Mon, 17 Jan 2022 05:26:47 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> src/java.desktop/share/classes/javax/swing/undo/UndoableEditSupport.java 
>> line 150:
>> 
>>> 148: 
>>> 149:     /**
>>> 150:      * Start UndoableEdit.
>> 
>> What does it mean? Is it clear enough for the class which "managing 
>> <code>UndoableEdit</code> listeners"?
>
> It seems it other places like CompoundEdit class, "assemble little 
> UndoableEdits " statement is used so it seems ok to use UndoableEdit here. At 
> least it's better than (to me)
> 
> "DEADLOCK WARNING: Calling this method may call
>      * <code>undoableEditHappened</code> in all listeners."
> 
>    
>  used in endUpdate...

Can we update the docs for those methods with a more meaningful description? In 
a separate issue.

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

PR: https://git.openjdk.java.net/jdk/pull/7004

Reply via email to