Web lists-archives.com

Re: Review Request 129183: [baloo] CLazy fixes: removed unused member var, add Q_OBJECT




This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129183/

This change has been marked as submitted.


Review request for Baloo.
By Christian Ehrlicher.

Updated Nov. 6, 2016, 10:56 a.m.

Changes

Submitted with commit edcda57297a1fc4ff74555afd402cf9c69a2db25 by Christian Ehrlicher to branch master.
Repository: baloo

Description

CLayz warns about an unused member var in IOHandler and a missing Q_OBJECT macro in TimeEstimator which produces a lot of noise in the compile output.

Ping! No interest in this patch? Should I discard it?

Testing

Compiles fine

Diffs

  • src/file/extractor/app.cpp (6843165)
  • src/file/extractor/autotests/iohandlertest.cpp (2c8eeaa)
  • src/file/extractor/iohandler.h (c0dfbbb)
  • src/file/extractor/iohandler.cpp (b49a2b2)
  • src/file/timeestimator.h (11137d3)

View Diff