Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Annotate the new repeat API on StringBuilder and friends. #84

Merged
merged 1 commit into from
Oct 5, 2024

Conversation

cpovirk
Copy link
Collaborator

@cpovirk cpovirk commented Oct 4, 2024

I'm not sure that annotation in AbstractStringBuilder matter much, but
I'm not ruling it out that an expression could have that implicit type
if you wrote something like b ? new StringBuilder() : new StringBuffer(). And anyway, it was nice to keep AbstractStringBuilder
up to date while having reference to its implementation so that I could
then easily cross-reference between it and the subclasses.

I'm not sure that annotation in `AbstractStringBuilder` matter much, but
I'm not ruling it out that an expression could have that implicit type
if you wrote something like `b ? new StringBuilder() : new
StringBuffer()`. And anyway, it was nice to keep `AbstractStringBuilder`
up to date while having reference to its implementation so that I could
then easily cross-reference between it and the subclasses.
@cpovirk cpovirk merged commit 8b6c927 into main Oct 5, 2024
18 of 30 checks passed
@cpovirk cpovirk deleted the stringbuild branch October 5, 2024 12:37
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.

2 participants