Refactor Actions. SubAction is a (mostly) proper JSAction. JSActions have (more)...
authorJoe Wreschnig <joe.wreschnig@gmail.com>
Tue, 26 Feb 2013 23:31:03 +0000 (00:31 +0100)
committerJoe Wreschnig <joe.wreschnig@gmail.com>
Tue, 26 Feb 2013 23:31:03 +0000 (00:31 +0100)
16 files changed:
Config.h
Config.m
JSAction.h
JSAction.m
JSActionAnalog.h
JSActionAnalog.m
JSActionButton.h
JSActionButton.m
JSActionHat.h
JSActionHat.m
Joystick.h
Joystick.m
JoystickController.h
JoystickController.m
SubAction.h
SubAction.m

index 1120edea47f1e78d2fe462a729c94c868b575472..d11826205fae75676396577b09dc1b7ad04effd5 100644 (file)
--- a/Config.h
+++ b/Config.h
@@ -6,20 +6,18 @@
 //  Copyright 2009 University of Otago. All rights reserved.
 //
 
-#import <Cocoa/Cocoa.h>
+#import <Foundation/Foundation.h>
+
 @class Target;
+@class JSAction;
 
-@interface Config : NSObject {
-       NSString *name;
-       BOOL protect;
-       NSMutableDictionary *entries;
-}
+@interface Config : NSObject
 
-@property(readwrite) BOOL protect;
-@property(readwrite, copy) NSString* name;
-@property(readonly) NSMutableDictionary* entries;
+@property (assign) BOOL protect;
+@property (copy) NSString *name;
+@property (readonly) NSMutableDictionary *entries;
 
--(void) setTarget:(Target*)target forAction:(id)jsa;
--(Target*) getTargetForAction: (id) jsa;
+- (void)setTarget:(Target *)target forAction:(JSAction *)jsa;
+- (Target *)getTargetForAction:(JSAction *)sa;
 
 @end
index 27ca4f0e91912a23585ecd6fbca2c4235e91e1af..f62c40fdf49a1571442bd25becb1ffd9f7bd13ac 100644 (file)
--- a/Config.m
+++ b/Config.m
@@ -5,22 +5,25 @@
 //  Created by Sam McCall on 4/05/09.
 //
 
-@implementation Config
+#import "Config.h"
 
--(id) init {
-       if(self=[super init]) {
-               entries = [[NSMutableDictionary alloc] init];
-       }
-       return self;
-}
+@implementation Config
 
 @synthesize protect, name, entries;
 
