Re: Keytest: One Screenshot per Key Press (and new bugs found)

[Abdelrazak Younes]

Jürgen Spitzmüller wrote:

Abdelrazak Younes wrote:

This looks safer:

+       if (!view() || view()->buffer()->isReadonly() ||
view()->cursor().inMathed())

The context of the condition is

       if (!buf)
            [...]
       [...]
       else if (name == "symbols") {
           if (buf->isReadonly() || !view() || view()->cursor().inMathed())
                enable = false;
           else {
                InsetCode ic = view()->cursor().inset().lyxCode();

So we are sure buf is valid, no?

It seems to indeed. But then the test for (!view()) is puperfluous...

Abdel.