Skip to content

Conversation

@cagix
Copy link
Member

@cagix cagix commented Dec 4, 2025

Copy link
Member Author

@cagix cagix left a comment

Choose a reason for hiding this comment

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

lgtm

@cagix cagix merged commit 5042778 into master Dec 6, 2025
3 checks passed
@cagix cagix deleted the simple_b08 branch December 6, 2025 13:58
cagix added a commit to Compiler-CampusMinden/CB-Vorlesung-Bachelor that referenced this pull request Dec 7, 2025
@bcg7 Bitte schau mal drüber, ich habe versucht, den Umfang nochmal
deutlich zu drücken.

Insbesondere habe ich gestrichen:

- keine funktionsdeklarationen
- damit keine unbenannten parameter mehr
- keine globalen variablen (nur lokal)
- kein *this - wir erlauben einfach kein shadowing in den klassen, also
keine variablen/parameter, die zu feldern gleichnamig sind
- keine arrays

Damit man aber gleichzeitig noch "interessante" Programme interpretieren
kann, habe ich noch den eingebauten Datentyp "string" hinzugefügt und
die passende eingebaute Funktion für die Ausgabe.


tests siehe
Compiler-CampusMinden/student-support-code-template#9
github-actions bot pushed a commit to Compiler-CampusMinden/CB-Vorlesung-Bachelor that referenced this pull request Dec 7, 2025
@bcg7 Bitte schau mal drüber, ich habe versucht, den Umfang nochmal
deutlich zu drücken.

Insbesondere habe ich gestrichen:

- keine funktionsdeklarationen
- damit keine unbenannten parameter mehr
- keine globalen variablen (nur lokal)
- kein *this - wir erlauben einfach kein shadowing in den klassen, also
keine variablen/parameter, die zu feldern gleichnamig sind
- keine arrays

Damit man aber gleichzeitig noch "interessante" Programme interpretieren
kann, habe ich noch den eingebauten Datentyp "string" hinzugefügt und
die passende eingebaute Funktion für die Ausgabe.

tests siehe
Compiler-CampusMinden/student-support-code-template#9 e698b13
@bcg7
Copy link

bcg7 commented Dec 7, 2025

Ich habe keinen Test gefunden für den %-Operator auf int.
In operators.cpp ist für den Datentyü char nur ein Test auf < drin.
In func.cpp solle bar() hinter main stehen. Aber woltest du die Vorwärtdeklarationen nicht sowieso rauslassen?

@cagix
Copy link
Member Author

cagix commented Dec 7, 2025

Ich habe keinen Test gefunden für den %-Operator auf int. In operators.cpp ist für den Datentyü char nur ein Test auf < drin. In func.cpp solle bar() hinter main stehen. Aber woltest du die Vorwärtdeklarationen nicht sowieso rauslassen?

danke. die funktion ist bereits vorher definiert, aber der kommentar war noch kaputt. habe ich grad behoben.

die tests für die operatoren sind nicht vollständig. ich hatte exemplarisch ein paar dinge aufgeriffen, und dann noch vorrang. meinst du, ich sollte das vollständig abdecken?

Copy link

@bcg7 bcg7 left a comment

Choose a reason for hiding this comment

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

ok.

@bcg7
Copy link

bcg7 commented Dec 7, 2025

Die Implementierungen unterscheiden sich ja nicht.

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