Differential D13912

Use non deprecated replace in ForwardingSlave
ClosedPublic
Actions

Authored by jtamate on Jul 6 2018, 7:26 AM.

Details

Reviewers
dfaure
Group Reviewers
Frameworks
Commits
R241:30a0b32d10e4: Use non deprecated replace in ForwardingSlave
Summary

There is no contract that ensures entry does not have already those data inserted, so better safe than sorry.

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.
jtamate created this revision.Jul 6 2018, 7:26 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptJul 6 2018, 7:26 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
jtamate requested review of this revision.Jul 6 2018, 7:26 AM
dfaure accepted this revision.Jul 6 2018, 7:58 AM
This revision is now accepted and ready to land.Jul 6 2018, 7:58 AM
Closed by commit R241:30a0b32d10e4: Use non deprecated replace in ForwardingSlave (authored by jtamate). · Explain WhyJul 6 2018, 2:51 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/core/forwardingslavebase.cpp (6 lines)

Diff 37262

View Options

src/core/forwardingslavebase.cpp

Loading...