On Monday 25 June 2007 05:37:30 Will Stephenson wrote:
> On Saturday 23 June 2007, Guillermo A. Amaral said:
> >  Hi! well I replaced KopeteMimeSourceFactory and since it was located in
> > libkopete I just swapped it in Ala Indiana Jones (tm) Raiders of the lost
> > ark.
>
> Nice analogy!
>
> The patch looks fine after a cursory inspection, but could you indent
> consistently?  Some of the nested if()s are not indented from their
> parents.
>
> Will

  Sorry about that since some of the code comes from the original mime source 
factory and I wanted to mess as little as I could with the prior code I did  
not notice it, I will fix right away.

Cheers

-- 
Guillermo Antonio Amaral Bastidas
# Free & Open Source Software Advocate
$ irc: [EMAIL PROTECTED]
@ blog: http://blog.guillermoamaral.com/
@ site: http://www.guillermoamaral.com/
% gpg: http://downloads.guillermoamaral.com/public.asc

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel

Reply via email to