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.

Sign up for GitHub

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

Jump to bottom

Compiler warnings to be fixed #800

Closed
stleary opened this issue Oct 15, 2023 · 4 comments
Closed

Compiler warnings to be fixed #800

stleary opened this issue Oct 15, 2023 · 4 comments

Comments

@stleary
Copy link
Owner

stleary commented Oct 15, 2023

src/main/java/org/json/ParserConfiguration.java:77: warning: no @param for
public T withKeepStrings(final boolean newVal) {
src/main/java/org/json/ParserConfiguration.java:101: warning: no @param for
public T withMaxNestingDepth(int maxNestingDepth) {
@farukhfrk
Copy link

farukhfrk commented Oct 16, 2023 •
edited

I would like to work on this issue, could you please assign it to me?

@stleary
Copy link
Owner Author

stleary commented Oct 16, 2023

@farukhfrk Issues are not assigned, contributers are free to work on any issue.

@theKnightsOfRohan theKnightsOfRohan mentioned this issue Oct 17, 2023
Closed
@theKnightsOfRohan
Copy link
Contributor

@farukhfrk Sorry for taking this issue from you, please look through my PR and provide any suggestions

@theKnightsOfRohan theKnightsOfRohan mentioned this issue Oct 17, 2023
Merged
@stleary
Copy link
Owner Author

stleary commented Oct 17, 2023

Closing due to fixed in #808

@stleary stleary closed this as completed Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@stleary @farukhfrk @theKnightsOfRohan