Skip to content

Don't output top annotations to .ajava files#6432

Open
mernst wants to merge 6 commits intotypetools:masterfrom
mernst:ainfer-no-top
Open

Don't output top annotations to .ajava files#6432
mernst wants to merge 6 commits intotypetools:masterfrom
mernst:ainfer-no-top

Conversation

@mernst
Copy link
Member

@mernst mernst commented Jan 30, 2024

Merge after #6431

@kelloggm
Copy link
Contributor

While this change looks sensible, I can't see a difference between the output of this PR and master on either ./gradlew ainferResourceLeakAjavaTest or ./gradlew ainferIndexAjavaTest, both of which seem to already not include any top annotations on master. Can you point to a test case for which you expect this PR to change the output (even if that test case's .ajava files need to be inspected by-hand to see the difference)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants