You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
After I made #266 and tested it, it passed all manual tests and also did perform normally. But after using it locally on real data it performs poorly, taking seconds to unescape everything in my history (I have a 500 entries history 😓 )
After investigating this issues, and reading into the source code of default_unescape_string I found out, that they unescape '' (two single quotes) to ' and tested the input of '' and copying that entry, it was unescaped to ' which is incorrect.
Also the performance problem lies into calling C functions is rather expensive, and doing it for ~500 entries takes a few seconds, even if the c function is fast, the transition costs to much in this case.
The best solution is to rewrite the exact needs of the unescape behavior ourselves in js, even if I wonder why this problem exists in the first case in gda.
I am working on fixing this 👍🏼
The text was updated successfully, but these errors were encountered:
Description
After I made #266 and tested it, it passed all manual tests and also did perform normally. But after using it locally on real data it performs poorly, taking seconds to unescape everything in my history (I have a 500 entries history 😓 )
After investigating this issues, and reading into the source code of
default_unescape_string
I found out, that they unescape''
(two single quotes) to'
and tested the input of''
and copying that entry, it was unescaped to'
which is incorrect.Also the performance problem lies into calling C functions is rather expensive, and doing it for ~500 entries takes a few seconds, even if the c function is fast, the transition costs to much in this case.
The best solution is to rewrite the exact needs of the unescape behavior ourselves in js, even if I wonder why this problem exists in the first case in gda.
I am working on fixing this 👍🏼
The text was updated successfully, but these errors were encountered: