On Tue, 5 Feb 2013, Danny Al-Gaaf wrote: > These patches fix some issues found by using cppchecker/clang++. A couple comments, but otherwise these look good. Thanks, Danny! sage > > 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 | 10 +++++++--- > src/include/types.h | 2 +- > src/include/xlist.h | 6 +++--- > src/messages/MOSDRepScrub.h | 2 +- > src/msg/Message.h | 4 ++-- > 8 files changed, 23 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 > > -- 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
