Richard Heck wrote:
LyXFunc.cpp: In function 'void lyx::<unnamed>::actOnUpdatedPrefs(const lyx::LyXRC&, const lyx::LyXRC&)': LyXFunc.cpp:1885: warning: enumeration value 'RC_FULL_SCREEN_LIMIT' not handled in switch LyXFunc.cpp:1885: warning: enumeration value 'RC_FULL_SCREEN_SCROLLBAR' not handled in switch LyXFunc.cpp:1885: warning: enumeration value 'RC_FULL_SCREEN_TABBAR' not handled in switch LyXFunc.cpp:1885: warning: enumeration value 'RC_FULL_SCREEN_TOOLBARS' not handled in switch LyXFunc.cpp:1885: warning: enumeration value 'RC_FULL_SCREEN_WIDTH' not handled in switch LyXFunc.cpp:1885: warning: enumeration value 'RC_VISUAL_CURSOR' not handled in switch
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...
Dov
