Fix some compiler warnings

Authored by schwarzer on Aug 9 2019, 12:23 PM.



While looking for something else, I got rid of some of the warnings, the compiler gave me here.

Test Plan


Diff Detail

R363 Print Manager
Lint Skipped
Unit Tests Skipped
schwarzer created this revision.Aug 9 2019, 12:23 PM
Restricted Application added a subscriber: kde-utils-devel. · View Herald TranscriptAug 9 2019, 12:23 PM
schwarzer requested review of this revision.Aug 9 2019, 12:23 PM
dakon added a subscriber: dakon.Aug 9 2019, 12:30 PM
dakon added inline comments.
36 ↗(On Diff #63400)

Can you initialize this directly in the class?

schwarzer added inline comments.Aug 9 2019, 2:30 PM
36 ↗(On Diff #63400)

Of course. :)

schwarzer updated this revision to Diff 63417.Aug 9 2019, 2:31 PM

init member in header

I think I updated to a new diff but it is not showing the new changes ... Any hint?

schwarzer updated this revision to Diff 63438.Aug 9 2019, 5:21 PM

My bad, this should now be the right diff.

Any more complaints/suggestions? I do not know who would be a good assignee for print-manager since several people more or less sporadically work on it.

dakon accepted this revision.Aug 14 2019, 7:47 PM

Since noone else steps in, I'll put my module maintainer head on.

This revision is now accepted and ready to land.Aug 14 2019, 7:47 PM