Skip to content

Fix Javadoc issues, hide more Error Prone warnings#10360

Open
zbynek wants to merge 3 commits into
gwtproject:mainfrom
zbynek:ep-javadoc
Open

Fix Javadoc issues, hide more Error Prone warnings#10360
zbynek wants to merge 3 commits into
gwtproject:mainfrom
zbynek:ep-javadoc

Conversation

@zbynek

@zbynek zbynek commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator

As the Error Prone upgrade and filtering PRs were merged around the same time, compiling GWT now produces thousands of warnings. This PR adds more suppressios and fixes warnings related to Javadoc.

Partial fix for #8428

@zbynek zbynek added the ready This PR has been reviewed by a maintainer and is ready for a CI run. label Jul 5, 2026

@niloc132 niloc132 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can probably close (or at least reference) #8428 in the description

Comment thread user/src/com/google/gwt/dom/client/MediaElement.java Outdated
Comment thread user/src/com/google/gwt/dom/client/Touch.java Outdated
Comment thread user/src/com/google/gwt/event/dom/client/TouchEvent.java Outdated
Comment thread user/src/com/google/web/bindery/autobean/vm/impl/FactoryHandler.java Outdated
Co-authored-by: Colin Alworth <colin@vertispan.com>
@niloc132 niloc132 added this to the 2.14 milestone Jul 9, 2026
@zbynek

zbynek commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator Author

Can probably close (or at least reference) #8428 in the description

I got a bit closer to that by resolving the remaining issues spotted by Error Prone.

There are still 86 issues reported in console by the Javadoc task (down from 100+).

@niloc132

Copy link
Copy Markdown
Member

Do you want to finish them up, or maybe still link and mark as partial? I'd generally love to get to the point of having tickets for many/most changes just as a practical matter.

@zbynek

zbynek commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator Author

If you don't mind this PR getting bigger, I'll fix them here.

@niloc132

Copy link
Copy Markdown
Member

Go for it - I've read each change (and marked as read) so incremental additions won't be too bad.

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

Labels

ready This PR has been reviewed by a maintainer and is ready for a CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants