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

Commit aa1e21f3 authored by Chih-hung Hsieh's avatar Chih-hung Hsieh Committed by Automerger Merge Worker
Browse files

Merge "List checks that should not occur in this module." am: cf34c0da am: 511cf655 

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/1546695

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: Ie821392bdbe5eeb884d9d28c38a7d59a43265b6a
parents 01c5c1a6 511cf655

libs/binder/Android.bp

+12 −1
Original line number Original line Diff line number Diff line
@@ -181,15 +181,26 @@ cc_library { 
        "--header-filter=^.*frameworks/native/libs/binder/.*.h$",

        "--header-filter=^.*frameworks/native/libs/binder/.*.h$",

    ],

    ],

    tidy_checks_as_errors: [

    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.CallAndMessage",

        "-clang-analyzer-core.uninitialized.Assign",

        "-clang-analyzer-core.uninitialized.Assign",

        "-clang-analyzer-unix.Malloc",

        "-clang-analyzer-unix.Malloc",

        "-clang-analyzer-deadcode.DeadStores",

        "-clang-analyzer-deadcode.DeadStores",

        "-clang-analyzer-optin.cplusplus.UninitializedObject",

        "-clang-analyzer-optin.cplusplus.UninitializedObject",

        "google-*",

        "-google-readability-*",

        "-google-runtime-references",

        "misc-*",

        "-misc-no-recursion",

        "-misc-no-recursion",

        "-misc-redundant-expression",

        "-misc-redundant-expression",

        "-misc-unused-using-decls",

        "-misc-unused-using-decls",

        "performance*",

        "portability*",

    ],

    ],

}

}

libs/binder/ndk/Android.bp

+14 −1
Original line number Original line Diff line number Diff line
@@ -104,15 +104,28 @@ cc_library { 
        "--header-filter=^.*frameworks/native/libs/binder/.*.h$",

        "--header-filter=^.*frameworks/native/libs/binder/.*.h$",

    ],

    ],

    tidy_checks_as_errors: [

    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.CallAndMessage",

        "-clang-analyzer-core.uninitialized.Assign",

        "-clang-analyzer-core.uninitialized.Assign",

        "-clang-analyzer-unix.Malloc",

        "-clang-analyzer-unix.Malloc",

        "-clang-analyzer-deadcode.DeadStores",

        "-clang-analyzer-deadcode.DeadStores",

        "-clang-analyzer-optin.cplusplus.UninitializedObject",

        "-clang-analyzer-optin.cplusplus.UninitializedObject",

        "google-*",

        "-google-readability-*",

        "-google-runtime-references",

        "misc-*",

        "-misc-no-recursion",

        "-misc-no-recursion",

        "-misc-non-private-member-variables-in-classes",

        "-misc-redundant-expression",

        "-misc-redundant-expression",

        "-misc-unused-parameters",

        "-misc-unused-using-decls",

        "-misc-unused-using-decls",

        "performance*",

        "portability*",

    ],

    ],

}

}

libs/binder/ndk/include_cpp/android/binder_interface_utils.h

+2 −1
Original line number Original line Diff line number Diff line
@@ -83,7 +83,8 @@ class SharedRefBase { 
    template <class T, class... Args>

    template <class T, class... Args>

    static std::shared_ptr<T> make(Args&&... args) {

    static std::shared_ptr<T> make(Args&&... args) {

        T* t = new T(std::forward<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); }

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