--(void) setTarget:(Target*)target forAction:(id)jsa {
-       [entries setValue:target forKey: [jsa stringify]];
+- (id)init {
+    if ((self = [super init])) {
+        entries = [[NSMutableDictionary alloc] init];
+    }
+    return self;
 }
--(Target*) getTargetForAction: (id) jsa {
-       return entries[[jsa stringify]];
+
+- (void)setTarget:(Target *)target forAction:(JSAction *)jsa {
+    entries[[jsa stringify]] = target;
+}
+
+- (Target *)getTargetForAction:(JSAction *)jsa {
+    return entries[[jsa stringify]];
 }
 
 @end
index 1e60cf2e11f3ec9bb86d80e13e5f225918e9fbd9..9de469659445cc69113046868a6df3ae1326cf72 100644 (file)
@@ -6,28 +6,23 @@
 //  Copyright 2009 University of Otago. All rights reserved.
 //
 
-#import <Cocoa/Cocoa.h>
-#import <IOKit/hid/IOHIDLib.h>
+#import <Foundation/Foundation.h>
+#import <IOKit/hid/IOHIDBase.h>
 
 @interface JSAction : NSObject {
-       int usage, index;
-       void* cookie;
-       NSArray *subActions;
        id base;
        NSString *name;
 }
 
-@property(readwrite) int usage;
-@property(readwrite) void* cookie;
-@property(readonly) int index;
-@property(copy) NSArray* subActions;
-@property(readwrite, strong) id base;
-@property(copy) NSString* name;
-@property(readonly) BOOL active;
+@property (assign) void* cookie;
+@property (assign) int index;
+@property (copy) NSArray *subActions;
+@property (strong) id base;
+@property (copy) NSString *name;
+@property (readonly) BOOL active;
 
--(void) notifyEvent: (IOHIDValueRef) value;
--(NSString*) stringify;
--(NSArray*) subActions;
--(id) findSubActionForValue: (IOHIDValueRef) value;
+- (void)notifyEvent:(IOHIDValueRef)value;
+- (NSString *)stringify;
+- (id)findSubActionForValue:(IOHIDValueRef)value;
 
 @end
index a0fab80fde6461ebadd721b28f87cda7647d76f7..6bc15291d3c37fd72100017123498ba8a168e9e8 100644 (file)
@@ -5,23 +5,31 @@
 //  Created by Sam McCall on 4/05/09.
 //
 
+#import "JSAction.h"
+
 @implementation JSAction
 
-@synthesize usage, cookie, index, subActions, base, name;
+@synthesize cookie;
+@synthesize index;
+@synthesize subActions;
+@synthesize base;
+@synthesize name;
 
--(id) findSubActionForValue: (IOHIDValueRef) value {
-       return NULL;
+- (id)findSubActionForValue:(IOHIDValueRef)value {
+    return NULL;
 }
 
--(NSString*) stringify {
-       return [[NSString alloc] initWithFormat: @"%@~%d",[base stringify],(int)cookie];
+- (NSString *)stringify {
+    return [NSString stringWithFormat: @"%@~%p", [base stringify], cookie];
 }
--(void) notifyEvent: (IOHIDValueRef) value {
-       [self doesNotRecognizeSelector:_cmd];
+
+- (void)notifyEvent:(IOHIDValueRef)value {
+    [self doesNotRecognizeSelector:_cmd];
 }
--(BOOL) active {
-       [self doesNotRecognizeSelector:_cmd];
-       return NO;
+
+- (BOOL)active {
+    [self doesNotRecognizeSelector:_cmd];
+    return NO;
 }
 
 @end
index f1276d509df3ddfc685c4df75b5b193714770b92..f351a4ab06bba9280b19412f39f01440ea7b4a4b 100644 (file)
@@ -7,16 +7,15 @@
 //
 
 #import <Cocoa/Cocoa.h>
-@class JSAction;
 
-@interface JSActionAnalog : JSAction {
-       double offset, scale;
-}
+#import "JSAction.h"
 
-@property(readwrite) double offset;
-@property(readwrite) double scale;
+@interface JSActionAnalog : JSAction
 
-- (id) initWithIndex: (int)newIndex;
--(double) getRealValue: (int) value;
+@property (assign) float offset;
+@property (assign) float scale;
+
+- (id)initWithIndex:(int)newIndex offset:(float)offset scale:(float)scale;
+- (float)getRealValue:(int)value;
 
 @end
index 027bd5db95f8436a7f7040e9057e7f95e6e869ab..e7d6eba8f6b2d08687e8e775f241fc30ffd97db0 100644 (file)
@@ -5,53 +5,49 @@
 //  Created by Sam McCall on 5/05/09.
 //
 
+// TODO: Dead zone should be configurable per-device.
+#define DEAD_ZONE 0.3
+
+#import "JSActionAnalog.h"
+
 @implementation JSActionAnalog
 
-- (id) initWithIndex: (int)newIndex {
-       if(self = [super init]) {
-       subActions = @[[[SubAction alloc] initWithIndex: 0 name: @"Low" base: self],
-               [[SubAction alloc] initWithIndex: 1 name: @"High" base: self],
-        [[SubAction alloc] initWithIndex: 2 name: @"Analog" base: self]];
-       index = newIndex;
-       name = [[NSString alloc] initWithFormat: @"Axis %d", (index+1)];
-   }
-       return self;
-}
+@synthesize offset, scale;
 
--(id) findSubActionForValue: (IOHIDValueRef) value {
-    if ([subActions[2] active]) {
-        return subActions[2]; // TODO?
+- (id)initWithIndex:(int)newIndex offset:(float)offset_ scale:(float)scale_ {
+    if ((self = [super init])) {
+        self.subActions = @[[[SubAction alloc] initWithIndex:0 name:@"Low" base:self],
+                            [[SubAction alloc] initWithIndex:1 name:@"High" base:self]];
+        self.index = newIndex;
+        self.offset = offset_;
+        self.scale = scale_;
+        self.name = [[NSString alloc] initWithFormat: @"Axis %d", self.index + 1];
     }
-    
-    //Target* target = [[base->configsController currentConfig] getTargetForAction: [subActions objectAtIndex: 0]];
-    
-       int raw = IOHIDValueGetIntegerValue(value);
-    double parsed = [self getRealValue: raw];
-       
-       if(parsed < -0.3) // fixed?!
-               return subActions[0];
-       else if(parsed > 0.3)
-               return subActions[1];
-       return NULL;
+    return self;
 }
 
--(void) notifyEvent: (IOHIDValueRef) value {
-    // Analog action is always active
-    [subActions[2] setActive: true];
+- (id)findSubActionForValue:(IOHIDValueRef)value {
+    int raw = IOHIDValueGetIntegerValue(value);
+    float parsed = [self getRealValue:raw];
     
-       int raw = IOHIDValueGetIntegerValue(value);
-    double parsed = [self getRealValue: raw];
-       
-       [subActions[0] setActive: (parsed < -0.3)];
-       [subActions[1] setActive: (parsed > 0.3)];
+    if (parsed < -DEAD_ZONE)
+        return self.subActions[0];
+    else if (parsed > DEAD_ZONE)
+        return self.subActions[1];
+    else
+        return nil;
 }
 
--(double) getRealValue: (int)value {
-       double parsed = offset + scale * value;
-    return parsed;
+- (void)notifyEvent:(IOHIDValueRef)value {
+    int raw = IOHIDValueGetIntegerValue(value);
+    float parsed = [self getRealValue:raw];
+    [self.subActions[0] setActive:parsed < -DEAD_ZONE];
+    [self.subActions[1] setActive:parsed > DEAD_ZONE];
 }
 
-@synthesize offset, scale;
+- (float)getRealValue:(int)value {
+    return offset + scale * value;
+}
 
 
 @end
index 4720b736f53e4e996465c62f246dc650d4a9ae90..5a9a4c0629039e0842c8bf7ae4d5ae8ed6fc7695 100644 (file)
@@ -7,14 +7,13 @@
 //
 
 #import <Cocoa/Cocoa.h>
-@class JSAction;
 
-@interface JSActionButton : JSAction {
-       int max;
-       BOOL active;
-}
+#import "JSAction.h"
 
--(id)initWithIndex: (int)newIndex andName: (NSString*)newName;
-@property(readwrite) int max;
+@interface JSActionButton : JSAction
+
+- (id)initWithName:(NSString *)name idx:(int)idx max:(int)max;
+
+@property (assign) int max;
 
 @end
index 98a186a8b49d30d68eb6c3f0bd9e0697eeb1d1a5..c358c8dfc873d38d5ca56bbc741c56f3eac9cf15 100644 (file)
@@ -5,27 +5,31 @@
 //  Created by Sam McCall on 5/05/09.
 //
 
-@implementation JSActionButton
+@implementation JSActionButton {
+    BOOL active;
+}
 
-@synthesize max, active;
+@synthesize max;
+@synthesize active;
 
--(id)initWithIndex: (int)newIndex andName: (NSString *)newName {
-       if(self= [ super init]) {
-               subActions = NULL;
-               index = newIndex;
-               name = [[NSString alloc] initWithFormat: @"Button %d %@", (index+1), newName];
-       }
-       return self;
+- (id)initWithName:(NSString *)name_ idx:(int)idx max:(int)max_ {
+    if ((self = [super init])) {
+        self.index = idx;
+        self.max = max_;
+        if (name_.length)
+            self.name = [NSString stringWithFormat:@"Button %d - %@", self.index + 1, name_];
+        else
+            self.name = [NSString stringWithFormat:@"Button %d", self.index + 1];
+    }
+    return self;
 }
 
--(id) findSubActionForValue: (IOHIDValueRef) val {
-       if(IOHIDValueGetIntegerValue(val) == max)
-               return self;
-       return NULL;
+- (id)findSubActionForValue:(IOHIDValueRef)val {
+    return (IOHIDValueGetIntegerValue(val) == max) ? self : nil;
 }
 
--(void) notifyEvent: (IOHIDValueRef) value {
-       active = IOHIDValueGetIntegerValue(value) == max;
+- (void)notifyEvent:(IOHIDValueRef)value {
+    active = IOHIDValueGetIntegerValue(value) == max;
 }
 
 @end
index bab893f61e354539ac2b41e7f49c1980ca6275c7..5a1fa22de7aa6bb4ef5f25f5f7c6db357b39a61a 100644 (file)
@@ -7,10 +7,9 @@
 //
 
 #import <Cocoa/Cocoa.h>
-@class JSAction;
 
-@interface JSActionHat : JSAction {
-       
-}
+#import "JSAction.h"
+
+@interface JSActionHat : JSAction
 
 @end
index 13abcda9d3958a72d347525ad566261345d555fa..4e11b49b4dcfaafd71ba9cd8b8d810c8c5cb0dd1 100644 (file)
@@ -6,85 +6,90 @@
 //
 
 static BOOL active_eightway[36] = {
-NO,  NO,  NO,  NO , // center
-YES, NO,  NO,  NO , // N
-YES, NO,  NO,  YES, // NE
-NO,  NO,  NO,  YES, // E
-NO,  YES, NO,  YES, // SE
-NO,  YES, NO,  NO , // S
-NO,  YES, YES, NO , // SW
-NO,  NO,  YES, NO , // W
-YES, NO,  YES, NO , // NW
+    NO,  NO,  NO,  NO , // center
+    YES, NO,  NO,  NO , // N
+    YES, NO,  NO,  YES, // NE
+    NO,  NO,  NO,  YES, // E
+    NO,  YES, NO,  YES, // SE
+    NO,  YES, NO,  NO , // S
+    NO,  YES, YES, NO , // SW
+    NO,  NO,  YES, NO , // W
+    YES, NO,  YES, NO , // NW
 };
+
 static BOOL active_fourway[20] = {
-NO,  NO,  NO,  NO , // center
-YES, NO,  NO,  NO , // N
-NO,  NO,  NO,  YES, // E
-NO,  YES, NO,  NO , // S
-NO,  NO,  YES, NO , // W
+    NO,  NO,  NO,  NO , // center
+    YES, NO,  NO,  NO , // N
+    NO,  NO,  NO,  YES, // E
+    NO,  YES, NO,  NO , // S
+    NO,  NO,  YES, NO , // W
 };
 
 @implementation JSActionHat
 
-- (id) init {
-       if(self = [super init]) {
-               subActions = @[[[SubAction alloc] initWithIndex: 0 name: @"Up" base: self],
-                                         [[SubAction alloc] initWithIndex: 1 name: @"Down" base: self],
-                                         [[SubAction alloc] initWithIndex: 2 name: @"Left" base: self],
-                                         [[SubAction alloc] initWithIndex: 3 name: @"Right" base: self]];
-               name = @"Hat switch";
-       }
-       return self;
+- (id)init {
+    if ((self = [super init])) {
+        self.subActions = @[[[SubAction alloc] initWithIndex: 0 name: @"Up" base: self],
+                            [[SubAction alloc] initWithIndex: 1 name: @"Down" base: self],
+                            [[SubAction alloc] initWithIndex: 2 name: @"Left" base: self],
+                            [[SubAction alloc] initWithIndex: 3 name: @"Right" base: self]];
+        // TODO(jfw): Should have an indexed name, like everything else.
+        self.name = @"Hat switch";
+    }
+    return self;
 }
 
--(id) findSubActionForValue: (IOHIDValueRef) value {
-       int parsed = IOHIDValueGetIntegerValue(value);
-       if(IOHIDElementGetLogicalMax(IOHIDValueGetElement(value)) == 7) {
-               // 8-way
-               switch(parsed) {
-                       case 0: return subActions[0];
-                       case 4: return subActions[1];
-                       case 6: return subActions[2];
-                       case 2: return subActions[3];
-               }
-       } else  if(IOHIDElementGetLogicalMax(IOHIDValueGetElement(value)) == 8) {
-               // 8-way
-               switch(parsed) {
-                       case 1: return subActions[0];
-                       case 5: return subActions[1];
-                       case 7: return subActions[2];
-                       case 3: return subActions[3];
-               }
-       } else if(IOHIDElementGetLogicalMax(IOHIDValueGetElement(value)) == 3) {
-                       // 4-way
-               switch(parsed) {
-                       case 0: return subActions[0];
-                       case 2: return subActions[1];
-                       case 3: return subActions[2];
-                       case 1: return subActions[3];
-               }
-       } else if(IOHIDElementGetLogicalMax(IOHIDValueGetElement(value)) == 4) {
-               // 4-way
-               switch(parsed) {
-                       case 1: return subActions[0];
-                       case 3: return subActions[1];
-                       case 4: return subActions[2];
-                       case 2: return subActions[3];
-               }
-       }
-       return NULL;
+- (id)findSubActionForValue:(IOHIDValueRef)value {
+    int parsed = IOHIDValueGetIntegerValue(value);
+    switch (IOHIDElementGetLogicalMax(IOHIDValueGetElement(value))) {
+        case 7: // 8-way switch, 0-7.
+            switch (parsed) {
+                case 0: return self.subActions[0];
+                case 4: return self.subActions[1];
+                case 6: return self.subActions[2];
+                case 2: return self.subActions[3];
+                default: return nil;
+            }
+        case 8: // 8-way switch, 1-8 (neutral 0).
+            switch (parsed) {
+                case 1: return self.subActions[0];
+                case 5: return self.subActions[1];
+                case 7: return self.subActions[2];
+                case 3: return self.subActions[3];
+                default: return nil;
+            }
+        case 3: // 4-way switch, 0-3.
+            switch (parsed) {
+                case 0: return self.subActions[0];
+                case 2: return self.subActions[1];
+                case 3: return self.subActions[2];
+                case 1: return self.subActions[3];
+                default: return nil;
+            }
+        case 4: // 4-way switch, 1-4 (neutral 0).
+            switch (parsed) {
+                case 1: return self.subActions[0];
+                case 3: return self.subActions[1];
+                case 4: return self.subActions[2];
+                case 2: return self.subActions[3];
+                default: return nil;
+            }
+        default:
+            return nil;
+    }
 }
 
--(void) notifyEvent: (IOHIDValueRef) value {
-       int parsed = IOHIDValueGetIntegerValue(value);
-       int size = IOHIDElementGetLogicalMax(IOHIDValueGetElement(value));
-       if(size == 7 || size == 3) {
-               parsed++;
-               size++;
-       }
-       BOOL* activeSubactions = (size == 8) ? active_eightway : active_fourway;
-       for(int i=0; i<4; i++)
-               [subActions[i] setActive: activeSubactions[parsed * 4 + i]];
+- (void)notifyEvent:(IOHIDValueRef)value {
+    int parsed = IOHIDValueGetIntegerValue(value);
+    int size = IOHIDElementGetLogicalMax(IOHIDValueGetElement(value));
+    // Skip first row in table if 0 is not neutral.
+    if (size & 1) {
+        parsed++;
+        size++;
+    }
+    BOOL *activeSubactions = (size == 8) ? active_eightway : active_fourway;
+    for (int i = 0; i < 4; i++)
+        [self.subActions[i] setActive:activeSubactions[parsed * 4 + i]];
 }
 
 @end
index 6d38ce1e27c1349f649c723882f7ab31bf771091..04c5686f42d5ec955c934ddbf791a48d1504f636 100644 (file)
@@ -6,7 +6,8 @@
 //  Copyright 2009 University of Otago. All rights reserved.
 //
 
-#import <Cocoa/Cocoa.h>
+#import <Foundation/Foundation.h>
+#import <IOKit/hid/IOHIDBase.h>
 
 @class JSAction;
 
index 9f3c8265688564acc0d6c1df615e55fc7b7cb2a4..a65304e304b84a434a06dd31fa64a830e9d34fdb 100644 (file)
@@ -5,6 +5,8 @@
 //  Created by Sam McCall on 4/05/09.
 //
 
+#import "Joystick.h"
+
 static NSArray *ActionsForElement(IOHIDDeviceRef device, id base) {
     CFArrayRef elements = IOHIDDeviceCopyMatchingElements(device, NULL, kIOHIDOptionsTypeNone);
     NSMutableArray *children = [NSMutableArray arrayWithCapacity:CFArrayGetCount(elements)];
@@ -21,30 +23,31 @@ static NSArray *ActionsForElement(IOHIDDeviceRef device, id base) {
         int min = IOHIDElementGetPhysicalMin(element);
         CFStringRef elName = IOHIDElementGetName(element);
         
-        JSAction *action = NULL;
+        JSAction *action = nil;
         
-        if(!(type == kIOHIDElementTypeInput_Misc
-             || type == kIOHIDElementTypeInput_Axis
-             || type == kIOHIDElementTypeInput_Button))
-            continue;
+        if (!(type == kIOHIDElementTypeInput_Misc
+              || type == kIOHIDElementTypeInput_Axis
+              || type == kIOHIDElementTypeInput_Button))
+             continue;
         
         if (max - min == 1 || usagePage == kHIDPage_Button || type == kIOHIDElementTypeInput_Button) {
-            action = [[JSActionButton alloc] initWithIndex:buttons++ andName:(__bridge NSString *)elName];
-            [(JSActionButton*)action setMax:max];
+            action = [[JSActionButton alloc] initWithName:(__bridge NSString *)elName
+                                                      idx:buttons++
+                                                      max:max];
         } else if (usage == kHIDUsage_GD_Hatswitch) {
             action = [[JSActionHat alloc] init];
+        } else if (usage >= kHIDUsage_GD_X && usage <= kHIDUsage_GD_Rz) {
+            // TODO(jfw): Scaling equation doesn't seem right if min != 0.
+            action = [[JSActionAnalog alloc] initWithIndex:axes++
+                                                    offset:-1.f
+                                                     scale:2.f / (max - min)];
         } else {
-            if (usage >= kHIDUsage_GD_X && usage <= kHIDUsage_GD_Rz) {
-                action = [[JSActionAnalog alloc] initWithIndex: axes++];
-                [(JSActionAnalog*)action setOffset:(double)-1.0];
-                [(JSActionAnalog*)action setScale:(double)2.0/(max - min)];
-            } else
-                continue;
+            continue;
         }
         
-        [action setBase:base];
-        [action setUsage:usage];
-        [action setCookie:IOHIDElementGetCookie(element)];
+        // TODO(jfw): Should be moved into better constructors.
+        action.base = base;
+        action.cookie = IOHIDElementGetCookie(element);
         [children addObject:action];
     }
     return children;
index f0530dcafc4ffe4d8ba01393498d23da8c30d785..c698c17679c5a0208bd3d5f6dc80ca2bef266951 100644 (file)
@@ -8,9 +8,10 @@
 
 #import <Cocoa/Cocoa.h>
 #import <IOKit/hid/IOHIDLib.h>
+
 @class Joystick;
+@class JSAction;
 @class ConfigsController;
-
 @class TargetController;
 
 @interface JoystickController : NSObject {
@@ -31,9 +32,9 @@
 -(void) setup;
 -(Joystick*) findJoystickByRef: (IOHIDDeviceRef) device;
 
-@property(strong, readonly) id selectedAction;
-@property(readonly) NSMutableArray *joysticks;
-@property(readonly) NSMutableArray *runningTargets;
-@property(readwrite) BOOL frontWindowOnly;
+@property (readonly) JSAction *selectedAction;
+@property (readonly) NSMutableArray *joysticks;
+@property (readonly) NSMutableArray *runningTargets;
+@property (readwrite) BOOL frontWindowOnly;
 
 @end
index 389532850b4925c419cb084fa13e3c975ddeff2f..54734a8f7598165272f8351ef193520796a07118 100644 (file)
@@ -5,7 +5,7 @@
 //  Created by Sam McCall on 4/05/09.
 //
 
-#import "CoreFoundation/CoreFoundation.h"
+#import "JoystickController.h"
 
 @implementation JoystickController
 
@@ -174,15 +174,12 @@ static void remove_callback(void *ctx, IOReturn inResult, void *inSender, IOHIDD
     CFRunLoopAddTimer(CFRunLoopGetCurrent(), timer, kCFRunLoopDefaultMode);
 }
 
--(id) determineSelectedAction {
-       id item = [outlineView itemAtRow: [outlineView selectedRow]];
-       if(!item)
-               return NULL;
-       if([item isKindOfClass: [JSAction class]] && [item subActions] != NULL)
-               return NULL;
-       if([item isKindOfClass: [Joystick class]])
-               return NULL;
-       return item;
+- (JSAction *)selectedAction {
+    id item = [outlineView itemAtRow:outlineView.selectedRow];
+       if ([item isKindOfClass: [JSAction class]] && ![item subActions])
+               return item;
+    else
+        return nil;
 }
 
 /* outline view */
@@ -227,9 +224,8 @@ static void remove_callback(void *ctx, IOReturn inResult, void *inSender, IOHIDD
 
 - (void)outlineViewSelectionDidChange: (NSNotification*) notification {
        [targetController reset];
-       selectedAction = [self determineSelectedAction];
        [targetController load];
-       if(programmaticallySelecting)
+       if (programmaticallySelecting)
                [targetController focusKey];
        programmaticallySelecting = NO;
 }
index d09fa3a472d4ac64139fd53f1eea0980e4430f0d..51a85098a4da667099669faba5c77fab6dc83194 100644 (file)
@@ -7,20 +7,13 @@
 //
 
 #import <Cocoa/Cocoa.h>
-@class JSAction;
 
-@interface SubAction : NSObject {
-       JSAction *base;
-       NSString *name;
-       int index;
-       BOOL active;
-}
+#import "JSAction.h"
 
--(id) initWithIndex:(int)newIndex name: (NSString*)newName  base: (JSAction*)newBase;
+@interface SubAction : JSAction
 
-@property(readwrite, strong) JSAction* base;
-@property(readwrite, copy) NSString* name;
-@property(readwrite) int index;
-@property(readwrite) BOOL active;
+- (id)initWithIndex:(int)newIndex name:(NSString *)newName  base:(JSAction *)newBase;
+
+@property (assign) BOOL active;
 
 @end
index cfca78b3d6c7bf1bb546a1d58fe0a4fe1b734560..07011ca90427f6ac09a877ee728ba42611c2a2ba 100644 (file)
@@ -5,21 +5,27 @@
 //  Created by Sam McCall on 5/05/09.
 //
 
-@implementation SubAction
+// TODO(jfw): This class is useless and can just be replaced w/ JSAction probably.
 
-@synthesize base, name, index, active;
+#import "SubAction.h"
 
--(id) initWithIndex:(int)newIndex name: (NSString*)newName base: (JSAction*)newBase {
-       if(self = [super init]) {
-               [self setName: newName];
-               [self setBase: newBase];
-               [self setIndex: newIndex];
+@implementation SubAction {
+       BOOL active;
+}
+
+@synthesize active;
+
+- (id)initWithIndex:(int)newIndex name:(NSString *)newName base:(JSAction *)newBase {
+       if ((self = [super init])) {
+        self.name = newName;
+        self.base = newBase;
+        self.index = newIndex;
        }
        return self;
 }
 
 -(NSString*) stringify {
-       return [[NSString alloc] initWithFormat: @"%@~%d", [base stringify], index];
+       return [[NSString alloc] initWithFormat: @"%@~%d", [base stringify], self.index];
 }
 
 @end