Repository navigation
fix(perc-toolkit): Xlint batch 1 — real generics + easy wins (#2030) - #2420
Merged
Merged
Conversation
Partial cleanup under parent #2200. Live main-source project -Xlint diagnostics 87→26 via real generics and mechanical easy wins. - Replace for-removal boxed Long/Double/Boolean constructors - Add serialVersionUID; annotate QueuedEdition/PublishEditionService @deprecated - Type servlet map/enumeration overrides, JAXB Class, List/Iterator consumers - SlotItemComparator.hashCode; unit tests for URL parse/mutable URL/comparator Refs #2200 Partial for #2030 > Co-Authored by Grok Build using grok-4.5 with agent main.
This was referenced Aug 8, 2026
Contributor
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (27 files)
Reviewed by step-3.7-flash · Input: 237.3K · Output: 32.6K · Cached: 3M Review guidance: REVIEW.md from base branch |
natechadwick
approved these changes
Aug 8, 2026
This was referenced Aug 9, 2026
vijaya-boddipudi
pushed a commit
that referenced
this pull request
Aug 11, 2026
Re-inventory after main-source zero (#2658/#2419): remaining project -Xlint diagnostics were all test-source raw/unchecked (+ one static-method site). Prefer real generics and typed Mockito matchers; no blanket @SuppressWarnings. Refs #2030 #2200 #2419 #2420 > Co-Authored by Grok Build using grok-4.5 with agent main.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Partial Xlint cleanup for perc-toolkit (issue #2030, parent tracker #2200). First PR-sized batch prefers real generics and mechanical easy wins over blanket suppressions.
Live main-source project
-Xlintdiagnostics: 87 → 26 (~61 cleared).Main changes
ImageEditorWizard,ImageResizeManagerImpl,PSOFileUploadValidation→ primitive casts /Boolean.valueOfArchivedException,UniqueIdLocatorSet,TrashTask.FatalTaskExceptionQueuedEditionclass +PublishEditionServicedeprecated API surface (javadoc-only → annotation)MutableHttpServletRequestWrapperMap/Enumerationoverrides match Jakarta APIPSOMutableUrl,PSOParseUrlQueryString,SimplifyParameters,PSOListTools, relationship/folder/extension rawIteratorconsumersItemRestServiceImpl/ItemServiceHelperuseJAXBContext.newInstance(Item.class)PSOSlotContents.SlotItemComparatorTests
PSOMutableUrlTest— toString / setParamPSOParseUrlQueryStringTest— single + multi-value accumulationPSOSlotContentsTest.testSlotItemComparatorEqualsHashCodeContractUniqueIdLocatorSetTest.testDefinesSerialVersionUidResidual
Remaining diagnostics tracked in #2419 (this-escape sites, PSORequestContext parent override cascade, PropertyData serial field).
Test plan
cd modules/perc-toolkit && ../../mvnw.cmd clean install— BUILD SUCCESSGates
docs/ai-generated/code-reviews/issue-2030-perc-toolkit-xlint-batch1-erlang.mdPartial for #2030
Refs #2200
Residual: #2419