Merge "Apilint: Enforce final builders"
am: 2886d307bd
Change-Id: I065ff3fede26d527860606ee60b2b6ca8a6d4fbd
This commit is contained in:
@@ -1109,6 +1109,9 @@ def verify_builder(clazz):
|
|||||||
if not has_build:
|
if not has_build:
|
||||||
warn(clazz, None, None, "Missing build() method")
|
warn(clazz, None, None, "Missing build() method")
|
||||||
|
|
||||||
|
if "final" not in clazz.split:
|
||||||
|
error(clazz, None, None, "Builder should be final")
|
||||||
|
|
||||||
|
|
||||||
def verify_aidl(clazz):
|
def verify_aidl(clazz):
|
||||||
"""Catch people exposing raw AIDL."""
|
"""Catch people exposing raw AIDL."""
|
||||||
|
|||||||
Reference in New Issue
Block a user