Skip to content

Remove all super(); calls - #5009

Merged
kaikreuzer merged 1 commit into
openhab:mainfrom
dilyanpalauzov:remove_super_noarg
Sep 7, 2025
Merged

Remove all super(); calls#5009
kaikreuzer merged 1 commit into
openhab:mainfrom
dilyanpalauzov:remove_super_noarg

Conversation

@dilyanpalauzov

Copy link
Copy Markdown
Contributor

by calling:

sed -i "/super();/d" $(git grep -l "super();")

From The Java Tutorials > Using the Keyword super:

If a constructor does not explicitly invoke a superclass constructor, the Java compiler automatically inserts a call to the no-argument constructor of the superclass.

by calling:
  sed -i "/super();/d" $(git grep -l "super();")
@dilyanpalauzov
dilyanpalauzov requested a review from a team as a code owner September 4, 2025 11:37

@holgerfriedrich holgerfriedrich 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.

LGTM, thanks!

@holgerfriedrich holgerfriedrich 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.

Could you please sign off your commit? DCO check is failing. Thanks!

@kaikreuzer

Copy link
Copy Markdown
Member

@kaikreuzer
kaikreuzer merged commit 961de1a into openhab:main Sep 7, 2025
3 of 4 checks passed
@kaikreuzer kaikreuzer added this to the 5.1 milestone Sep 7, 2025
@dilyanpalauzov
dilyanpalauzov deleted the remove_super_noarg branch September 7, 2025 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants