Pavel Sanda wrote:
I purposely left RC_VISUAL_CURSOR out of this function, because something
is fishy about it. AFAICT, this function doesn't do anything... JMarc took
a look and I think he also wasn't sure about it... So before just fixing
the warnings, we ought to try and figure out what this function *should* be
doing, and whether it's necessary at all. I'm pretty sure I went quite far
back in history, and it hasn't been doing anything for ages...
but there is some code for certain cases, no?
Yes, but that code is never executed, which is even more disturbing.
Notice that the just before the switch, tag is set to RC_LAST, which is
the *last* case in the switch...
pavel
