JSON decoder errors aren't properly logged

[Vadim Peretokin]

I saw this in the errors view:

[ERROR:] object:<JSON decoder error:> function:<json_to_value>
         <>

With no JSON error attached. Looking at the code:

            string e;
            if( lua_isstring( L, 1 ) )
            {
                e = "Lua error:";
                e += lua_tostring( L, 1 );
            }
            QString _n = "JSON decoder error:";
            QString _f = "json_to_value";
            logError( e, _n, _f );

It should be using -1 as the index for the top of the stack, because 1 might not be the top (which seems to be the case in my error).

[Vadim Peretokin (vperetokin)]

That is to say, the fixed code should be:

            string e;
            if( lua_isstring( L, -1 ) )
            {
                e = "Lua error:";
                e += lua_tostring( L, -1 );
            }
            QString _n = "JSON decoder error:";
            QString _f = "json_to_value";
            logError( e, _n, _f );

[Chris (chrismudlet)]

https://github.com/Chris7/Mudlet2/commit/797265e243b0bf7c72ab040a808026d0cfaea06a

[Chris (chrismudlet)]

in Mudlet/development