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

Timestamp:

11/28/07 01:29:26 (16 years ago)

Author:

Nicholas Riley

Message:

English.lproj/Alarms.nib: Specify alternating row coloring in the nib,
now we're 10.4+.

English.lproj/InfoPlist.strings: Updated for 1.1b6.

English.lproj/Localizable.strings: Quote alarm message in pretty
description (used in tooltip). Change voice error now it no longer
incorporates OSStatus.

English.lproj/MainMenu.nib: Add speech prefs again; turn repetitions
field into a NJRValidatingField and hook up its delegate.

Info-Pester.plist: Updated for 1.1b6.

NJRHotKey.m: Switch to new Objective-C exception style.

NJRIntervalField.[hm]: Now a subclass of NJRValidatingField.

NJRTableDelegate.m: Get rid of our own tooltip support as NSTableView
now supports them (though with a minor visual glitch on the first
tooltip).

NJRTableView.[hm]: Remove tooltip support. Remove alternating row
coloring support.

NJRValidatingField.[hm]: Contains validation sheet stuff from
NJRIntervalField.

NJRVoicePopUpButton.[hm]: Switch to NSSpeechSynthesizer.

PSAlarm.m: Quote alarm message in pretty description (used in
tooltip). Fix repeating alarms not restoring as repeating if they
didn't expire while Pester was not running. No longer set timer on
Pester 1.0 alarm import, to help make importing atomic.

PSAlarmSetController.[hm]: Use NJRValidatingField for repetitions
field. Switch to new Objective-C exception style. Fix validation
issues on in/at changing. Temporary changes to restore speech support
and allow the sound popup to be removed entirely from the nib (rather
than being dragged out of the visible area, as it was in 1.1b5).
Changes for NSSpeechSynthesizer, which uses different voice names.

PSAlarms.m: Switch to new Objective-C exception style. Fix
duplication and error handling in Pester 1.0 alarm import, making
atomic.

PSAlarmsController.m: Use new tooltip support (since it's implemented
in the delegate rather than the data source, we have to proxy it).

PSAlerts.m: Wrap initialization in exception block so we don't leak.

PSApplication.m: Switch to new Objective-C exception style.

PSMediaAlert.m: Clamp repetitions at 1..99 so the user can't type an
invalid value, then quit and have it saved.

PSSpeechAlert.[hm]: Switch to NSSpeechSynthesizer. Throw an
intelligible exception if the voice is unavailable.

PSTimer.m: Switch to new Objective-C exception style.

Pester.xcodeproj: Remove VERSION generation; rename targets to be more
understandable.

Read Me.rtfd: Updated for 1.1b6.

SUSpeaker.[hm]: Gone in switch to NSSpeechSynthesizer.

VERSION: Gone - we use agvtool for everything now.

Updates/release-notes.html: Updated for 1.1b6.

Updates/updates.xml: Updated for 1.1b6.

package-Pester.sh: Use agvtool to get version. Atomically update
file on Web server to avoid partial downloads.

File:

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

trunk/Cocoa/Pester/Source/PSAlarms.m

@@ -178 +178 @@
     unsigned indexCount = [indices count], i = 0, alarmIndex;
     unsigned *indexArray = (unsigned *)malloc(indexCount * sizeof(unsigned));
-    NS_DURING
+    @try {
         while ( (n = [e nextObject]) != nil) {
             alarmIndex = [n intValue];
@@ -186 +186 @@
         }
         [alarms removeObjectsFromIndices: indexArray numIndices: indexCount];
+    } @finally {
         free(indexArray); indexArray = NULL;
         [self _changed];
-    NS_HANDLER
-        free(indexArray);
-        [self _changed];
-        [localException raise];
-    NS_ENDHANDLER
+    }
 }
 
@@ -314 +311 @@
     NSData *alarmData;
     PSAlarm *alarm;
-    while ( (alarmData = [e nextObject]) != nil) {
-        NS_DURING
-            alarm = [NSUnarchiver unarchiveObjectWithData: alarmData];
-        NS_HANDLER
-            alarm = nil;
-            // XXX
-        NS_ENDHANDLER
-        if (alarm != nil)
-            [alarms addObject: alarm];
+    NSMutableArray *importedAlarms = [[NSMutableArray alloc] initWithCapacity: [alarmsData count]];
+    @try {
+        while ( (alarmData = [e nextObject]) != nil) {
+            alarm = [NSUnarchiver unarchiveObjectWithData: alarmData];
+            if (alarm == nil)
+                @throw [NSException exceptionWithName: NSInternalInconsistencyException reason: @"Failed to decode Pester 1.0 alarm." userInfo: nil];
+            [importedAlarms addObject: alarm];
+            if (![alarm setTimer]) // expired
+                [alarms addObject: alarm];
+        }
+    } @catch (NSException *exception) {
+        [self removeAlarms: [NSSet setWithArray: importedAlarms]];
+        @throw;
+    } @finally {
+        [importedAlarms release];
     }
 }