Merge "Apilint: Enforce final builders" am: 2886d307bd

am: bb74d55781 Change-Id: I6eeec8fed36c1c9c950395a834eaa33e31fbaea2
2019-02-28 02:19:28 -08:00
parent be412a4ecf bb74d55781
commit 77864f7dac
1 changed files with 3 additions and 0 deletions
--- a/tools/apilint/apilint.py
+++ b/tools/apilint/apilint.py
@@ -1109,6 +1109,9 @@ def verify_builder(clazz):
    if not has_build:
        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):
    """Catch people exposing raw AIDL."""