Differential D24484

Remove warnings about ignoring return value of removeHeader()
ClosedPublic
Actions

Authored by gjditchfield on Oct 8 2019, 2:54 AM.

Details

Reviewers
dvratil
Commits
R180:aa13f0d66db0: Remove warnings about ignoring return value of removeHeader()
Summary

Most uses of Content::RemoveHeader() do not check its return value;
they want to remove the header if it is present, but don't care if it
isn't. This change silences the unnecessary compilation warnings.

Diff Detail

Repository
R180 PIM: KMime
Branch
removeHeader (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17427
Build 17445: arc lint + arc unit
gjditchfield created this revision.Oct 8 2019, 2:54 AM
Restricted Application added a project: KDE PIM. · View Herald TranscriptOct 8 2019, 2:54 AM
Restricted Application added a subscriber: kde-pim. · View Herald Transcript
gjditchfield requested review of this revision.Oct 8 2019, 2:54 AM
dvratil accepted this revision.Oct 8 2019, 9:32 AM
This revision is now accepted and ready to land.Oct 8 2019, 9:32 AM
Closed by commit R180:aa13f0d66db0: Remove warnings about ignoring return value of removeHeader() (authored by gjditchfield). · Explain WhyOct 8 2019, 2:06 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/kmime_content.h (2 lines)

Diff 67475

View Options

src/kmime_content.h

Loading...