The application closes immediately if mkxp.json is malformed

[Eblo]

If mkxp-z reads in an mkxp.json file with malformed JSON, it will close immediately with no information reported whatsoever. It would be preferable to handle this gracefully and report the error to the user in some way.

[Splendide-Imaginarius]

Hmm, so I think we already do this?

mkxp-z/src/config.cpp

Lines 111 to 116 in 52c932b

	catch (const std::exception &e) {
	Debug() << "Failed to parse" << path << ":" << e.what();
	}
	catch (const Exception &e) {
	Debug() << "Failed to parse" << path << ":" << "Unknown encoding";
	}

I wouldn't mind an error popup rather than the stderr logging that we do now though.

[Eblo]

It looks like cmd in Windows just doesn't care to report an application's output. Running the game from MSYS2 does in fact produce that output. I would like to see more detail in the error, at the very least the line number of the problem encountered.

[Splendide-Imaginarius]

It looks like cmd in Windows just doesn't care to report an application's output. Running the game from MSYS2 does in fact produce that output. I would like to see more detail in the error, at the very least the line number of the problem encountered.

I think cmd is sort-of deprecated in Windows anyway in favor of PowerShell, and I think PowerShell does display that output. That said, expecting people to run mkxp-z from a command-line of any kind just to see why it's exiting as soon as it starts seems unreasonable, so yeah, let's do an error popup.