Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(#346): remove redundant if statements #429

Merged

Conversation

volodya-lombrozo
Copy link
Member

@volodya-lombrozo volodya-lombrozo commented Aug 29, 2024

In this PR I remove an outdated puzzle and redundant if statements.
Closes: #346


PR-Codex overview

The focus of this PR is to remove an ad-hoc solution for storing array elements in StoreArray.

Detailed summary

  • Removed ad-hoc solution for array elements storage.
  • Simplified logic by unifying array opcodes handling.
  • Added res.addAll(this.array.opcodes()) for all cases.
  • Improved code readability and maintainability.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@volodya-lombrozo
Copy link
Member Author

@rultor merge

@volodya-lombrozo volodya-lombrozo merged commit ef4adb1 into objectionary:master Aug 29, 2024
11 checks passed
@rultor
Copy link
Contributor

rultor commented Aug 29, 2024

@rultor merge

@volodya-lombrozo The pull request is closed already, so I can't merge it

@rultor
Copy link
Contributor

rultor commented Aug 29, 2024

@rultor merge

@volodya-lombrozo I'm sorry, I don't understand you :( Check this page and try again please

@0crat
Copy link

0crat commented Aug 29, 2024

@volodya-lombrozo Thanks for your contribution! You've earned +4 points: +4 base, -16 for no code review, -4 for insufficient comments (1), -8 for low hits-of-code (14), +28 adjustment. Remember, code reviews and more substantial contributions (>20 hits-of-code) can significantly boost your points. Keep improving! Your balance: +176.

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.

StoreArray.java:39-44: Remove ad-hoc solution with...
3 participants