Looks fine Stuart. -Chris.
> On 30 Mar 2017, at 02:20, Stuart Marks <stuart.ma...@oracle.com> wrote: > > Hi all, > > Please review this small, non-normative textual change to j.u.Observable. > This class, and its companion Observer interface, were deprecated earlier in > JDK 9. Doug Lea suggested adding a link to j.u.c.Flow in case people confused > these APIs with reactive streams. (AFAICS reactive streams doesn't use the > terms observer/observable, but RxJava does.) > > Thanks, > > s'marks > > # HG changeset patch > # User smarks > # Date 1490835538 25200 > # Wed Mar 29 17:58:58 2017 -0700 > # Node ID 7b9ea7fa93c00698325d38cfc108cd6e79bffe10 > # Parent 6b43c4698752779793d58813f46d3687c17dde75 > 8155052: add notes and links to j.u.Observer/Observable deprecation comments > Reviewed-by: XXX > > diff -r 6b43c4698752 -r 7b9ea7fa93c0 > src/java.base/share/classes/java/util/Observable.java > --- a/src/java.base/share/classes/java/util/Observable.java Tue Mar 28 > 18:12:38 2017 -0700 > +++ b/src/java.base/share/classes/java/util/Observable.java Wed Mar 29 > 17:58:58 2017 -0700 > @@ -69,6 +69,8 @@ > * {@link java.beans} package. For reliable and ordered > * messaging among threads, consider using one of the concurrent data > * structures in the {@link java.util.concurrent} package. > + * For reactive streams style programming, see the > + * {@link java.util.concurrent.Flow} API. > */ > @Deprecated(since="9") > public class Observable {