[KPhotoAlbum] Very experimental image-grouping patch
ml at batmat.net
Wed Nov 28 13:11:01 CET 2007
2007/11/28, Jan Kundrát <jkt at gentoo.org>:
> Paul Fleischer wrote:
> > As i wrote in a comment to Bug 152438, I have created a very
> > experimental system for grouping images in KPhotoAlbum.
> Wow, looks good.
I didn't have a look yet. I'll try to try in the next days :-).
Some comments that I have at a first glance:
> b) You've used another whitespace convention, you should've used 4
> spaces per indent
Surely I'm out of the way, moreover I didn't look at the code about it.
But shouldn't this indentation size (done by spaces if I understand
correctly) be put in some global parameter ?
This way we could later consider to allow defining new size for this
Baptiste <Batmat> MATHUS
BMathus at Batmat point net - http://batmat.net
Si chacun de nous a une idée et que nous les partageons, nous
repartirons tous les deux avec deux idées... C'est ça le Libre.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the KPhotoAlbum