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
Fix a few use-after-free bugs #2116
Conversation
Signed-off-by: TheKodeToad <TheKodeToad@proton.me>
Signed-off-by: TheKodeToad <TheKodeToad@proton.me>
Signed-off-by: TheKodeToad <TheKodeToad@proton.me>
Exception(const QString& message) : std::exception(), m_message(message) { qCritical() << "Exception:" << message; } | ||
Exception(const Exception& other) : std::exception(), m_message(other.cause()) {} | ||
Exception(const QString& message) : std::exception(), m_message(message.toUtf8()) { qCritical() << "Exception:" << message; } | ||
Exception(const Exception& other) : std::exception(), m_message(other.m_message) {} |
Check warning
Code scanning / CodeQL
Inconsistent definition of copy constructor and assignment ('Rule of Two') Warning
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.
lgtm
but do not forget to run clang-format(see the failing checks)
Signed-off-by: TheKodeToad <TheKodeToad@proton.me>
Git push to origin failed for release-8.x with exitcode 1 |
Successfully created backport PR for |
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin release-8.x
git worktree add -d .worktree/backport-2116-to-release-8.x origin/release-8.x
cd .worktree/backport-2116-to-release-8.x
git switch --create backport-2116-to-release-8.x
git cherry-pick -x ded77e61832661557a6985d9aa587f3b5d1e1f87 ff989b4305d5ae4578d2154afe27ce6bf72cadaf 7f4c74ffbe413fcf191842606c12056fbc6493c7 8cf2a04f317c59920e8af90d7e16fea872b2c122 |
No description provided.