Kaydet (Commit) 8ef38360 authored tarafından Herbert Dürr's avatar Herbert Dürr

#i124201# rename apple_remote's MainController to AppleRemoteController

having a MainController class in the global namespace of an application with
gazillions of MVC patterns is not a good idea. Renaming it to better match
its scope (i.e. the Apple Remote) cleans this up.
üst 17b67844
...@@ -37,7 +37,7 @@ ...@@ -37,7 +37,7 @@
// Sample Code 3: Multi Click Behavior and Hold Event Simulation // Sample Code 3: Multi Click Behavior and Hold Event Simulation
// ------------------------------------------------------------------------------------------- // -------------------------------------------------------------------------------------------
@implementation MainController @implementation AppleRemoteMainController
- (id) init { - (id) init {
self = [super init]; // because we redefined our own init instead of use the fu..nny awakeFromNib self = [super init]; // because we redefined our own init instead of use the fu..nny awakeFromNib
...@@ -73,11 +73,11 @@ ...@@ -73,11 +73,11 @@
// to give the binding mechanism a chance to see the change of the attribute // to give the binding mechanism a chance to see the change of the attribute
[self setValue: container forKey: @"remoteControl"]; [self setValue: container forKey: @"remoteControl"];
#ifdef DEBUG #ifdef DEBUG
NSLog(@"MainController init done"); NSLog(@"AppleRemoteMainController init done");
#endif #endif
} }
else else
NSLog(@"MainController init failed"); NSLog(@"AppleRemoteMainController init failed");
return self; return self;
} }
......
...@@ -37,7 +37,7 @@ ...@@ -37,7 +37,7 @@
//static void sendTheEvent( unichar, int ); //static void sendTheEvent( unichar, int );
@interface MainController : NSObject { @interface AppleRemoteMainController : NSObject {
@public // else remoteControl is not reachable from GetSalData()->mpMainController @public // else remoteControl is not reachable from GetSalData()->mpMainController
RemoteControl* remoteControl; RemoteControl* remoteControl;
@private @private
......
...@@ -54,7 +54,7 @@ SalData::SalData() ...@@ -54,7 +54,7 @@ SalData::SalData()
maCursors( POINTER_COUNT, INVALID_CURSOR_PTR ), maCursors( POINTER_COUNT, INVALID_CURSOR_PTR ),
mbIsScrollbarDoubleMax( false ), mbIsScrollbarDoubleMax( false ),
mnSystemVersion( OSX_VER_LION ), mnSystemVersion( OSX_VER_LION ),
mpMainController( NULL ), mpAppleRemoteMainController( NULL ),
mpDockIconClickHandler( nil ), mpDockIconClickHandler( nil ),
mnDPIX( 0 ), mnDPIX( 0 ),
mnDPIY( 0 ) mnDPIY( 0 )
...@@ -89,8 +89,8 @@ SalData::~SalData() ...@@ -89,8 +89,8 @@ SalData::~SalData()
osl_destroyThreadKey( s_aAutoReleaseKey ); osl_destroyThreadKey( s_aAutoReleaseKey );
s_aAutoReleaseKey = 0; s_aAutoReleaseKey = 0;
} }
if ( mpMainController ) if ( mpAppleRemoteMainController )
[mpMainController release]; [mpAppleRemoteMainController release];
} }
void SalData::ensureThreadAutoreleasePool() void SalData::ensureThreadAutoreleasePool()
......
...@@ -190,8 +190,7 @@ static void initNSApp() ...@@ -190,8 +190,7 @@ static void initNSApp()
else else
NSLog(@"Unable to obtain system version: %ld", (long)err); NSLog(@"Unable to obtain system version: %ld", (long)err);
// Initialize Apple Remote GetSalData()->mpAppleRemoteMainController = [[AppleRemoteMainController alloc] init];
GetSalData()->mpMainController = [[MainController alloc] init]; // TODO: rename MainController to AppleRemoteController
[[NSDistributedNotificationCenter defaultCenter] addObserver: NSApp [[NSDistributedNotificationCenter defaultCenter] addObserver: NSApp
selector: @selector(applicationWillBecomeActive:) selector: @selector(applicationWillBecomeActive:)
......
...@@ -450,15 +450,16 @@ ...@@ -450,15 +450,16 @@
{ {
(void)pNotification; (void)pNotification;
SalData* pSalData = GetSalData(); SalData* pSalData = GetSalData();
if( pSalData->mpMainController && pSalData->mpMainController->remoteControl) AppleRemoteMainController* pAppleRemoteCtrl = pSalData->mpAppleRemoteMainController;
if( pAppleRemoteCtrl && pAppleRemoteCtrl->remoteControl)
{ {
// [remoteControl startListening: self]; // [remoteControl startListening: self];
// does crash because the right thing to do is // does crash because the right thing to do is
// [GetSalData()->mpMainController->remoteControl startListening: self]; // [pAppleRemoteCtrl->remoteControl startListening: self];
// but the instance variable 'remoteControl' is declared protected // but the instance variable 'remoteControl' is declared protected
// workaround : declare remoteControl instance variable as public in RemoteMainController.m // workaround : declare remoteControl instance variable as public in RemoteMainController.m
[pSalData->mpMainController->remoteControl startListening: self]; [pAppleRemoteCtrl->remoteControl startListening: self];
#ifdef DEBUG #ifdef DEBUG
NSLog(@"Apple Remote will become active - Using remote controls"); NSLog(@"Apple Remote will become active - Using remote controls");
#endif #endif
...@@ -477,15 +478,16 @@ ...@@ -477,15 +478,16 @@
{ {
(void)pNotification; (void)pNotification;
SalData* pSalData = GetSalData(); SalData* pSalData = GetSalData();
if( pSalData->mpMainController && pSalData->mpMainController->remoteControl) AppleRemoteMainController* pAppleRemoteCtrl = pSalData->mpAppleRemoteMainController;
if( pAppleRemoteCtrl && pAppleRemoteCtrl->remoteControl)
{ {
// [remoteControl stopListening: self]; // [remoteControl stopListening: self];
// does crash because the right thing to do is // does crash because the right thing to do is
// [GetSalData()->mpMainController->remoteControl stopListening: self]; // [pAppleRemoteCtrl->remoteControl stopListening: self];
// but the instance variable 'remoteControl' is declared protected // but the instance variable 'remoteControl' is declared protected
// workaround : declare remoteControl instance variable as public in RemoteMainController.m // workaround : declare remoteControl instance variable as public in RemoteMainController.m
[pSalData->mpMainController->remoteControl stopListening: self]; [pAppleRemoteCtrl->remoteControl stopListening: self];
#ifdef DEBUG #ifdef DEBUG
NSLog(@"Apple Remote will resign active - Releasing remote controls"); NSLog(@"Apple Remote will resign active - Releasing remote controls");
#endif #endif
......
...@@ -106,7 +106,7 @@ struct SalData ...@@ -106,7 +106,7 @@ struct SalData
bool mbIsScrollbarDoubleMax; // TODO: support DoubleMin and DoubleBoth too bool mbIsScrollbarDoubleMax; // TODO: support DoubleMin and DoubleBoth too
SInt32 mnSystemVersion; // Store System Version SInt32 mnSystemVersion; // Store System Version
MainController* mpMainController; // Apple Remote AppleRemoteMainController* mpAppleRemoteMainController;
NSObject* mpDockIconClickHandler; NSObject* mpDockIconClickHandler;
long mnDPIX; // #i100617# read DPI only once per office life long mnDPIX; // #i100617# read DPI only once per office life
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment