-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mark deprecated functionality with C++ attributes and resolve internal usage of deprecated functions #4551
base: main
Are you sure you want to change the base?
Conversation
f2c5384
to
bc4b50a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure if @deprecated
is needed/used by PocoDoc
, @obiltschnig would know
bc4b50a
to
3999dcd
Compare
@@ -85,13 +85,13 @@ class Foundation_API LocalDateTime | |||
/// Creates a LocalDateTime from the UTC time given in dateTime, | |||
/// using the time zone differential of the current time zone. | |||
|
|||
//@ deprecated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@obiltschnig, @aleks-f: Some ctors/functions in this class seem not to have an alternative to be used. They are for example used in Dynamic/Var.h.
Is complete class deprecated?
//@deprecated | ||
class RSAKey; | ||
class [[deprecated]] ECKey; | ||
class [[deprecated]] RSAKey; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@obiltschnig , what shall be used instead of these?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…eate valid binaries.
3999dcd
to
f63afd3
Compare
f63afd3
to
da8037a
Compare
Closes #4426.