On Sun, Apr 09, 2006 at 04:15:10PM +0200, Erik Johansson wrote:
> I've created two versions of the patch. The only difference between the two 
> is 
> that patch 2 moves the logic for switching tabs to the CETabBar instead of 
> having the same code in three places (CETabBar, CETabWidget and 
> UserEventTabDlg).

I applied the 2nd patch to SVN. It was fine, but I just modified some
braces.

In general, I prefer:

if (x)
{
    //
}
else
{
    //
}

There is a README.CodingStyle in licq/doc if you want to see more.

Thanks,

Jon


-- 
________________________________________________________
Jon Keating                ICQ: 16325723
[EMAIL PROTECTED]               MSN: [EMAIL PROTECTED]
http://www.licq.org        GPG: 2290A71F
http://www.thejon.org      HOME: Minamiashigara, Japan

Attachment: pgpQMNqmtHUtt.pgp
Description: PGP signature

Reply via email to