Mysticial / y-cruncher

Bug-Tracking and open-sourced parts of y-cruncher.
189 stars 16 forks source link

Errors in computation may ignore the `pause:-1` command. #11

Open Mysticial opened 6 years ago

Mysticial commented 6 years ago

When the program errors out, it will sometimes ignore the pause:-1 option and pause anyway waiting for the user to press ENTER.

Initially this was by-design. As the user will usually want to know about errors. But I've already had 2 requests to suppress the pause even in the event of an error.

The correct solution is to dump the errors to a log file and exit with the appropriate error code so it can be picked up by scripts.


Internally, the error/exception handling is a mess. Half of the program properly uses C++ exceptions. The other half does an "error-and-exit".

Mysticial commented 6 years ago

Version 0.7.6.9484 adds a pause:-2 which should force even errors to not pause. I haven't had the time to really test it yet.

Logging of errors to a file is non-trivial and will be deferred to a future release.

Mysticial commented 5 years ago

No longer a bug as it's been patched. But lot of internal cleanup needed internally to make this right.