Changeset 51 for trunk/Cocoa/Pester/Source/PSAlarms.m

Timestamp:

11/18/02 08:57:41 (21 years ago)

Author:

Nicholas Riley

Message:

Alarms.nib: Removed horizontal scroll bar. Turned on grid. Set delegate to NJRTableDelegate instead of PSAlarmSetController.

NJRTableDelegate: In general, made functional (was previously unused). Fixed MyCompanyName. Changed ORDER_BY_CONTEXT to use key-value coding instead of assuming data consists of a dictionary of dictionaries. Added sorting support (reorderedData, replaces oData) with autosave support for sort context. Added _positionTypeSelectDisplay, which adjusts position and justification of type select display control based on the current sort column. Added support for reverse sorting in type select string. Use table data source instead of sorted data so text matches as displayed (this will break with non-text cells...).

NJRTableView: Adapted from iTableView (Jaguar table alternate table background color), TableTester (most everything else) and NJROutlineView (keyDown, moveToBeginning/EndOfDocument). Support for type selection, delete shortcut for row deletion, and iTunes-alike background colors and frame.

NSCharacterSet-NJRExtensions: Moved _typeSelectSet from NJROutlineView as it's now shared with NJRTableView. Still need to factor NJROutlineView as embedded in HostLauncher some day.

PSAlarm: Reorganized, renamed and categorized methods. Added time accessor for the benefit of sorting. Renamed compare: to compareDate: for clarity. Added compareMessage:, though it's currently unused. Renamed cancel to cancelTimer for clarity.

PSAlarmSetController: More fun with initial first responder on window show/hide; still need to work around bug properly (subclass NSComboBox?) and fix it for real. As is, works for OS X 10.1.

PSAlarms: Added alarms accessor, returning alarm array. Fixed memory leak on successful alarm removal (oops). Added removeAlarms:, needed with sorted alarm list.

PSAlarmsController: Set window resize increment. Changes to table delegate methods to use reordered alarm list. Register for NSTableViewSelectionDidChangeNotification now we're no longer the table view delegate. Fixed autoselection in alarmsChanged by using data reordering support in NJRTableView. Implement NJRTableViewDataSource to permit deletion from table view.

Pester.pbproj: Added new files.

Read Me.rtfd: Added TableTester/iTableView acknowledgements. Updated release notes.

File:

• trunk/Cocoa/Pester/Source/PSAlarms.m (modified) (5 diffs)

trunk/Cocoa/Pester/Source/PSAlarms.m

@@ -46 +46 @@
     nextAlarm = nil;
     // sort alarms so earliest is first
-    [alarms sortUsingSelector: @selector(compare:)];
+    [alarms sortUsingSelector: @selector(compareDate:)];
     // find first un-expired alarm
     e = [alarms objectEnumerator];
@@ -126 +126 @@
 }
 
+- (NSArray *)alarms;
+{
+    return alarms;
+}
+
 - (int)alarmCount;
 {
@@ -138 +143 @@
 - (void)removeAlarmAtIndex:(int)index;
 {
-    [(PSAlarm *)[alarms objectAtIndex: index] cancel];
+    [(PSAlarm *)[alarms objectAtIndex: index] cancelTimer];
     [alarms removeObjectAtIndex: index];
 }
@@ -151 +156 @@
         while ( (n = [e nextObject]) != nil) {
             alarmIndex = [n intValue];
-            [(PSAlarm *)[alarms objectAtIndex: alarmIndex] cancel];
+            [(PSAlarm *)[alarms objectAtIndex: alarmIndex] cancelTimer];
             indexArray[i] = alarmIndex;
             i++;
         }
         [alarms removeObjectsFromIndices: indexArray numIndices: indexCount];
+        free(indexArray); indexArray = NULL;
         [self _changed];
     NS_HANDLER
@@ -164 +170 @@
 }
 
+- (void)removeAlarms:(NSSet *)alarmsToRemove;
+{
+    NSEnumerator *e = [alarms objectEnumerator];
+    PSAlarm *alarm;
+    NSMutableArray *indices = [NSMutableArray arrayWithCapacity: [alarmsToRemove count]];
+    int alarmIndex = 0;
+
+    while ( (alarm = [e nextObject]) != nil) {
+        if ([alarmsToRemove containsObject: alarm])
+            [indices addObject: [NSNumber numberWithInt: alarmIndex]];
+        alarmIndex++;
+    }
+    [self removeAlarmsAtIndices: indices];
+}
+
 @end