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
[modernize] use-equals-default #16669
Conversation
Moving default/empty ctors/dtors/... from cpp to h can have side effects (compiler errors), because you make the ctors/dtors/... inline with this. |
yes, it can have consequences, in particular if you have forwarded members. |
What I said! Look at Jenkins' state. |
I dont think it is good to include the header for CBinaryAddonCache in ServiceBroker.h. |
i totally agree, if the c/dtor need to be in cpp to avoid includes, that's what you do. just, if there is no reason, i prefer to see it in the header (cause that's what i look at to study classes, and the indirection to find = default in the cpp is then confusing (i expect an implementation in that case). |
+1
+1 |
5aea93c
to
390825e
Compare
390825e
to
4372ea9
Compare
Fine by me. |
Do it.
|
4372ea9
to
7498e0b
Compare
7498e0b
to
eecc666
Compare
Description
run clang-tidy modernize-use-equals-default and applied the suggested fixes
Types of change
Checklist: