Re: Reworked qt terminal is much faster – and fails to work on Mac

[Mojca M. <moj...@gm...>]

On Mon, Feb 10, 2014 at 1:57 AM, sfeam wrote:
> On Sunday, 09 February 2014 12:29:34 PM Mojca Miklavec wrote:
...
>> Total number in stack (recursive counted multiple, when >=5):
>>
>> Sort by top of stack, same collapsed (when >= 5):
>>         __select  (in libsystem_kernel.dylib)        2510
>>         kevent  (in libsystem_kernel.dylib)        2510
>>         QIODevice::bytesAvailable() const  (in QtCore)        941
>>         qt_sendFont()  (in gnuplot)        379
>>         QAbstractSocket::bytesAvailable() const  (in QtNetwork)        248
>>         QLocalSocket::bytesAvailable() const  (in QtNetwork)        236
>>         QLocalSocket::waitForReadyRead(int)  (in QtNetwork)        195
>>         DYLD-STUB$$QIODevice::bytesAvailable() const  (in QtNetwork)        157
>>         QLocalSocket::state() const  (in QtNetwork)        147
>>         DYLD-STUB$$QLocalSocket::waitForReadyRead(int)  (in gnuplot)        110
>>         DYLD-STUB$$QLocalSocket::bytesAvailable() const  (in gnuplot)        97
>>
>> Mojca
>
> Finally a trace that makes sense in that it involves a recent change.
> This bit belongs to the patch:
>
> 2014-01-26  Jérôme Lodewyck  <lod...@us...>
>
>         * src/qtterminal/qt_term.cpp: Implement font metric caching. This solves
>         a flickering issue when a large number of font changes are called (for
>         example when rotating the world plot in world2.dem).
>
>         * src/qtterminal/QtGnuplotInstance.*: New public function that sends a
>         command to gnuplot and blocks until it receives the answer.
>
> Can you back out just that one set of changes from current CVS and see
> if that makes your qt terminal work?

I don't know if it makes much sense to debug infinite loops when it's
clear that the loops happened when gnuplot wasn't even connected
properly/initialized at startup (the code was executed with badly
initialized gnuplot under assumption that connection was successful).

But if I revert that patch alone, the first plot still doesn't work properly.


However if I revert the following commit from
    2014-01-29 Jérôme Lodewyck Postpone show() in QtGnuplotWindow
then the first plot displays just fine and then basically everything
works as expected.

Mojca