-
Notifications
You must be signed in to change notification settings - Fork 145
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
fix(#2435): tests that uses memory #2450
Merged
rultor
merged 3 commits into
objectionary:master
from
maxonfjvipon:fix/#2435/tests-that-uses-memory
Aug 29, 2023
Merged
fix(#2435): tests that uses memory #2450
rultor
merged 3 commits into
objectionary:master
from
maxonfjvipon:fix/#2435/tests-that-uses-memory
Aug 29, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
@Graur please have a look |
Graur
suggested changes
Aug 29, 2023
@Graur please have a look one more time |
Graur
approved these changes
Aug 29, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@maxonfjvipon Good one!
@yegor256 Please have a look
@yegor256 please have a look |
@rultor merge |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes: #2435
PR-Codex overview
This PR focuses on fixing memory-related issues in the code.
Detailed summary
.as-int
to^.malloc.next
inheap.eo
tuple-tests.eo
EOmemoryTest.java
assert-that > res
toassert-that > @
inheap-tests.eo
nop > @
inmallocs-do-not-overlap
test inheap-tests.eo
assert-that > res
toassert-that > @
inmalloc-return-error
test inheap-tests.eo
nop > @
inwrite-and-read-without-error
test inheap-tests.eo
assert-that > res
initerates-over-simple-counter
test inbool-tests.eo
^.^.^ > x
tox.write
initerates-over-simple-counter
test inbool-tests.eo
nop > @
initerates-over-simple-counter
test inbool-tests.eo
assert-that > res
incomplex-bool-expression-in-while
test inbool-tests.eo
m.write (m.as-int.minus 1)
tom.write
andm.as-int.minus 1
incomplex-bool-expression-in-while
test inbool-tests.eo
nop > @
incomplex-bool-expression-in-while
test inbool-tests.eo
@Disabled
annotation inloopsOverAbstractObjects
test inEOboolEOwhileTest.java
new PhCopy(new PhMethod(toggle, "while"))
totoggle.attr("as-bool").get().attr("while").get().copy()
inloopsOverAbstractObjects
test inEOboolEOwhileTest.java
@Disabled
annotation indataizesComplexBooleanToggle
test inEOboolEOwhileTest.java
final Phi parent = new Parent(Phi.Φ);
tofinal Phi toggle = new PhMethod(new Parent(Phi.Φ), "toggle");
indataizesComplexBooleanToggle
test inEOboolEOwhileTest.java