Skip to content

Adds the For Loop#64

Merged
MarkusEble merged 10 commits intoMarkusEble:mainfrom
Sebastian-Heinzenburger:main
Jun 7, 2025
Merged

Adds the For Loop#64
MarkusEble merged 10 commits intoMarkusEble:mainfrom
Sebastian-Heinzenburger:main

Conversation

@Sebastian-Heinzenburger
Copy link
Contributor

No description provided.

run: mvn -B package --file pom.xml

- name: Test with Maven
run: mvn -B test --file pom.xml No newline at end of file
Copy link
Owner

Choose a reason for hiding this comment

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

ist diese Änderung Absicht?

String input = new String("""
{
PRINT 4 + 3 - 2;
DECLARE i;
Copy link
Owner

Choose a reason for hiding this comment

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

Änderungen an InterpreterMain bitte aus dem Change rausnehmen, sonst drohen Mergekonflikte


@Override
public void codegen(CompileEnvIntf env) {
InstrBlock initBlock = env.createBlock("for_init");
Copy link
Owner

Choose a reason for hiding this comment

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

sieht gut aus
der for_init block und der for_update block sind nicht notwendig, der Content kann auch in den Vorgängerblock geschrieben werden (kann man aber auch so lassen, wenn es euch besser gefällt)

@MarkusEble
Copy link
Owner

der ConstFoldUnaryExprTest geht bei mir lokal durch

@Sebastian-Heinzenburger Sebastian-Heinzenburger marked this pull request as draft May 23, 2025 13:10
@Sebastian-Heinzenburger
Copy link
Contributor Author

Die PullRequest ist noch nicht fertig. Ich hatte vergessen sie als Draft zu kennzeichnen 😅

@Sebastian-Heinzenburger
Copy link
Contributor Author

In dieser PR wird auch der InterpreterDumpTestBase.java überarbeitet, sodass dieser in testInterpreter eine RegEx als expected output akzeptiert. Das wird dann für den ConstFoldUnaryExprTest genutzt. Dieser Schlug beim Ausführen der Pipeline im laufe der Entwicklung immer mal wieder (je nach Testreihenfolge und aktuellem Register-Counter) fehl.

@Sebastian-Heinzenburger Sebastian-Heinzenburger marked this pull request as ready for review June 5, 2025 03:05
Sebastian Heinzenburger added 2 commits June 5, 2025 05:12
…naryExprTest as they are not needed anymore after 16a422a
…naryExprTest as they are not needed anymore after 16a422a
@Sebastian-Heinzenburger
Copy link
Contributor Author

Die im vorherigen Kommentar beschriebenen Änderungen sind nach 16a422a nicht mehr benötigt. Also wurden sie wieder rückgängig gemacht. Sollte jetzt passen (on my machine)

@MarkusEble MarkusEble merged commit 7529f65 into MarkusEble:main Jun 7, 2025
1 check passed
@MarkusEble
Copy link
Owner

Danke, passt.

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.

2 participants