-
Notifications
You must be signed in to change notification settings - Fork 613
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
[docs] Add missing JavaDocs #6125
[docs] Add missing JavaDocs #6125
Conversation
a541d41
to
c28205d
Compare
How many JavaDoc warnings are there now? |
2411 warnings. At this rate, we may need a small army of volunteers if we want it done within a month. |
Seems like maybe some of the functional changes should be a different PR that's easier to review (ie protected -> private members) |
C++ already had them as private members. |
You sure it's 2411 warnings? When I build docs for main, I get 2423 warnings. I refuse to believe you only removed 12 warnings, it should be way more. |
I got 2216 warnings. That's 207 warnings removed for #5808. |
I'm using JDK 21. I got 2756 warnings on main. |
c28205d
to
0e609ba
Compare
0e609ba
to
8563cd3
Compare
No description provided.