Clean up documentation 46/109846/2
authorRobert Varga <robert.varga@pantheon.tech>
Thu, 18 Jan 2024 18:52:54 +0000 (19:52 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Thu, 18 Jan 2024 19:27:16 +0000 (20:27 +0100)
We are about to duplicate a number of methods, clean up their
documentation, so we do not accumulate work.

Change-Id: I119b4009ebef1fe8af4a7cc8057ee86bdf2e1621
JIRA: MDSAL-819
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
binding/mdsal-binding-api/src/main/java/org/opendaylight/mdsal/binding/api/DataTreeChangeService.java
dom/mdsal-dom-api/src/main/java/org/opendaylight/mdsal/dom/api/DOMDataTreeChangeService.java

index 36846695ba326159b2af6da02fcad600b7997946..8ab7aa61abf9117049e0940f0b318b1d42b1fe3b 100644 (file)
@@ -36,9 +36,9 @@ public interface DataTreeChangeService extends BindingService {
      * {@link Registration#close()} method on the returned object.
      *
      * <p>
-     * You MUST explicitly unregister your listener when you no longer want to receive
-     * notifications. This is especially true in OSGi environments, where failure to
-     * do so during bundle shutdown can lead to stale listeners being still registered.
+     * You <b>MUST</b> explicitly unregister your listener when you no longer want to receive notifications. This is
+     * especially true in OSGi environments, where failure to do so during bundle shutdown can lead to stale listeners
+     * being still registered.
      *
      * @param treeId Data tree identifier of the subtree which should be watched for changes
      * @param listener Listener instance which is being registered
@@ -49,32 +49,30 @@ public interface DataTreeChangeService extends BindingService {
         @NonNull DataTreeChangeListener<T> listener);
 
     /**
-     * Registers a {@link DataTreeChangeListener} to receive
-     * notifications when data changes under a given path in the conceptual data tree.
+     * Registers a {@link DataTreeChangeListener} to receive notifications when data changes under a given path in the
+     * conceptual data tree.
      *
      * <p>
-     * You are able to register for notifications  for any node or subtree
-     * which can be represented using {@link DataTreeIdentifier}.
+     * You are able to register for notifications  for any node or subtree which can be represented using
+     * {@link DataTreeIdentifier}.
      *
      * <p>
-     * This method returns a {@link Registration} object. To
-     * "unregister" your listener for changes call the {@link Registration#close()}
-     * method on the returned object.
+     * This method returns a {@link Registration} object. To "unregister" your listener for changes call the
+     * {@link Registration#close()} method on the returned object.
      *
      * <p>
-     * You MUST explicitly unregister your listener when you no longer want to receive
-     * notifications. This is especially true in OSGi environments, where failure to
-     * do so during bundle shutdown can lead to stale listeners being still registered.
-     * @implSpec This method provides {@link DataListenerAdapter} as listener during
-     *      the registration of {@link DataTreeChangeListener}. This would allow users
-     *      to know the last state of data instead of getting details about what changed
-     *      in the entire tree.
-     * @param treeId Data tree identifier of the subtree which should be watched for
-     *            changes.
+     * You <b>MUST</b> explicitly unregister your listener when you no longer want to receive notifications. This is
+     * especially true in OSGi environments, where failure to do so during bundle shutdown can lead to stale listeners
+     * being still registered.
+     *
+     * @implSpec This method provides {@link DataListenerAdapter} as listener during the registration of
+     *           {@link DataTreeChangeListener}. This would allow users to know the last state of data instead of
+     *           getting details about what changed in the entire tree.
+     *
+     * @param treeId Data tree identifier of the subtree which should be watched for changes.
      * @param listener Listener instance which is being registered
-     * @return Listener registration object, which may be used to unregister
-     *         your listener using {@link Registration#close()} to stop
-     *         delivery of change events.
+     * @return Listener registration object, which may be used to unregister your listener using
+     *         {@link Registration#close()} to stop delivery of change events.
      */
     default <T extends DataObject> @NonNull Registration registerDataListener(
             final @NonNull DataTreeIdentifier<T> treeId, final @NonNull DataListener<T> listener) {
@@ -82,34 +80,29 @@ public interface DataTreeChangeService extends BindingService {
     }
 
     /**
-     * Registers a {@link DataTreeChangeListener} to receive
-     * notifications about the last data state when it changes under a given path in the conceptual data
-     * tree.
+     * Registers a {@link DataTreeChangeListener} to receive notifications about the last data state when it changes
+     * under a given path in the conceptual data tree.
      *
      * <p>
-     * You are able to register for notifications  for any node or subtree
-     * which can be represented using {@link DataTreeIdentifier}.
+     * You are able to register for notifications  for any node or subtree which can be represented using
+     * {@link DataTreeIdentifier}.
      *
      * <p>
-     * This method returns a {@link Registration} object. To
-     * "unregister" your listener for changes call the {@link Registration#close()}
-     * method on the returned object.
+     * This method returns a {@link Registration} object. To "unregister" your listener for changes call the
+     * {@link Registration#close()} method on the returned object.
      *
      * <p>
-     * You MUST explicitly unregister your listener when you no longer want to receive
-     * notifications. This is especially true in OSGi environments, where failure to
-     * do so during bundle shutdown can lead to stale listeners being still registered.
+     * You <b>MUST</b> explicitly unregister your listener when you no longer want to receive notifications. This is
+     * especially true in OSGi environments, where failure to do so during bundle shutdown can lead to stale listeners
+     * being still registered.
      *
-     * @implSpec This method provides {@link DataChangeListenerAdapter} as listener during
-     *      the registration of {@link DataTreeChangeListener}, which provides a comparison
-     *      of before-value and after-value.
+     * @implSpec This method provides {@link DataChangeListenerAdapter} as listener during the registration of
+     *           {@link DataTreeChangeListener}, which provides a comparison of before-value and after-value.
      *
-     * @param treeId Data tree identifier of the subtree which should be watched for
-     *            changes.
+     * @param treeId Data tree identifier of the subtree which should be watched for changes.
      * @param listener Listener instance which is being registered
-     * @return Listener registration object, which may be used to unregister
-     *         your listener using {@link Registration#close()} to stop
-     *         delivery of change events.
+     * @return Listener registration object, which may be used to unregister your listener using
+     *         {@link Registration#close()} to stop  delivery of change events.
      */
     default <T extends DataObject> @NonNull Registration registerDataChangeListener(
             final @NonNull DataTreeIdentifier<T> treeId, final @NonNull DataChangeListener<T> listener) {
index 127c2653c27b866582e86033eaa34238839c3fd1..8585feae4e9e06440bf523e53449f6ddfb1f3b6f 100644 (file)
@@ -44,7 +44,7 @@ public interface DOMDataTreeChangeService extends DOMDataBroker.Extension {
      * @param listener Listener instance which is being registered
      * @return A {@link Registration} object, which may be used to unregister your listener using
      *         {@link Registration#close()} to stop delivery of change events.
-     * @throws NullPointerException if any of the arguments is null
+     * @throws NullPointerException if any of the arguments is {@code null}
      */
     Registration registerDataTreeChangeListener(DOMDataTreeIdentifier treeId, DOMDataTreeChangeListener listener);
 }