Dumb down the token cursor some dealing with meta data and readers

[PEZ]

With Calva v2.0.247 we introduced some changes to the token cursor, making it meta-data-”aware”.

It has caused two critical bugs with structural editing hanging on unbalanced code:

• Version 2.0.247 regression with structural editing, hangs at unbalance + structural delete #1573
• Structural editing hangs in specific cases of unbalanced forms #1585

They are now fixed. Fixing #1585 I choose to dumb down the token cursor operation backwardUpList which was placing the cursor in front of any metadata. The problem is that backwardUpList is being used from many places and in some unbalance cases this caused an infinite loop. Now this placing of the final cursor is done by the Paredit backwardUpList command instead, which was where it was needed. Now the token cursor function can safely be used again.

There are more places where this metadata-awareness was added to the token cursor, and for many of them we should do the same. Dumb them back down again, and do the adjustments at the call sites needing it, or make special versions of the functions.