necouchman commented on a change in pull request #546:
URL: https://github.com/apache/guacamole-client/pull/546#discussion_r512902068
##########
File path: guacamole-ext/src/main/java/org/apache/guacamole/net/auth/User.java
##########
@@ -103,7 +103,26 @@
* If an error occurs while reading the history of this user, or if
* permission is denied.
*/
+ @Deprecated
Review comment:
Okay, so what if the `default` implementations of these methods throw a
`GuacamoleUnsupportedException()` exception, which is what several of the
implementations of `User` do, anyway? Then, any classes wishing to implement
have to provide actual implementations of those methods, and can choose which
they choose to implement?
That seems too simple, so there must be some case I'm not thinking of that
would not be covered by this?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
