Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit cf34c0da authored by Chih-hung Hsieh's avatar Chih-hung Hsieh Committed by Gerrit Code Review
Browse files

Merge "List checks that should not occur in this module."

parents 90e39e44 09d11b38
Loading
Loading
Loading
Loading
+12 −1
Original line number Diff line number Diff line
@@ -181,15 +181,26 @@ cc_library {
        "--header-filter=^.*frameworks/native/libs/binder/.*.h$",
    ],
    tidy_checks_as_errors: [
        "*",
        // Explicitly list the checks that should not occur in this module.
        "abseil-*",
        "android-*",
        "bugprone-*",
        "cert-*",
        "clang-analyzer-*",
        "-clang-analyzer-core.CallAndMessage",
        "-clang-analyzer-core.uninitialized.Assign",
        "-clang-analyzer-unix.Malloc",
        "-clang-analyzer-deadcode.DeadStores",
        "-clang-analyzer-optin.cplusplus.UninitializedObject",
        "google-*",
        "-google-readability-*",
        "-google-runtime-references",
        "misc-*",
        "-misc-no-recursion",
        "-misc-redundant-expression",
        "-misc-unused-using-decls",
        "performance*",
        "portability*",
    ],
}

+14 −1
Original line number Diff line number Diff line
@@ -104,15 +104,28 @@ cc_library {
        "--header-filter=^.*frameworks/native/libs/binder/.*.h$",
    ],
    tidy_checks_as_errors: [
        "*",
        // Explicitly list the checks that should not occur in this module.
        "abseil-*",
        "android-*",
        "bugprone-*",
        "cert-*",
        "clang-analyzer-*",
        "-clang-analyzer-core.CallAndMessage",
        "-clang-analyzer-core.uninitialized.Assign",
        "-clang-analyzer-unix.Malloc",
        "-clang-analyzer-deadcode.DeadStores",
        "-clang-analyzer-optin.cplusplus.UninitializedObject",
        "google-*",
        "-google-readability-*",
        "-google-runtime-references",
        "misc-*",
        "-misc-no-recursion",
        "-misc-non-private-member-variables-in-classes",
        "-misc-redundant-expression",
        "-misc-unused-parameters",
        "-misc-unused-using-decls",
        "performance*",
        "portability*",
    ],
}

+2 −1
Original line number Diff line number Diff line
@@ -83,7 +83,8 @@ class SharedRefBase {
    template <class T, class... Args>
    static std::shared_ptr<T> make(Args&&... args) {
        T* t = new T(std::forward<Args>(args)...);
        return t->template ref<T>();
        // warning: Potential leak of memory pointed to by 't' [clang-analyzer-unix.Malloc]
        return t->template ref<T>();  // NOLINT(clang-analyzer-unix.Malloc)
    }

    static void operator delete(void* p) { std::free(p); }