Fixes from SCA

[dalgaaf]

Some common fixes:

• make single argument constructors explicit
• mark some functions with override
• init some variables in the init list for performance reasons

[dalgaaf]

Again not sure if this really caused by the changes. It compiles fine and make check also works here (without the integration tests)

[jrse]

make check-valgrind fails, due to uninitialized member of RadosSaveLogEntry.
RadosSaveLogEntry(const std::string &oid_, const std::string &ns_, const std::string &pool_, const std::string &op_) : oid(oid_),
ns(ns),
pool(pool_),
op(op_) {}

the member ns is not properly initialized! little typo :-) (missing underline _)

you can re-check if you do make check-valgrind in the dovecot-ceph-plugin/src/tests directory.

[dalgaaf]

@jrse thanks for spotting this issue.