Better displaying of errors/warning
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Armagetron Advanced |
New
|
Wishlist
|
Unassigned |
Bug Description
Replace the current
#define tERR_ERROR_
#define tERR_ERROR(mess) { std::ostringstream s; s << "Error in " << __PRETTY_FUNCTION__ << " in " << __FILE__<< ':' << __LINE__ << " : \n \t" << mess << '\n'; st_PresentError( "Error", s.str().c_str() ); }
#define tERR_MESSAGE(mess) { std::ostringstream s; s << "Message from " << __PRETTY_FUNCTION__ << " in " << __FILE__<< ':' << __LINE__ << " : \n \t" << mess << '\n'; st_PresentMessage( "Message", s.str().c_str() ); }
#define tERR_WARN(mess) {con << "Warning in " << __PRETTY_FUNCTION__ << " in " << __FILE__<< ':' << __LINE__ << " : \n \t" << mess << '\n';}
for something prettier. I tried to do something along https:/
1. It shouldnt be displayed either on con or cerr, the message should be displayed in both places.
2. Captions are not really necessary, it ends up showing "Warning: warning in" or "Error: error in".
3. Use the DEBUG flag to show the __PRETTY_
4. Move the formatting to tError.cpp instead.
5. Add fatal error or include a "kill" flag to tERR_ERROR[_INT] as it is hardcoded to dont die (tError.
6. src/tools/
Changed in armagetronad: | |
importance: | Undecided → Wishlist |