Andy Nichols authored 1 year ago and Andy Nichols committed 1 year ago

This crash happened when the lite-xl tried to access a path that didn't
exist.  We create the path at QStandardPaths::AppDataLocation at startup
(it is where we load all the templates), but not
QStandardPaths::AppCacheLocation which is where we told the lite-xl to
look for the session.lua file. Normally if the file doesn't exist, it
will create a new one, but if the path doesn't exist it throws and
exception (which translates as a crash in our case). So I fixed it to
use the same folder we already know exists and where all the rest of the
app data is.

Andy Nichols authored 1 year ago and Andy Nichols committed 1 year ago

Here we really reduce down how many times we step the lua engine. This
should further reduce the CPU usage esspecially during idle. Basically
what is being down is that we only step the lua thread when there are
new events, or we need a repaint, and for 2 seconds after any user
interaction. We also wake up once per second to process background
processes when nothing else is occuring.

Further refinement may be necessary, but CPU usage should finally be
reasonable.

Andy Nichols authored 1 year ago and Andy Nichols committed 1 year ago

If building with Qt 6.7+ it is possible to render text with QSGTextNode
so this changes CodeItem to be a regular QQuickItem instead of a
QQuickPaintedItem.  The code is #ifdef'ed to handle both cases still
though.

This will have limitations due to OS specific limits on how many files can
be monitored, but it is what it is.

Switch which QThread pattern is used (no more run())