Here another set of patches to fix issues found by clang/clang++ and cppchecker. Danny Al-Gaaf (15): kv_flat_btree_async.cc: use vector instead of VLA's common/config.h: declaration of config_option as struct src/msg/msg_types.h: pass function parameter by reference fuse_ll.cc: fix -Wgnu-designator common/AsyncReserver.h: use empty() instead of size() common/WorkQueue.h: use empty() instead of size() common/config.cc: fix -Wgnu-designator src/log/Entry.h: pass function parameter by reference src/mon/PGMonitor.cc: remove unused variable src/msg/Messenger.h: pass function parameter by reference src/osd/OSD.h: use empty() instead of size() src/osd/PG.h: use empty() instead of size() src/osd/osd_types.h: pass function parameter by reference test_mon_workloadgen.cc: fix -Wgnu librados/librados.cc: fix implicitly-defined namespace 'std' src/client/fuse_ll.cc | 68 +++++++++++++++--------------- src/common/AsyncReserver.h | 2 +- src/common/WorkQueue.h | 2 +- src/common/config.cc | 14 +++--- src/common/config.h | 2 +- src/key_value_store/kv_flat_btree_async.cc | 14 +++--- src/librados/librados.cc | 16 ++++++- src/log/Entry.h | 2 +- src/mon/PGMonitor.cc | 1 - src/msg/Messenger.h | 2 +- src/msg/msg_types.h | 2 +- src/osd/OSD.h | 4 +- src/osd/PG.h | 2 +- src/osd/osd_types.h | 2 +- src/test/mon/test_mon_workloadgen.cc | 3 +- 15 files changed, 74 insertions(+), 62 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