[Adium-devl] AIExceptionController

[Colin H Barrett]

On line 43 of AIExceptionController.m we set our exceptionHandling mask. 
I'm concerned in particular about two of the masks: 
NSHandleTopLevelExceptionsMask and NSHandleOtherExceptionMask.

NSHandleOtherExceptionMask is described by the docs[1] as: Handle 
exceptions that will be caught at lower levels. This has been giving 
Andreas some trouble. I've recommended him to remove it in his local copy, 
since it's causing the crash reporter to bring down all of Adium when an 
exception comes back over the Java bridge.

NSHandleTopLevelExceptionsMask I'm a bit more fuzzy on. It seems like it 
could actually potentially be useful, if we want to catch exceptions that 
NSApp itself would normally catch and ignore, but given the seemingly 
broken use of NSHandleOtherExceptionMask, I'm suspicious.

Both of these flags are described in the docs as "debugging aids", and 
don't seem to be recommended for use in a shipping application. Is there a 
particular reason we're setting these flags? If so, can (should?) we work 
around it?

-Colin

[1]: file:///Developer/Documentation/DocSets/com.apple.ADC_Reference_Library.CoreReference.docset/Contents/Resources/Documents/documentation/Cocoa/Conceptual/Exceptions/Tasks/ControllingAppResponse.html