Skip to content

Clean Code for bundles/org.eclipse.osgi.services#63

Closed
laeubi wants to merge 1 commit into
masterfrom
clean-code/bundles/org.eclipse.osgi.services
Closed

Clean Code for bundles/org.eclipse.osgi.services#63
laeubi wants to merge 1 commit into
masterfrom
clean-code/bundles/org.eclipse.osgi.services

Conversation

@laeubi
Copy link
Copy Markdown
Owner

@laeubi laeubi commented Jul 9, 2025

The following cleanups were applied:

  • Add final modifier to private fields
  • Add missing '@Deprecated' annotations
  • Add missing '@Override' annotations
  • Add missing '@Override' annotations to implementations of interface methods
  • Convert control statement bodies to block
  • Make inner classes static where possible
  • Remove trailing white spaces on all lines
  • Remove unnecessary array creation for varargs
  • Remove unnecessary suppress warning tokens
  • Remove unused imports
  • Replace deprecated calls with inlined content where possible
  • Use pattern matching for instanceof

The following Manifest cleanups where applied:

  • Calculate 'uses' directive for public packages
  • Remove unused dependencies

@laeubi laeubi force-pushed the clean-code/bundles/org.eclipse.osgi.services branch from 09c8ed5 to 7dd2128 Compare July 9, 2025 13:12
@laeubi laeubi added the cleanup label Jul 9, 2025
@laeubi
Copy link
Copy Markdown
Owner Author

laeubi commented Jul 9, 2025

@laeubi laeubi force-pushed the clean-code/bundles/org.eclipse.osgi.services branch 2 times, most recently from 6624086 to 7e90762 Compare July 10, 2025 04:55
@laeubi laeubi closed this Jul 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants