[Freemint-list] TosWin2 fixes

Miro Kropáček miro.kropacek at gmail.com
Mon May 8 15:27:35 MSD 2017


On 8 May 2017 at 05:19, Vincent Rivière <vincent.riviere at freesbee.fr> wrote:

> It only works if I manually open the console once after startup.
>
OK, your test case was a bit imprecise -- one has to set the option, save
settings, quit TW2 and then start it. It was a double bug and thanks to
this investigation I discovered a regression for which I can thank Helmut ;)

On 13 January 2017 at 09:20, Helmut Karlowski <helmut.karlowski at ish.de>
 wrote:

> I flew over it and wondered about:
>
> @@ -726,6 +727,14 @@ bool config_load(void)
>            buffer[strlen(buffer) - 1] = '\0';
>          parse_line(buffer);
>        }
> +
> +     if (gl_con_log)
> +     {
> +       bool on = gl_con_log;
> +       gl_con_log = FALSE;
> +       gl_con_log = log_console(on);
> +     }
> +
>
> Why do you set gl_con_log twice?


Of course this wasn't without meaning, the code is so ugly so using global
variables on different places (incl. log_console()) sometimes expects
certain state of things. Shouldn't have blindly trust him and keep it since
logging had worked before that.

Anyways, it's all fixed now, grab & test from the same place. Hopefully
it's final this time.

-- 
MiKRO / Mystic Bytes
http://mikro.atari.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.atariforge.org/pipermail/freemint-list/attachments/20170508/2cb96ed5/attachment.html 


More information about the Freemint-list mailing list