Re: Class Variables - no attrAutoCompleteAfter patch

On Thursday 15 February 2007, Andrew Lowe wrote:
> Whoops... I found a mistake - I moved a statement inside an if that I
> should not have
> sorry, attached is the fixed diff

I'll review today. I just have a wish: please do not put the "Author" 
lines everywhere. If the patch goes in and it has enough impact, it 
will be mentioned in the beginning of the file. For small patches, we 
usually mention it in two places:
- the commit log
- the changelog file

From the commit itself it will be clear what was your work.

And if you have a KDE account, than it is clear that you are the one who 
committed. The reason for this is that if two many persons make small 
fixes, the author list would be too big in the file. But of course, if 
the author wants to be mentioned, it has to be done and this is why 
there are other places to be mentioned as well.

For example I did lots of compilation or similar changes in KDE (even 
bugfixes), but if they were big enough I didn't put my name in the 
files I modified. 

Andras

-- 
Quanta Plus developer - http://quanta.kdewebdev.org
K Desktop Environment - http://www.kde.org

_______________________________________________
quanta-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/quanta-devel

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (GNU/Linux)

iD8DBQBF1BLjTQdfac6L/08RAtAbAJ4vcmwqxWElipiWHRfn3G7Dr4WyXACeKaIu
1ZXdk44JwoZuHzPw64HEvkk=
=5R3R
-----END PGP SIGNATURE-----