Dov Feldstern <[EMAIL PROTECTED]> writes: > Hi! > > For the past few months I've been working on visual mode for bidi > cursor movement. After a few false starts, I finally feel that I've > got an implementation which is relatively clean. It's not totally > finished yet, but it is well enough along to submit for comments. > > Attached are three patches: > > 1. Adding a preference for visual cursor, which will later be used for > determining how the arrow keys should behave. Nothing very > sophisticated here, but I have a couple of questions: > > a. I'm getting the warning > > LyXFunc.cpp:2162: warning: enumeration value 'RC_VISUAL_CURSOR' not > handled in switch > > and I see the nice comment at the top explaining why we use a > switch. But why is there anything at all? Why do *any* elements need > to be handled? Isn't the function doing absolutely nothing? What are > all the if statements in there about --- they'll never be reached as > far as I can see? What am I missing, and what should I do about my new > item? (BTW, the option seems to be working just fine as-is)
In LyXRC a similar trick is used, but it makes sense because RC_LAST is in the first position. Here I see it in the last position, which looks pretty strange. JMarc
