On Tue, 14 Feb 2017, liuchang0812 wrote: > Hi, Sage and Cephers: > > I saw a issue "add override where possible when implementing an > interface". So I used a LLVM tool called clang-tidy to do it > automatically. . > > I have made a big changes: > > 246 files changed, 1305 insertions(+), 1305 deletions(-) > > I could build it successfully, should I make a Pull-Request? Yeah sure! If you could break it up by subsystem (rgw, osd, rbd, mds, everything else) that would help. Thanks! > BTW, I am pleasure to write a simple documentation about this tool if > we need this. Sure! Something in doc/dev, or even just an email to the list? sage -- 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