forked from StarOpenSource/Engine
Allow throwing all exceptions during engine init
Due to some Java limitation we are required to throw a RuntimeException with our original exception contained within and afterwards unwrap it so we can throw any exception we like. That is dumb.
This commit is contained in:
parent
a21ef1bc5f
commit
d3a01c2c66
1 changed files with 37 additions and 31 deletions
|
@ -185,36 +185,40 @@ public final class Engine extends SubsystemClass {
|
|||
* @throws IllegalStateException when running in an incompatible environment
|
||||
* @since v1-alpha6
|
||||
*/
|
||||
private Engine() throws IllegalStateException {
|
||||
private Engine() throws RuntimeException {
|
||||
long initTime = Miscellaneous.measureExecutionTime(() -> {
|
||||
instance = this;
|
||||
state = EngineState.EARLY_STARTUP;
|
||||
try {
|
||||
instance = this;
|
||||
state = EngineState.EARLY_STARTUP;
|
||||
|
||||
new EngineConfiguration();
|
||||
EngineConfiguration.getInstance().loadConfiguration();
|
||||
new EngineConfiguration();
|
||||
EngineConfiguration.getInstance().loadConfiguration();
|
||||
|
||||
LOGGER.info("Initializing engine");
|
||||
initializeClasses(); // Initialize classes
|
||||
if (checkEnvironment()) // Check environment
|
||||
throw new IllegalStateException("Running in an incompatible environment");
|
||||
ensureEnvironment(); // Prepare the environment and ensure safety
|
||||
populateCrashContent(); // Populate crash content
|
||||
cacheEvents(); // Cache event listeners
|
||||
startThreads(); // Start threads
|
||||
LOGGER.info("Initializing engine");
|
||||
initializeClasses(); // Initialize classes
|
||||
if (checkEnvironment()) // Check environment
|
||||
throw new IllegalStateException("Running in an incompatible environment");
|
||||
ensureEnvironment(); // Prepare the environment and ensure safety
|
||||
populateCrashContent(); // Populate crash content
|
||||
cacheEvents(); // Cache event listeners
|
||||
startThreads(); // Start threads
|
||||
|
||||
LOGGER.verb("Completing early initialization stage");
|
||||
state = EngineState.STARTUP;
|
||||
LOGGER.verb("Completing early initialization stage");
|
||||
state = EngineState.STARTUP;
|
||||
|
||||
// Perform automatic subsystem initialization
|
||||
if (EngineConfiguration.getInstance().isInitialPerformSubsystemInitialization()) {
|
||||
collectSubsystems(); // Collect subsystems
|
||||
// Perform automatic subsystem initialization
|
||||
if (EngineConfiguration.getInstance().isInitialPerformSubsystemInitialization()) {
|
||||
collectSubsystems(); // Collect subsystems
|
||||
|
||||
// Initialize subsystems
|
||||
try {
|
||||
initializeSubsystems();
|
||||
} catch (Exception exception) {
|
||||
LOGGER.error("Subsystem dependency resolution failed");
|
||||
// Initialize subsystems
|
||||
try {
|
||||
initializeSubsystems();
|
||||
} catch (Exception exception) {
|
||||
LOGGER.error("Subsystem dependency resolution failed");
|
||||
}
|
||||
}
|
||||
} catch (Exception exception) {
|
||||
throw new RuntimeException(exception);
|
||||
}
|
||||
});
|
||||
|
||||
|
@ -235,14 +239,16 @@ public final class Engine extends SubsystemClass {
|
|||
try {
|
||||
if (instance == null)
|
||||
new Engine();
|
||||
} catch (IllegalStateException exception) {
|
||||
throw exception;
|
||||
} catch (Exception exception) {
|
||||
LOGGER.error("Engine initialization failed");
|
||||
LOGGER.error(Miscellaneous.getStackTraceHeader(exception));
|
||||
for (String line : Miscellaneous.getStackTraceAsString(exception, true).split("\n"))
|
||||
LOGGER.error(line);
|
||||
throw new RuntimeException("Engine initialization failed", exception);
|
||||
} catch (RuntimeException exception) {
|
||||
if (exception.getCause() instanceof IllegalStateException)
|
||||
throw (IllegalStateException) exception.getCause();
|
||||
else {
|
||||
LOGGER.error("Engine initialization failed");
|
||||
LOGGER.error(Miscellaneous.getStackTraceHeader(exception.getCause()));
|
||||
for (String line : Miscellaneous.getStackTraceAsString(exception.getCause(), true).split("\n"))
|
||||
LOGGER.error(line);
|
||||
throw new RuntimeException("Engine initialization failed", exception.getCause());
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue