Merge "Apilint: Enforce final builders"
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