[Adium-devl] Importer API

[disposable at infinitenexus.com]

Inline self-correction follows.

On Apr 10, 2007, at 4:06 PM, disposable at infinitenexus.com wrote:

>
> On Apr 10, 2007, at 2:18 PM, Peter Hosey wrote:
>
>> On Apr 10, 2007, at 11:59:59, Graham Booker wrote:
>>> //These return false if they failed for some reason
>>> - (BOOL)importAccounts;
>>> - (BOOL)importStatus;
>>> //Note, the log import will take place in a thread.
>>> - (BOOL)importLogs;
>>
>> I suggest returning NSError instead.
>>
>
> I'd say this would be even better as it matches patterns (at least in
> my experience):
>
>   -(BOOL)importAccounts:(NSError **)err;

-(BOOL)importAccountsWithError:(NSError **)err;

/* Course-grained BOOL return is used for testing success, the error  
pointer is set if any error occurs as seen in AppKit. */

>
> < / pedantic >
>
> - brian 'bgannin' ganninger
>
>
>>
>> _______________________________________________
>> Adium-devl mailing list
>> Adium-devl at adiumx.com
>> http://adiumx.com/mailman/listinfo/adium-devl_adiumx.com
>
>
> _______________________________________________
> Adium-devl mailing list
> Adium-devl at adiumx.com
> http://adiumx.com/mailman/listinfo/adium-devl_adiumx.com
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://adium.im/pipermail/devel_adium.im/attachments/20070410/91df3fc3/attachment-0001.html>