<div dir="ltr"><div>I think I made that warning icon way too big...<br></div><div>Also maybe showerror itself should give you a few options like "Temporarily ignore errors" and "View error log"<br></div><div><br></div>I've noticed today that when a script fails to import (corrupted .hs file) or there's a problem distributing a game, you can get spammed with 7 error messages in a row, most of them useless or misleading (and also often the initial error either wasn't detected, or wasn't usefully reported), and that's really not fun. So logging the errors and then telling the user to review them afterwards allays such cases of poor error handling too.<br></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 1 Oct 2018 at 02:14, Ralph Versteegen <<a href="mailto:teeemcee@gmail.com">teeemcee@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Viewing errors generated in-game in Custom? That's another idea. Of course it would pretty important for script errors but I don't see why viewing them in Game wouldn't be sufficient.<br><br></div>Anyway, mockup<br><div><br><div><img src="cid:ii_jmovz5q40" alt="errormockup.png" width="566" height="374"><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 1 Oct 2018 at 01:20, James Paige <<a href="mailto:Bob@hamsterrepublic.com" target="_blank">Bob@hamsterrepublic.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I really like the idea of making the error log viewable in the F9 menu. It could also just show the section of g_debug relevant to the current (or most recent) run of Test Game<div><br></div><div><br><br>On Sunday, September 30, 2018, Ralph Versteegen <<a href="mailto:teeemcee@gmail.com" target="_blank">teeemcee@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>(Part 1: just talking to myself)<br></div><div><br>Our error reporting is quite a mess, and I certainly made it far messier when I added different error levels.<br>This commit is just an exercise in hill-climbing: making a small improvement without thinking about the global optimum: whether the whole thing should actually be replaced.<br></div><div>I'm pretty sick of telling people to look in g/c_debug.txt when they have a problem - it's not just for us; those error messages are often useful for people too.<br></div><div>We usually just call debug to log an error. There are about 750 calls to debug which aren't commented out. Some are just debug noise, some are errors, some indicate corrupt data, some are engine bugs, some are fatal errors.<br></div><div>I hope that by adding (in particular) showbug and debugerror that these will get used more instead of debug.<br><br></div><div>I guess we don't want to pop up error messages unnecessarily while you're playing a game, just like we don't want to show script errors, if there's a possibility that the error can just be ignored.<br></div><div>I also don't like how showerror prompts the user to quit the program (once 3 errors have occurred), because you don't want to do that while editing a game unless you're really stuck.<br><br></div><div>(Part 2: some ideas)<br><br></div><div>So maybe a certain severity of error (I guess that's debugerror; debug is used too much) should be logged if it happens in-game, but pops up a message if it happens while editing (or in Test Game)?<br></div><div></div><div>But then there's a still a risk of it turning into spamming.<br></div><div><br>So, instead, maybe what we should do is that when debugerror is called, we display a warning icon in the corner of the screen for a few seconds with some text like "F9 to see error", and in Custom's F9 menu is an option "See error log" (probably excluding debuginfo).<br></div><div>Such an option in the F9 menu would be great either way.<br></div><div>The warning icon could be shown in-game too, because a lot of errors indicate data that couldn't be loaded and you'd likely want to know that as a player. If it just displays for a few seconds it doesn't interrupt the game a fraction as badly as calling showerror.<br></div><div></div><div><br></div><div>Also, I'd like to use CrashRpt to report whenever a bug is detected. Can send a minidump/stacktrace. We can do that on Linux/Mac too, just need extra code to upload to the server.<br></div><br></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 1 Oct 2018 at 00:39, <<a href="mailto:subversion@hamsterrepublic.com" target="_blank">subversion@hamsterrepublic.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">teeemcee<br>
2018-09-30 04:39:04 -0700 (Sun, 30 Sep 2018)<br>
543<br>
Add debugerror, showbug, fatalbug functions and rearrange ErrorLevelEnum levels<br>
<br>
There are now aliases for most debugc error levels:<br>
debuginfo<br>
debug<br>
visible_debug<br>
debugerror<br>
showerror<br>
showbug<br>
fatalerror<br>
fatalbug<br>
So I will mostly stop using debugc.<br>
<br>
Got rid of errShowError, renamed errShow to errShowError and errFatal to<br>
errFatalError, added errShowInfo and errShowDebug (the latter is<br>
visible_debug). Simplified visible_debug and dist_info.<br>
<br>
Also, forgot to update errorlevel.h and C/C++ code after renaming ErrorLevelEnum<br>
constants.<br>
---<br>
U wip/allmodex.bas<br>
U wip/array.c<br>
U wip/common.rbas<br>
U wip/common_base.bas<br>
U wip/<a href="http://common_base.bi" rel="noreferrer" target="_blank">common_base.bi</a><br>
U wip/<a href="http://const.bi" rel="noreferrer" target="_blank">const.bi</a><br>
U wip/distribmenu.bas<br>
U wip/errorlevel.h<br>
U wip/filelayer.cpp<br>
U wip/gfx_sdl2.bas<br>
U wip/misc.h<br>
U wip/surface.cpp<br>
_______________________________________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org" target="_blank">ohrrpgce@lists.motherhamster.org</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer" target="_blank">http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org</a><br>
</blockquote></div>
</blockquote></div>
_______________________________________________<br>
Ohrrpgce mailing list<br>
<a href="mailto:ohrrpgce@lists.motherhamster.org" target="_blank">ohrrpgce@lists.motherhamster.org</a><br>
<a href="http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org" rel="noreferrer" target="_blank">http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org</a><br>
</blockquote></div>
</blockquote></div>