diff options
| author | Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> | 2023-11-06 17:00:53 +0000 |
|---|---|---|
| committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2023-11-06 17:00:53 +0000 |
| commit | 4dda59a86ebf61202d4208b813a922d9b81e3553 (patch) | |
| tree | bef7a1ea5f49adeed5e35c11e5977aa82eda02ef | |
| parent | 857d197dbf8c2ab7d0a25bab9ff2c022dc3a08f2 (diff) | |
| parent | 9d927f93670a90d693b11ff2237900c6e6a38993 (diff) | |
| download | docs-4dda59a86ebf61202d4208b813a922d9b81e3553.tar.gz | |
Merge "Add section for Manager classes." into main
| -rw-r--r-- | api-guidelines/classes.md | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/api-guidelines/classes.md b/api-guidelines/classes.md index e998352..4499a5c 100644 --- a/api-guidelines/classes.md +++ b/api-guidelines/classes.md @@ -167,6 +167,12 @@ worthwhile to have a wrapper layer around it, to match other API guidelines and to make it easier to use the same public API for a new version of the IPC interface, if that ever becomes necessary. +### `Manager` classes must be `final`. + +Manager classes should be declared as `final`. Manager classes talk to system +services and are the single point of interaction. There is no need for +customization so declare it as `final`. + ### Do not use `CompletableFuture` or `Future` <a name="classes-avoid-future"></a> `java.util.concurrent.CompletableFuture` has a large API surface that permits |