save_executable doesn't do anything with a bad return from obj_run_linker
obj_run_linker
returns -1 if something bad happened. save_executable
accepts the return value but does nothing with it. We should at least print a message if obj_run_linker
returns -1. As it is now, we just print "Done" independent of what obj_run_linker
returns. We should at least tell the user that we failed.
(Side note: This is super-annoying that we always exit if we fail to create the executable. So everything on the heap is lost and you have to start all over to create the executable.)