On Mon, 27 Jul 2020 20:32:46 GMT, Nir Lisker <nlis...@openjdk.org> wrote:
> Adds clarifications to the documentation in various places. Some notes: > > * Point 6 should probably be deferred until it is verified that the tutorials > are correct enough, seeing as they were > updated to Java 8 only. > * Point 8 has been deferred until all the animation bugs have been resolevd. > * Point 5: I wrote new documentation about the `extractor` for the > `observableArrayList(Callback<E, Observable[]> > extractor)` method. Later I found that `observableList(List<E> list, > Callback<E, Observable[]> extractor)` already > talks about it (I updated it too). I'm not sure which of them we want to > keep, or maybe merge them. > * Point 1: I think that it's necessary to mention the internal implementation > behavior even if it requires a caveat that > this is only the current behavior and may change in the future. What > constitutes a "change" is extremely important and > there is no way for the user to know it. I've tripped on this hard when > using ReactFX which uses object equality > instead, so when the JavaFX observables are wrapped by ReactFX observables, > the behavior changes silently. > I think that in the future we will want to let the user define what a change > is (for example, by creating an > overridable method with the current behavior as the default, or using object > equality and letting the user override > that, although that's more risky). Even a `HashMap` that uses object equality > has the sister implementation > `IndentityHashMap` to deal with this ambiguity. This pull request has now been integrated. Changeset: 208d8282 Author: Nir Lisker <nlis...@openjdk.org> URL: https://git.openjdk.java.net/jfx/commit/208d8282 Stats: 126 lines in 9 files changed: 16 ins; 63 del; 47 mod 8228570: Add various documentation clarifications Reviewed-by: arapte, kcr ------------- PR: https://git.openjdk.java.net/jfx/pull/276