search

Drakulix / zwreec

Twee to Z-Code Compiler in Rust
BSD 2-Clause "Simplified" License
7 stars 3 forks source link

Feature/remove output #270

Closed Drakulix closed 9 years ago

Drakulix commented 9 years ago

Um einen Punkt in #242 zu schließen wird nun die Output Datei gelöscht wenn ein Panic auftritt.

Bitte um Feedback, keine Output Datei zu erzeugen wird mit parse_output nix, eventuell sollte auch die Datei nicht gelöscht werden, wenn -f gesetzt ist? (was meint ihr?)

pothos commented 9 years ago

Wurde sie denn schon unbrauchbar gemacht, bevor sie gelöscht wird oder kann passieren, dass eine vorherige, intakte Datei gelöscht wird?

Drakulix commented 9 years ago

Hmm, gute Nachfrage. Also ich denke der einzige Panic der ein Problem sein könnte, wäre wenn die Input fehlt, dann wird der Output gar nicht gecheckt aber der Pfad könnte gültig sein. In jedem anderen Fall denke ich wäre die Datei schon geöffnet und damit ruiniert. Ich guck mal, ob ich das hinkriege noch zu fixen.

Drakulix commented 9 years ago

Hab nachgebessert. Bitte reviewn

Drakulix commented 9 years ago

Mit dem letzten Commit sollten die Punkte 2 und 3 aus #242 so gut wie mit unserer Implementierung möglich jetzt behandelt sein.

felinira commented 9 years ago

Habs nur mal manuell gemerged, um zu schauen dass es wirklich keine Probleme gibt. Kann so rein. +1

msaehn commented 9 years ago

find ich gut => merge