Skip to content

Commit 699d42e

Browse files
committedMar 17, 2015
Revert "Server::step throw is never catched. Replace it with an errorstream + assert"
This reverts commit 5f8e48c.
1 parent ceac41e commit 699d42e

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed
 

Diff for: ‎src/server.cpp

+3-6
Original file line numberDiff line numberDiff line change
@@ -479,13 +479,10 @@ void Server::step(float dtime)
479479
JMutexAutoLock lock(m_step_dtime_mutex);
480480
m_step_dtime += dtime;
481481
}
482-
// Assert if fatal error occurred in thread
482+
// Throw if fatal error occurred in thread
483483
std::string async_err = m_async_fatal_error.get();
484-
if(async_err != "") {
485-
errorstream << "UNRECOVERABLE error occurred. Stopping server. "
486-
<< "Please fix the following error:" << std::endl
487-
<< async_err << std::endl;
488-
assert(false);
484+
if(async_err != ""){
485+
throw ServerError(async_err);
489486
}
490487
}
491488

1 commit comments

Comments
 (1)

nerzhul commented on Mar 17, 2015

@nerzhul
Contributor

Okay for the client side problem. But you didn't fix the server side problem. The server crashes because we throw in the nether.

Please sign in to comment.