On 09/08/2018 11:20, Emanuele wrote:
>>
> Why this? Shouldn't it be:
>
> if (g_strcmp0(old_path, path)) {
> qtest_end(); /* handles global_qtest = NULL */
> g_free(old_path); /* handles NULL */
> old_path = path;
> global_qtest = qtest_start(path);
> } else ....Yes, of course. Though I'd have thought that my version has "just" a memory leak. Even better, this could call qos_invalidate_command_line. Can you post the fixes to a separate commit on github? Paolo
