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

Don't crash if burstValues() has only one argument #54

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

swankjesse
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@JakeWharton JakeWharton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this emit a warning? Because the parameter is useless in this case, no?

@swankjesse
Copy link
Collaborator Author

It’s totally unnecessary but I don’t think it’s worth a warning. Like an enum with only one constant?

@JakeWharton
Copy link
Collaborator

I think I would warn for that, too. Although it's less likely because the overhead of declaring an enum is higher.

@swankjesse swankjesse force-pushed the jwilson.1028.booleans branch from d364b69 to e631e2d Compare October 29, 2024 23:48
@swankjesse swankjesse force-pushed the jwilson.1028.single_burstValues branch from 7d0f326 to 3d514a0 Compare October 29, 2024 23:50
@swankjesse
Copy link
Collaborator Author

Yeah good call. Added a tracking issue!

Base automatically changed from jwilson.1028.booleans to main October 29, 2024 23:56
@swankjesse swankjesse force-pushed the jwilson.1028.single_burstValues branch from 3d514a0 to e8dd9ee Compare October 29, 2024 23:57
@swankjesse swankjesse merged commit 589fd45 into main Oct 30, 2024
1 check passed
@swankjesse swankjesse deleted the jwilson.1028.single_burstValues branch October 30, 2024 00:14
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