Differential D26897

Fix two warnings, missing Q_FALLTHROUGH
ClosedPublic
Actions

Authored by meven on Jan 24 2020, 1:59 PM.

Details

Reviewers
dfaure
davidedmundson
Group Reviewers
Frameworks
Commits
R241:e6807dea4735: Fix two warnings, missing Q_FALLTHROUGH

Diff Detail

Repository
R241 KIO
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
meven created this revision.Jan 24 2020, 1:59 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJan 24 2020, 1:59 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
meven requested review of this revision.Jan 24 2020, 1:59 PM
davidedmundson accepted this revision.Jan 24 2020, 2:02 PM
davidedmundson added a subscriber: davidedmundson.
davidedmundson added inline comments.
src/core/copyjob.cpp
1223 ↗(On Diff #74312)

The comments are somewhat redundant now

x2

This revision is now accepted and ready to land.Jan 24 2020, 2:02 PM
Closed by commit R241:e6807dea4735: Fix two warnings, missing Q_FALLTHROUGH (authored by meven). · Explain WhyJan 24 2020, 2:33 PM
This revision was automatically updated to reflect the committed changes.
meven marked an inline comment as done.

Revision Contents
Changeset List

PathPackages
Msrc/core/copyjob.cpp (2 lines)

Diff 74313

View Options

src/core/copyjob.cpp

Loading...