Skip to content

JavaRedux: Integer methods + List::remove + NoSuchElementException - #3719

Merged
wadoon merged 4 commits into
KeYProject:mainfrom
FliegendeWurst:javaredux-additions
Aug 20, 2026
Merged

JavaRedux: Integer methods + List::remove + NoSuchElementException#3719
wadoon merged 4 commits into
KeYProject:mainfrom
FliegendeWurst:javaredux-additions

Conversation

@FliegendeWurst

@FliegendeWurst FliegendeWurst commented Jan 19, 2026

Copy link
Copy Markdown
Member

Intended Change

The Integer class is a bit useless without providing at least valueOf and intValue, so I added specifications for those methods. And the field value to store the int value.

The List::remove method to remove elements by index is useful to have.

The NoSuchElementException class is required to implement iterators.

Type of pull request

  • New feature (non-breaking change which adds functionality)
  • Other: change to JavaRedux (bootstrap classpath)

Ensuring quality

  • I have tested the feature as follows: used the newly specified methods in a simple example

Additional information and contact(s)

The contributions within this pull request are licensed under GPLv2 (only) for inclusion in KeY.

@FliegendeWurst FliegendeWurst changed the title JavaRedux: Integer methods + NoSuchElementException JavaRedux: Integer methods + List::remove + NoSuchElementException Jan 21, 2026

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

Thanks! I like the additions, when introducing value in Integer, perhaps be a bit more complete. ...

@mattulbrich mattulbrich added the Reviewer Feedback Feedback from the review needs to be addressed label Aug 16, 2026
@KeYProject KeYProject deleted a comment from codecov-commenter Aug 20, 2026
@KeYProject KeYProject deleted a comment from codecov Bot Aug 20, 2026
@wadoon
wadoon force-pushed the javaredux-additions branch from e26aadc to f847479 Compare August 20, 2026 14:58
@wadoon wadoon added this to the v3.1.0 milestone Aug 20, 2026
@wadoon wadoon self-assigned this Aug 20, 2026
@wadoon
wadoon enabled auto-merge August 20, 2026 15:00
@wadoon
wadoon added this pull request to the merge queue Aug 20, 2026
Merged via the queue into KeYProject:main with commit 27627aa Aug 20, 2026
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Reviewer Feedback Feedback from the review needs to be addressed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants