Fix pylint warning and refactor project_list.
Bug: 31377083 Test: run warn.py --byproject build.log Change-Id: Id442cf6608bf5c14784063c05f5ff0b731f17486
This commit is contained in:
Chih-Hung Hsieh
147
tools/warn.py
147
tools/warn.py
@@ -1661,82 +1661,91 @@ warn_patterns = [
|
|||||||
]
|
]
|
||||||
|
|
||||||
|
|
||||||
|
def project_name_and_pattern(name, pattern):
|
||||||
|
return [name, '(^|.*/)' + pattern + '/.*: warning:']
|
||||||
|
|
||||||
|
|
||||||
|
def simple_project_pattern(pattern):
|
||||||
|
return project_name_and_pattern(pattern, pattern)
|
||||||
|
|
||||||
|
|
||||||
# A list of [project_name, file_path_pattern].
|
# A list of [project_name, file_path_pattern].
|
||||||
# project_name should not contain comma, to be used in CSV output.
|
# project_name should not contain comma, to be used in CSV output.
|
||||||
project_list = [
|
project_list = [
|
||||||
# pylint:disable=bad-whitespace,g-inconsistent-quotes,line-too-long
|
simple_project_pattern('art'),
|
||||||
['art', r"(^|.*/)art/.*: warning:"],
|
simple_project_pattern('bionic'),
|
||||||
['bionic', r"(^|.*/)bionic/.*: warning:"],
|
simple_project_pattern('bootable'),
|
||||||
['bootable', r"(^|.*/)bootable/.*: warning:"],
|
simple_project_pattern('build'),
|
||||||
['build', r"(^|.*/)build/.*: warning:"],
|
simple_project_pattern('cts'),
|
||||||
['cts', r"(^|.*/)cts/.*: warning:"],
|
simple_project_pattern('dalvik'),
|
||||||
['dalvik', r"(^|.*/)dalvik/.*: warning:"],
|
simple_project_pattern('developers'),
|
||||||
['developers', r"(^|.*/)developers/.*: warning:"],
|
simple_project_pattern('development'),
|
||||||
['development', r"(^|.*/)development/.*: warning:"],
|
simple_project_pattern('device'),
|
||||||
['device', r"(^|.*/)device/.*: warning:"],
|
simple_project_pattern('doc'),
|
||||||
['doc', r"(^|.*/)doc/.*: warning:"],
|
|
||||||
# match external/google* before external/
|
# match external/google* before external/
|
||||||
['external/google', r"(^|.*/)external/google.*: warning:"],
|
project_name_and_pattern('external/google', 'external/google.*'),
|
||||||
['external/non-google', r"(^|.*/)external/.*: warning:"],
|
project_name_and_pattern('external/non-google', 'external'),
|
||||||
['frameworks/av/camera',r"(^|.*/)frameworks/av/camera/.*: warning:"],
|
simple_project_pattern('frameworks/av/camera'),
|
||||||
['frameworks/av/cmds', r"(^|.*/)frameworks/av/cmds/.*: warning:"],
|
simple_project_pattern('frameworks/av/cmds'),
|
||||||
['frameworks/av/drm', r"(^|.*/)frameworks/av/drm/.*: warning:"],
|
simple_project_pattern('frameworks/av/drm'),
|
||||||
['frameworks/av/include',r"(^|.*/)frameworks/av/include/.*: warning:"],
|
simple_project_pattern('frameworks/av/include'),
|
||||||
['frameworks/av/media', r"(^|.*/)frameworks/av/media/.*: warning:"],
|
simple_project_pattern('frameworks/av/media'),
|
||||||
['frameworks/av/radio', r"(^|.*/)frameworks/av/radio/.*: warning:"],
|
simple_project_pattern('frameworks/av/radio'),
|
||||||
['frameworks/av/services', r"(^|.*/)frameworks/av/services/.*: warning:"],
|
simple_project_pattern('frameworks/av/services'),
|
||||||
['frameworks/av/Other', r"(^|.*/)frameworks/av/.*: warning:"],
|
project_name_and_pattern('frameworks/av/Other', 'frameworks/av'),
|
||||||
['frameworks/base', r"(^|.*/)frameworks/base/.*: warning:"],
|
simple_project_pattern('frameworks/base'),
|
||||||
['frameworks/compile', r"(^|.*/)frameworks/compile/.*: warning:"],
|
simple_project_pattern('frameworks/compile'),
|
||||||
['frameworks/minikin', r"(^|.*/)frameworks/minikin/.*: warning:"],
|
simple_project_pattern('frameworks/minikin'),
|
||||||
['frameworks/native', r"(^|.*/)frameworks/native/.*: warning:"],
|
simple_project_pattern('frameworks/native'),
|
||||||
['frameworks/opt', r"(^|.*/)frameworks/opt/.*: warning:"],
|
simple_project_pattern('frameworks/opt'),
|
||||||
['frameworks/rs', r"(^|.*/)frameworks/rs/.*: warning:"],
|
simple_project_pattern('frameworks/rs'),
|
||||||
['frameworks/webview', r"(^|.*/)frameworks/webview/.*: warning:"],
|
simple_project_pattern('frameworks/webview'),
|
||||||
['frameworks/wilhelm', r"(^|.*/)frameworks/wilhelm/.*: warning:"],
|
simple_project_pattern('frameworks/wilhelm'),
|
||||||
['frameworks/Other', r"(^|.*/)frameworks/.*: warning:"],
|
project_name_and_pattern('frameworks/Other', 'frameworks'),
|
||||||
['hardware/akm', r"(^|.*/)hardware/akm/.*: warning:"],
|
simple_project_pattern('hardware/akm'),
|
||||||
['hardware/broadcom', r"(^|.*/)hardware/broadcom/.*: warning:"],
|
simple_project_pattern('hardware/broadcom'),
|
||||||
['hardware/google', r"(^|.*/)hardware/google/.*: warning:"],
|
simple_project_pattern('hardware/google'),
|
||||||
['hardware/intel', r"(^|.*/)hardware/intel/.*: warning:"],
|
simple_project_pattern('hardware/intel'),
|
||||||
['hardware/interfaces', r"(^|.*/)hardware/interfaces/.*: warning:"],
|
simple_project_pattern('hardware/interfaces'),
|
||||||
['hardware/libhardware',r"(^|.*/)hardware/libhardware/.*: warning:"],
|
simple_project_pattern('hardware/libhardware'),
|
||||||
['hardware/libhardware_legacy',r"(^|.*/)hardware/libhardware_legacy/.*: warning:"],
|
simple_project_pattern('hardware/libhardware_legacy'),
|
||||||
['hardware/qcom', r"(^|.*/)hardware/qcom/.*: warning:"],
|
simple_project_pattern('hardware/qcom'),
|
||||||
['hardware/ril', r"(^|.*/)hardware/ril/.*: warning:"],
|
simple_project_pattern('hardware/ril'),
|
||||||
['hardware/Other', r"(^|.*/)hardware/.*: warning:"],
|
project_name_and_pattern('hardware/Other', 'hardware'),
|
||||||
['kernel', r"(^|.*/)kernel/.*: warning:"],
|
simple_project_pattern('kernel'),
|
||||||
['libcore', r"(^|.*/)libcore/.*: warning:"],
|
simple_project_pattern('libcore'),
|
||||||
['libnativehelper', r"(^|.*/)libnativehelper/.*: warning:"],
|
simple_project_pattern('libnativehelper'),
|
||||||
['ndk', r"(^|.*/)ndk/.*: warning:"],
|
simple_project_pattern('ndk'),
|
||||||
# match vendor/unbungled_google/packages before other packages
|
# match vendor/unbungled_google/packages before other packages
|
||||||
['unbundled_google', r"(^|.*/)unbundled_google/.*: warning:"],
|
simple_project_pattern('unbundled_google'),
|
||||||
['packages', r"(^|.*/)packages/.*: warning:"],
|
simple_project_pattern('packages'),
|
||||||
['pdk', r"(^|.*/)pdk/.*: warning:"],
|
simple_project_pattern('pdk'),
|
||||||
['prebuilts', r"(^|.*/)prebuilts/.*: warning:"],
|
simple_project_pattern('prebuilts'),
|
||||||
['system/bt', r"(^|.*/)system/bt/.*: warning:"],
|
simple_project_pattern('system/bt'),
|
||||||
['system/connectivity', r"(^|.*/)system/connectivity/.*: warning:"],
|
simple_project_pattern('system/connectivity'),
|
||||||
['system/core', r"(^|.*/)system/core/.*: warning:"],
|
simple_project_pattern('system/core'),
|
||||||
['system/extras', r"(^|.*/)system/extras/.*: warning:"],
|
simple_project_pattern('system/extras'),
|
||||||
['system/gatekeeper', r"(^|.*/)system/gatekeeper/.*: warning:"],
|
simple_project_pattern('system/gatekeeper'),
|
||||||
['system/keymaster', r"(^|.*/)system/keymaster/.*: warning:"],
|
simple_project_pattern('system/keymaster'),
|
||||||
['system/libhwbinder', r"(^|.*/)system/libhwbinder/.*: warning:"],
|
simple_project_pattern('system/libhwbinder'),
|
||||||
['system/media', r"(^|.*/)system/media/.*: warning:"],
|
simple_project_pattern('system/media'),
|
||||||
['system/netd', r"(^|.*/)system/netd/.*: warning:"],
|
simple_project_pattern('system/netd'),
|
||||||
['system/security', r"(^|.*/)system/security/.*: warning:"],
|
simple_project_pattern('system/security'),
|
||||||
['system/sepolicy', r"(^|.*/)system/sepolicy/.*: warning:"],
|
simple_project_pattern('system/sepolicy'),
|
||||||
['system/tools', r"(^|.*/)system/tools/.*: warning:"],
|
simple_project_pattern('system/tools'),
|
||||||
['system/vold', r"(^|.*/)system/vold/.*: warning:"],
|
simple_project_pattern('system/vold'),
|
||||||
['system/Other', r"(^|.*/)system/.*: warning:"],
|
project_name_and_pattern('system/Other', 'system'),
|
||||||
['toolchain', r"(^|.*/)toolchain/.*: warning:"],
|
simple_project_pattern('toolchain'),
|
||||||
['test', r"(^|.*/)test/.*: warning:"],
|
simple_project_pattern('test'),
|
||||||
['tools', r"(^|.*/)tools/.*: warning:"],
|
simple_project_pattern('tools'),
|
||||||
# match vendor/google* before vendor/
|
# match vendor/google* before vendor/
|
||||||
['vendor/google', r"(^|.*/)vendor/google.*: warning:"],
|
project_name_and_pattern('vendor/google', 'vendor/google.*'),
|
||||||
['vendor/non-google', r"(^|.*/)vendor/.*: warning:"],
|
project_name_and_pattern('vendor/non-google', 'vendor'),
|
||||||
# keep out/obj and other patterns at the end.
|
# keep out/obj and other patterns at the end.
|
||||||
['out/obj', r".*/(gen|obj[^/]*)/(include|EXECUTABLES|SHARED_LIBRARIES|STATIC_LIBRARIES|NATIVE_TESTS)/.*: warning:"],
|
['out/obj',
|
||||||
['other', r".*: warning:"],
|
'.*/(gen|obj[^/]*)/(include|EXECUTABLES|SHARED_LIBRARIES|'
|
||||||
|
'STATIC_LIBRARIES|NATIVE_TESTS)/.*: warning:'],
|
||||||
|
['other', '.*'] # all other unrecognized patterns
|
||||||
]
|
]
|
||||||
|
|
||||||
project_patterns = []
|
project_patterns = []
|
||||||
|
|||||||
Reference in New Issue
Block a user