Changeset 8351 for code/trunk/src/external/ois/linux/LinuxKeyboard.cpp

Timestamp:

Apr 28, 2011, 7:15:14 AM (14 years ago)

Author:

rgrieder

Message:

Merged kicklib2 branch back to trunk (includes former branches ois_update, mac_osx and kicklib).

Notes for updating

Linux:
You don't need an extra package for CEGUILua and Tolua, it's already shipped with CEGUI.
However you do need to make sure that the OgreRenderer is installed too with CEGUI 0.7 (may be a separate package).
Also, Orxonox now recognises if you install the CgProgramManager (a separate package available on newer Ubuntu on Debian systems).

Windows:
Download the new dependency packages versioned 6.0 and use these. If you have problems with that or if you don't like the in game console problem mentioned below, you can download the new 4.3 version of the packages (only available for Visual Studio 2005/2008).

Key new features:

• *Support for Mac OS X*
• Visual Studio 2010 support
• Bullet library update to 2.77
• OIS library update to 1.3
• Support for CEGUI 0.7 —> Support for Arch Linux and even SuSE
• Improved install target
• Compiles now with GCC 4.6
• Ogre Cg Shader plugin activated for Linux if available
• And of course lots of bug fixes

There are also some regressions:

• No support for CEGUI 0.5, Ogre 1.4 and boost 1.35 - 1.39 any more
• In game console is not working in main menu for CEGUI 0.7
• Tolua (just the C lib, not the application) and CEGUILua libraries are no longer in our repository. —> You will need to get these as well when compiling Orxonox
• And of course lots of new bugs we don't yet know about

File:

• code/trunk/src/external/ois/linux/LinuxKeyboard.cpp (modified) (6 diffs)

code/trunk/src/external/ois/linux/LinuxKeyboard.cpp

@@ -33 +33 @@
 #include <iostream>
 //-------------------------------------------------------------------//
-LinuxKeyboard::LinuxKeyboard(InputManager* creator, bool buffered, bool grab, bool useXRepeat)
+LinuxKeyboard::LinuxKeyboard(InputManager* creator, bool buffered, bool grab)
         : Keyboard(creator->inputSystemName(), buffered, 0, creator)
 {
@@ -43 +43 @@
         grabKeyboard = grab;
         keyFocusLost = false;
-
-        xAutoRepeat = useXRepeat;
-        oldXAutoRepeat = false;
 
         //X Key Map to KeyCode
@@ -213 +210 @@
 
         keyFocusLost = false;
-
-        if( xAutoRepeat == false )
-        {
-                //We do not want to blindly turn on autorepeat later when quiting if
-                //it was not on to begin with.. So, let us check and see first
-                XKeyboardState old;
-                XGetKeyboardControl( display, &old );
-                oldXAutoRepeat = false;
-
-                if( old.global_auto_repeat == AutoRepeatModeOn )
-                        oldXAutoRepeat = true;
-
-                XAutoRepeatOff( display );
-        }
 }
 
@@ -234 +217 @@
         if( display )
         {
-                if( oldXAutoRepeat )
-                        XAutoRepeatOn(display);
-
                 if( grabKeyboard )
                         XUngrabKeyboard(display, CurrentTime);
@@ -303 +283 @@
 
         while( XPending(display) > 0 )
-        {               XNextEvent(display, &event);            if( KeyPress == event.type )
+        {
+                XNextEvent(display, &event);
+
+                if(KeyPress == event.type)
                 {
                         unsigned int character = 0;
 
-                        if( mTextMode != Off )
+                        if(mTextMode != Off)
                         {
                                 unsigned char buffer[6] = {0,0,0,0,0,0};
@@ -332 +315 @@
                         if( event.xkey.state & Mod1Mask && key == XK_Tab )
                                 linMan->_setGrabState(false);
-                }               else if( KeyRelease == event.type )
+                }
+                else if(KeyRelease == event.type)
                 {
-                        //Mask out the modifier states X sets.. or we will get improper values
-                        event.xkey.state &= ~ShiftMask;
-                        event.xkey.state &= ~LockMask;
-
-                        //Else, it is a valid key release
-                        XLookupString(&event.xkey,NULL,0,&key,NULL);
-                        _injectKeyUp(key);              }
+                        if(!_isKeyRepeat(event))
+                        {
+                                //Mask out the modifier states X sets.. or we will get improper values
+                                event.xkey.state &= ~ShiftMask;
+                                event.xkey.state &= ~LockMask;
+
+                                XLookupString(&event.xkey,NULL,0,&key,NULL);
+                                _injectKeyUp(key);
+                        }
+                }
         }