Skip to content

Clean Code for debug/org.eclipse.debug.ui#2693

Merged
laeubi merged 1 commit into
masterfrom
clean-code/debug/org.eclipse.debug.ui
Jun 3, 2026
Merged

Clean Code for debug/org.eclipse.debug.ui#2693
laeubi merged 1 commit into
masterfrom
clean-code/debug/org.eclipse.debug.ui

Conversation

@eclipse-platform-bot
Copy link
Copy Markdown
Contributor

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
  • Remove unused private constructors
  • Remove unused private fields
  • Remove unused private methods
  • Remove unused private types
  • 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

org.eclipse.core.variables,
org.eclipse.debug.core,
org.eclipse.debug.core.model,
org.eclipse.debug.internal.ui.launchConfigurations,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Seems debug.ui is leaking internal API...

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

Test Results

    54 files  ±0      54 suites  ±0   34m 25s ⏱️ - 1m 23s
 4 667 tests ±0   4 645 ✅ ±0   22 💤 ±0  0 ❌ ±0 
11 895 runs  ±0  11 742 ✅ ±0  153 💤 ±0  0 ❌ ±0 

Results for commit a1b293b. ± Comparison against base commit a569c2c.

@laeubi laeubi merged commit 2804d09 into master Jun 3, 2026
18 checks passed
@laeubi laeubi deleted the clean-code/debug/org.eclipse.debug.ui branch June 3, 2026 04:48
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