THanks- I just pulled this in, along with the rbd-fuse fix. On Wed, 6 Feb 2013, Danny Al-Gaaf wrote: > 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
