From 7e562273d5e0c032782d6ea5701464efddbfa9b5 Mon Sep 17 00:00:00 2001 From: jerry Date: Mon, 15 Sep 2014 15:17:07 +0000 Subject: [PATCH] [no-issue] revert to r2421 --- CMakeLists.txt | 10 ++--- ext/toolchain/commands1.py | 20 +-------- src/lib/common/basic_types.h | 4 -- src/lib/platform/OSXScreen.cpp | 65 ++++++++++++----------------- src/lib/platform/OSXScreenSaver.cpp | 6 +-- src/lib/synergy/DaemonApp.cpp | 4 +- 6 files changed, 38 insertions(+), 71 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index d2959b40..4dca34f3 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -164,14 +164,14 @@ if (UNIX) exec_program(uname ARGS -v OUTPUT_VARIABLE DARWIN_VERSION) string(REGEX MATCH "[0-9]+" DARWIN_VERSION ${DARWIN_VERSION}) message(STATUS "DARWIN_VERSION=${DARWIN_VERSION}") - # patch by Jake Petroules for issue 575 if (DARWIN_VERSION LESS 9) - # 10.4: Universal (32-bit and 64-bit Intel and PowerPC) - set(CMAKE_OSX_ARCHITECTURES "ppc;ppc64;i386:x86_64" + # 10.4: universal (32-bit intel and power pc) + set(CMAKE_OSX_ARCHITECTURES "ppc;i386" CACHE STRING "" FORCE) else() - # 10.5+: Intel only - set(CMAKE_OSX_ARCHITECTURES "i386;x86_64" + # 10.5+: 32-bit only -- missing funcs in 64-bit os libs + # such as GetGlobalMouse. + set(CMAKE_OSX_ARCHITECTURES "i386" CACHE STRING "" FORCE) endif() diff --git a/ext/toolchain/commands1.py b/ext/toolchain/commands1.py index 53ab4f67..ef492f2b 100644 --- a/ext/toolchain/commands1.py +++ b/ext/toolchain/commands1.py @@ -554,25 +554,7 @@ class InternalCommands: return (major, minor, rev) def getMacSdkDir(self): - # patch by Jake Petroules for issue 575 - sdkName = "macosx" + self.macSdk - - # Ideally we'll use xcrun (which is influenced by $DEVELOPER_DIR), then try a couple - # fallbacks to known paths if xcrun is not available - sdkPath = commands.getoutput("xcrun --show-sdk-path --sdk " + sdkName) - if sdkPath: - return sdkPath - - developerDir = os.getenv("DEVELOPER_DIR") - if not developerDir: - developerDir = "/Applications/Xcode.app/Contents/Developer" - - sdkDirName = sdkName.replace("macosx", "MacOSX") - sdkPath = developerDir + "/Platforms/MacOSX.platform/Developer/SDKs/" + sdkDirName + ".sdk" - if os.path.exists(sdkPath): - return sdkPath - - return "/Developer/SDKs/" + sdkDirName + ".sdk" + return "/Developer/SDKs/MacOSX" + self.macSdk + ".sdk" # http://tinyurl.com/cs2rxxb def fixCmakeEclipseBug(self): diff --git a/src/lib/common/basic_types.h b/src/lib/common/basic_types.h index 0d4c68a6..7f20cb5a 100644 --- a/src/lib/common/basic_types.h +++ b/src/lib/common/basic_types.h @@ -72,9 +72,6 @@ // Added this because it doesn't compile on OS X 10.6 because they are already defined in Carbon #if !defined(__MACTYPES__) -#if defined(__APPLE__) -#include -#else typedef signed TYPE_OF_SIZE_1 SInt8; typedef signed TYPE_OF_SIZE_2 SInt16; typedef signed TYPE_OF_SIZE_4 SInt32; @@ -82,7 +79,6 @@ typedef unsigned TYPE_OF_SIZE_1 UInt8; typedef unsigned TYPE_OF_SIZE_2 UInt16; typedef unsigned TYPE_OF_SIZE_4 UInt32; #endif -#endif // // clean up // diff --git a/src/lib/platform/OSXScreen.cpp b/src/lib/platform/OSXScreen.cpp index f628ce11..cf58aaa2 100644 --- a/src/lib/platform/OSXScreen.cpp +++ b/src/lib/platform/OSXScreen.cpp @@ -292,15 +292,13 @@ COSXScreen::getShape(SInt32& x, SInt32& y, SInt32& w, SInt32& h) const void COSXScreen::getCursorPos(SInt32& x, SInt32& y) const { - // patch by Jake Petroules for issue 575 - CGEventRef event = CGEventCreate(NULL); - CGPoint mouse = CGEventGetLocation(event); - x = mouse.x; - y = mouse.y; + Point mouse; + GetGlobalMouse(&mouse); + x = mouse.h; + y = mouse.v; m_cursorPosValid = true; - m_xCursor = x; - m_yCursor = y; - CFRelease(event); + m_xCursor = x; + m_yCursor = y; } void @@ -692,24 +690,21 @@ COSXScreen::fakeMouseMove(SInt32 x, SInt32 y) void COSXScreen::fakeMouseRelativeMove(SInt32 dx, SInt32 dy) const { - // patch by Jake Petroules for issue 575 - // OS X does not appear to have a fake relative mouse move function. // simulate it by getting the current mouse position and adding to // that. this can yield the wrong answer but there's not much else // we can do. // get current position - CGEventRef event = CGEventCreate(NULL); - CGPoint oldPos = CGEventGetLocation(event); - CFRelease(event); + Point oldPos; + GetGlobalMouse(&oldPos); // synthesize event CGPoint pos; - m_xCursor = static_cast(oldPos.x); - m_yCursor = static_cast(oldPos.y); - pos.x = oldPos.x + dx; - pos.y = oldPos.y + dy; + m_xCursor = static_cast(oldPos.h); + m_yCursor = static_cast(oldPos.v); + pos.x = oldPos.h + dx; + pos.y = oldPos.v + dy; postMouseEvent(pos); // we now assume we don't know the current cursor position @@ -1057,7 +1052,7 @@ COSXScreen::handleSystemEvent(const CEvent& event, void*) // get scroll amount r = GetEventParameter(*carbonEvent, kSynergyMouseScrollAxisX, - typeSInt32, + typeLongInteger, NULL, sizeof(xScroll), NULL, @@ -1067,7 +1062,7 @@ COSXScreen::handleSystemEvent(const CEvent& event, void*) } r = GetEventParameter(*carbonEvent, kSynergyMouseScrollAxisY, - typeSInt32, + typeLongInteger, NULL, sizeof(yScroll), NULL, @@ -1095,11 +1090,7 @@ COSXScreen::handleSystemEvent(const CEvent& event, void*) break; case kEventClassWindow: - // patch by Jake Petroules for issue 575 - // 2nd param was formerly GetWindowEventTarget(m_userInputWindow) which is 32-bit only, - // however as m_userInputWindow is never initialized to anything we can take advantage of - // the fact that GetWindowEventTarget(NULL) == NULL - SendEventToEventTarget(*carbonEvent, NULL); + SendEventToWindow(*carbonEvent, m_userInputWindow); switch (GetEventKind(*carbonEvent)) { case kEventWindowActivated: LOG((CLOG_DEBUG1 "window activated")); @@ -1523,17 +1514,15 @@ COSXScreen::getScrollSpeedFactor() const void COSXScreen::enableDragTimer(bool enable) { + UInt32 modifiers; + MouseTrackingResult res; + if (enable && m_dragTimer == NULL) { m_dragTimer = m_events->newTimer(0.01, NULL); m_events->adoptHandler(CEvent::kTimer, m_dragTimer, new TMethodEventJob(this, &COSXScreen::handleDrag)); - // patch by Jake Petroules for issue 575 - CGEventRef event = CGEventCreate(NULL); - CGPoint mouse = CGEventGetLocation(event); - m_dragLastPoint.h = (short)mouse.x; - m_dragLastPoint.v = (short)mouse.y; - CFRelease(event); + TrackMouseLocationWithOptions(NULL, 0, 0, &m_dragLastPoint, &modifiers, &res); } else if (!enable && m_dragTimer != NULL) { m_events->removeHandler(CEvent::kTimer, m_dragTimer); @@ -1545,15 +1534,15 @@ COSXScreen::enableDragTimer(bool enable) void COSXScreen::handleDrag(const CEvent&, void*) { - // patch by Jake Petroules for issue 575 - CGEventRef event = CGEventCreate(NULL); - CGPoint p = CGEventGetLocation(event); - CFRelease(event); + Point p; + UInt32 modifiers; + MouseTrackingResult res; - if ((short)p.x != m_dragLastPoint.h || (short)p.y != m_dragLastPoint.v) { - m_dragLastPoint.h = (short)p.x; - m_dragLastPoint.v = (short)p.y; - onMouseMove((SInt32)p.x, (SInt32)p.y); + TrackMouseLocationWithOptions(NULL, 0, 0, &p, &modifiers, &res); + + if (res != kMouseTrackingTimedOut && (p.h != m_dragLastPoint.h || p.v != m_dragLastPoint.v)) { + m_dragLastPoint = p; + onMouseMove((SInt32)p.h, (SInt32)p.v); } } diff --git a/src/lib/platform/OSXScreenSaver.cpp b/src/lib/platform/OSXScreenSaver.cpp index 01955713..8ae9dc5c 100644 --- a/src/lib/platform/OSXScreenSaver.cpp +++ b/src/lib/platform/OSXScreenSaver.cpp @@ -140,9 +140,9 @@ COSXScreenSaver::launchTerminationCallback( EventRef theEvent, void* userData) { OSStatus result; - ProcessSerialNumber psn; - EventParamType actualType; - ByteCount actualSize; + ProcessSerialNumber psn; + EventParamType actualType; + UInt32 actualSize; result = GetEventParameter(theEvent, kEventParamProcessID, typeProcessSerialNumber, &actualType, diff --git a/src/lib/synergy/DaemonApp.cpp b/src/lib/synergy/DaemonApp.cpp index aa03ee32..e8e8007b 100644 --- a/src/lib/synergy/DaemonApp.cpp +++ b/src/lib/synergy/DaemonApp.cpp @@ -302,9 +302,9 @@ CDaemonApp::handleIpcMessage(const CEvent& e, void*) LOG((CLOG_DEBUG "new command, elevate=%d command=%s", cm->elevate(), command.c_str())); CString debugArg("--debug"); - size_t debugArgPos = command.find(debugArg); + UInt32 debugArgPos = static_cast(command.find(debugArg)); if (debugArgPos != CString::npos) { - UInt32 from = static_cast(debugArgPos) + static_cast(debugArg.size()) + 1; + UInt32 from = debugArgPos + static_cast(debugArg.size()) + 1; UInt32 nextSpace = static_cast(command.find(" ", from)); CString logLevel(command.substr(from, nextSpace - from));