astitcher commented on a change in pull request #352:
URL: https://github.com/apache/qpid-proton/pull/352#discussion_r780256333



##########
File path: cpp/include/proton/terminus.hpp
##########
@@ -99,14 +102,15 @@ class terminus {
 
     /// Obtain a reference to the AMQP dynamic node properties for the
     /// terminus.  See also lifetime_policy.
+    /// @deprecated in favor of dynamic_properties()??
     PN_CPP_EXTERN value node_properties() const;
 
-    /// Extension capabilities that are supported/requested
+    /// **Unsettled API** Extension capabilities that are supported/requested
     PN_CPP_EXTERN std::vector<symbol> capabilities() const;
 
-    /// Obtain the AMQP dynamic node properties for the
+    /// **Unsettled API** Obtain the AMQP dynamic node properties for the
     /// terminus as a standard map.

Review comment:
       Good catch




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to