These patches fix some issues found by using cppchecker/clang++. --- v2: - [4/8]: don't copy append_buffer and last_p - [7/8]: use C++ initalize syntax for constructor Danny Al-Gaaf (8): WorkQueue.h: fix cast ceph_crypto.cc: remove unused shutdown() outside crypto ifdef's obj_bencher.cc: use vector instead of VLA's include/buffer.h: fix operator= include/types.h: change operator<< function parameter include/xlist.h: fix C-style pointer casting messages/MOSDRepScrub.h: initialize member variable in constructor msg/Message.h: fix C-style pointer casting src/common/WorkQueue.h | 4 ++-- src/common/ceph_crypto.cc | 1 - src/common/obj_bencher.cc | 13 +++++++------ src/include/buffer.h | 8 +++++--- src/include/types.h | 2 +- src/include/xlist.h | 6 +++--- src/messages/MOSDRepScrub.h | 5 ++++- src/msg/Message.h | 4 ++-- 8 files changed, 24 insertions(+), 19 deletions(-) -- 1.8.1.2 -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html