fixed: code style in OSXKeyState, released ref var

src/lib/platform/OSXKeyState.cpp

@@ -181,15 +181,13 @@ static const CKeyEntry	s_controlKeys[] = {
 //
 
 COSXKeyState::COSXKeyState(IEventQueue* events) :
-	CKeyState(events),
+	CKeyState(events)
-	m_deadKeyState(0)
 {
 	init();
 }
 
 COSXKeyState::COSXKeyState(IEventQueue* events, CKeyMap& keyMap) :
-	CKeyState(events, keyMap),
+	CKeyState(events, keyMap)
-	m_deadKeyState(0)
 {
 	init();
 }
@@ -201,16 +199,17 @@ COSXKeyState::~COSXKeyState()
 void
 COSXKeyState::init()
 {
-	// initialize modifier key values
+	m_deadKeyState = 0;
-	shiftPressed = false;
+	m_shiftPressed = false;
-	controlPressed = false;
+	m_controlPressed = false;
-	altPressed = false;
+	m_altPressed = false;
-	superPressed = false;
+	m_superPressed = false;
-	capsPressed = false;
+	m_capsPressed = false;
 
 	// build virtual key map
-	for (size_t i = 0; i < sizeof(s_controlKeys) /
+	for (size_t i = 0; i < sizeof(s_controlKeys) / sizeof(s_controlKeys[0]);
-								sizeof(s_controlKeys[0]); ++i) {
+		++i) {
+		
 		m_virtualKeyMap[s_controlKeys[i].m_virtualKey] =
 			s_controlKeys[i].m_keyID;
 	}
@@ -483,63 +482,97 @@ COSXKeyState::getKeyMap(CKeyMap& keyMap)
 void
 COSXKeyState::fakeKey(const Keystroke& keystroke)
 {
-	CGEventRef ref;
-
 	switch (keystroke.m_type) {
-	case Keystroke::kButton:
+	case Keystroke::kButton: {
-		{
+		
-			LOG((CLOG_DEBUG1 "  %03x (%08x) %s", keystroke.m_data.m_button.m_button, keystroke.m_data.m_button.m_client, keystroke.m_data.m_button.m_press ? "down" : "up"));
+		KeyButton button = keystroke.m_data.m_button.m_button;
+		bool keyDown = keystroke.m_data.m_button.m_press;
+		UInt32 client = keystroke.m_data.m_button.m_client;
+		CGEventSourceRef source = 0;
+		CGKeyCode virtualKey = mapKeyButtonToVirtualKey(button);
+		
+		LOG((CLOG_DEBUG1
+			"  button=0x%04x virtualKey=0x%04x keyDown=%s client=0x%04x",
+			button, virtualKey, keyDown ? "down" : "up", client));
+
+		CGEventRef ref = CGEventCreateKeyboardEvent(
+			source, virtualKey, keyDown);
 		
-		// let system figure out character for us
-		ref = CGEventCreateKeyboardEvent(0, mapKeyButtonToVirtualKey(
-									keystroke.m_data.m_button.m_button),
-								keystroke.m_data.m_button.m_press);
 		if (ref == NULL) {
 			LOG((CLOG_CRIT "unable to create keyboard event for keystroke"));
+			return;
 		}
 
-			UInt32 vk = mapKeyButtonToVirtualKey(keystroke.m_data.m_button.m_button);
+		// persist modifier state.
-			UInt32 modifierDown = keystroke.m_data.m_button.m_press;
+		if (virtualKey == s_shiftVK) {
+			m_shiftPressed = keyDown;
+		}
 		
-			// check the key for specials and store the value (persistent until changed)
+		if (virtualKey == s_controlVK) {
-			if (vk == s_shiftVK)    shiftPressed=modifierDown;
+			m_controlPressed = keyDown;
-			if (vk == s_controlVK)  controlPressed=modifierDown;
+		}
-			if (vk == s_altVK)      altPressed=modifierDown;
-			if (vk == s_superVK)    superPressed=modifierDown;
-			if (vk == s_capsLockVK) capsPressed=modifierDown;
 		
-			//Set the event flags for special keys - see following link:
+		if (virtualKey == s_altVK) {
-			//http://stackoverflow.com/questions/2008126/cgeventpost-possible-bug-when-simulating-keyboard-events
+			m_altPressed = keyDown;
-			CGEventFlags modifiers = 0;
+		}
-			if (shiftPressed)   modifiers |= kCGEventFlagMaskShift;
-			if (controlPressed) modifiers |= kCGEventFlagMaskControl;
-			if (altPressed)     modifiers |= kCGEventFlagMaskAlternate;
-			if (superPressed)   modifiers |= kCGEventFlagMaskCommand;
-			if (capsPressed)    modifiers |= kCGEventFlagMaskAlphaShift;
 		
-			CGEventSetFlags(ref, modifiers);
+		if (virtualKey == s_superVK) {
+			m_superPressed = keyDown;
+		}
 		
-			CGEventPost(kCGHIDEventTap, ref);
+		if (virtualKey == s_capsLockVK) {
+			m_capsPressed = keyDown;
+		}
 
-			// add a delay if client data isn't zero
+		// set the event flags for special keys
-			if (keystroke.m_data.m_button.m_client) {
+		// http://tinyurl.com/pxl742y
-				ARCH->sleep(0.01);
+		CGEventFlags modifiers = 0;
-			}
+		
+		if (m_shiftPressed) {
+			modifiers |= kCGEventFlagMaskShift;
+		}
+		
+		if (m_controlPressed) {
+			modifiers |= kCGEventFlagMaskControl;
+		}
+		
+		if (m_altPressed) {
+			modifiers |= kCGEventFlagMaskAlternate;
+		}
+		
+		if (m_superPressed) {
+			modifiers |= kCGEventFlagMaskCommand;
+		}
+		
+		if (m_capsPressed) {
+			modifiers |= kCGEventFlagMaskAlphaShift;
+		}
+		
+		CGEventSetFlags(ref, modifiers);
+		CGEventPost(kCGHIDEventTap, ref);
+		CFRelease(ref);
+
+		// add a delay if client data isn't zero
+		// FIXME -- why?
+		if (client != 0) {
+			ARCH->sleep(0.01);
 		}
 		break;
+	}
 
-	case Keystroke::kGroup:
+	case Keystroke::kGroup: {
+		SInt32 group = keystroke.m_data.m_group.m_group;
 		if (keystroke.m_data.m_group.m_absolute) {
-			LOG((CLOG_DEBUG1 "  group %d", keystroke.m_data.m_group.m_group));
+			LOG((CLOG_DEBUG1 "  group %d", group));
-			setGroup(keystroke.m_data.m_group.m_group);
+			setGroup(group);
 		}
 		else {
-			LOG((CLOG_DEBUG1 "  group %+d", keystroke.m_data.m_group.m_group));
+			LOG((CLOG_DEBUG1 "  group %+d", group));
-			setGroup(getEffectiveGroup(pollActiveGroup(),
+			setGroup(getEffectiveGroup(pollActiveGroup(), group));
-									keystroke.m_data.m_group.m_group));
 		}
 		break;
 	}
+	}
 }
 
 void

src/lib/platform/OSXKeyState.h

@@ -214,11 +214,9 @@ private:
 	mutable UInt32		m_deadKeyState;
 	GroupList			m_groups;
 	GroupMap			m_groupMap;
-	
+	bool				m_shiftPressed;
-	// Hold the current state of modifier keys
+	bool				m_controlPressed;
-	bool shiftPressed;
+	bool				m_altPressed;
-	bool controlPressed;
+	bool				m_superPressed;
-	bool altPressed;
+	bool				m_capsPressed;
-	bool superPressed;
-	bool capsPressed;
 };