diff options
| author | Makoto Onuki <omakoto@google.com> | 2020-06-22 10:17:02 -0700 |
|---|---|---|
| committer | Makoto Onuki <omakoto@google.com> | 2020-06-22 16:33:53 -0700 |
| commit | de6e6f2098042bb91f7dac965d2b047c74c920f2 (patch) | |
| tree | 021eaed668646fbd3fff57e9b53ef21285540e0d /tools/aapt2/cmd/Link.cpp | |
| parent | 42a637d4ee4b35f13bfd8424765b1c8ae5e2844e (diff) | |
Don't add API annotations in the internal R.java
I'm trying to enable a check for the following structure:
```
/** @hide */
public class Class1 {
/** @hide */
@SystemApi // Invalid because the class is hidden.
public void method1() { }
}
```
The internal R.java file violates this, which this change is going to fix.
Bug: 159162473
Test: build (treehugger)
Test: atest aapt2_tests
Change-Id: I613e8611ddaf5f8e4761d351d4cd0142d59c7cc9
Diffstat (limited to 'tools/aapt2/cmd/Link.cpp')
| -rw-r--r-- | tools/aapt2/cmd/Link.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tools/aapt2/cmd/Link.cpp b/tools/aapt2/cmd/Link.cpp index 3a3fb2826b74..72cb41a1b172 100644 --- a/tools/aapt2/cmd/Link.cpp +++ b/tools/aapt2/cmd/Link.cpp @@ -1272,7 +1272,8 @@ class Linker { return false; } - ClassDefinition::WriteJavaFile(manifest_class.get(), package_utf8, true, &fout); + ClassDefinition::WriteJavaFile(manifest_class.get(), package_utf8, true, + false /* strip_api_annotations */, &fout); fout.Flush(); if (fout.HadError()) { |