Differential D18576

Simpe C updates to ksysguardd code
ClosedPublic
Actions

Authored by tcanabrava on Jan 28 2019, 10:52 AM.

Details

Reviewers
apol
Commits
R106:b4bde28dc7af: Simpe C updates to ksysguardd code
Summary

Use C 11, it has better defaults
Simplify if - chain
Reduce variable scope
Don't check before free

Diff Detail

Repository
R106 KSysguard
Branch
simplify_c_code
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 7615
Build 7633: arc lint + arc unit
tcanabrava created this revision.Jan 28 2019, 10:52 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 28 2019, 10:52 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
tcanabrava requested review of this revision.Jan 28 2019, 10:52 AM
apol accepted this revision.Jan 29 2019, 5:35 PM
apol added a subscriber: apol.

LGTM +1

This revision is now accepted and ready to land.Jan 29 2019, 5:35 PM
Closed by commit R106:b4bde28dc7af: Simpe C updates to ksysguardd code (authored by tcanabrava). · Explain WhyFeb 14 2019, 9:31 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 50419

View Options

ksysguardd/CMakeLists.txt

Loading...
View Options

ksysguardd/Command.c

Loading...
View Options

ksysguardd/PWUIDCache.c

Loading...
View Options

ksysguardd/conf.c

Loading...
View Options

ksysguardd/ksysguardd.c

Loading...