Skip to content

Polishing#30267

Closed
ZouHongxue wants to merge 1 commit intospring-projects:mainfrom
ZouHongxue:main
Closed

Polishing#30267
ZouHongxue wants to merge 1 commit intospring-projects:mainfrom
ZouHongxue:main

Conversation

@ZouHongxue
Copy link
Copy Markdown

Base on SonarLint and IDEA suggestions, polishing the code.

  • Reorder the modifiers to comply with the Java Language Specification.
  • java:S4276 Refactor this code to use the more specialised Functional Interface 'UnaryOperator'
  • Merge this if statement with the enclosing one.
  • Use record for immutable classes.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Apr 3, 2023
@ZouHongxue
Copy link
Copy Markdown
Author

Rebase the commit, refer #30250

@sdeleuze sdeleuze self-assigned this Apr 3, 2023
@sdeleuze sdeleuze added in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Apr 3, 2023
@sdeleuze sdeleuze added this to the 6.0.8 milestone Apr 3, 2023
sdeleuze added a commit to sdeleuze/spring-framework that referenced this pull request Apr 3, 2023
@sdeleuze sdeleuze closed this in 534d1cd Apr 3, 2023
@sdeleuze
Copy link
Copy Markdown
Contributor

sdeleuze commented Apr 3, 2023

Thanks for your contribution, I merged a subset of those changes, without the one changing the public APIs and with the main goal of fixing HttpComponentsHeadersAdapter null-safety.

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

Labels

in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants