• K
    Improve/fix bugs for the cpp linter · 46812f68
    kailiu 提交于
    Summary:
    Previous our new `arc lint` has two annoying bugs:
    
    * Keeping sending false alarm that we'd put c++ system files first -- even though we've already done that.
      - this problem is caused by our linter, which doesn't give the underlying cpplint.py right file path (it gives "-" as file name), making cpplint.py work incorrectly.
    * Only works in rocksdb's root dir; Otherwise it'll throw exception saying "cannot find cpplint.py".
    
    I copied open source ArcanistCpplintLinter and modifiy it for our use.
    
    Test Plan: Ran arc lint and made sure the above-mentioned problem won't occur.
    
    Reviewers: haobo, sdong, igor, ljin, yhchiang, dhruba
    
    CC: leveldb
    
    Differential Revision: https://reviews.facebook.net/D16137
    46812f68
__phutil_library_map__.php 689 字节