On Sat, 26 Sep 2026 17:39:07 GMT, John Hendrikx <[email protected]> wrote:

>> modules/javafx.base/src/main/java/javafx/beans/Observable.java line 59:
>> 
>>> 57:  *         is not renotified if it has already been notified.
>>> 58:  *     <li>Nested notifications are depth-first, and only renotify the 
>>> listeners
>>> 59:  *         that have already been notified.
>> 
>> I would remove this guarantee. Not because it's wrong, but because I feel it 
>> is hard to understand without a thorough explanation and I'd be hard-pressed 
>> to even come up with a legitimate scenario in which application code would 
>> depend on such minutiae.
>
> Sure, I was dubious about adding it in the first place, so perhaps this was a 
> bit too much detail. Removed.

Either way is fine IMO.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1081#discussion_r4156228354

Reply via email to