[Adium-devl] Importer API

Graham Booker gbooker at cod3r.com
Thu Apr 12 18:13:08 UTC 2007


On Apr 12, 2007, at 11:24 AM, disposable at infinitenexus.com wrote:

>
> On Apr 12, 2007, at 11:07 AM, Graham Booker wrote:
>
>> <snip>
>> /*
>>  * Sets the current progress for a progress bar
>>  * @param current the log number just imported
>>  * @param total the total number of logs
>>  */
>> - (void)importedLog:(int)current of:(int)total;
>
> This seems odd to me in phrasing, as it's much like the above but  
> doesn't actually do any importing. The display of progress may also  
> include a status text line which isn't accounted for. How about:
>
> /* @param logPath the path (full or shortened) of the currently  
> importing file, for possible display */
>
> -(void)updateLogProgress:(int)current outOf:(int)total forFile: 
> (NSString *)logPath;
>

Agreed.  Changed to use a generic message instead of a file (in case  
the importer wants to say something else)

-(void)updateLogProgress:(int)current outOf:(int)total withStatus: 
(NSString *)message;
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://adium.im/pipermail/devel_adium.im/attachments/20070412/30f20a19/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 1601 bytes
Desc: not available
URL: <http://adium.im/pipermail/devel_adium.im/attachments/20070412/30f20a19/attachment.p7s>


More information about the devel mailing list