This code is actually from Marcel, but I was the one who put it in there with copy-paste method :) Ok, I'll fix it (and also I'll be more careful next time to review it with the coding style guidelines).<div><br></div>

<div>Marty<br><br><div class="gmail_quote">On Fri, Nov 19, 2010 at 16:13, Andi Clemens <span dir="ltr"><<a href="mailto:andi.clemens@gmx.net">andi.clemens@gmx.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Hi,<br>
<br>
the following code is not understandable, it is not clear wether the following<br>
lines are ment to be IN the foreach statement or outside of it:<br>
<br>
GSOC2010 branch:<br>
libs/database/imageinfo.cpp (lines 672-679).<br>
<br>
It is not clear if the second if-statement belongs to the foreach loop. If it<br>
does, the code is wrong.<br>
<br>
This is why I beg you (:-)):<br>
Please use curly brackets, ALWAYS!<br>
At work we had so much trouble because of someone avoiding them for single<br>
line code blocks, our customer finally found a bug because of this (which is<br>
BAD for a software company, the customer shouldn't find this :-)).<br>
<br>
The problem is that whenever you add code, chances are that you get misleading<br>
/ wrong code like the example above.<br>
<br>
So please, add curly brackets, everytime. It is not Python were indention is<br>
clear enough, in the above example it is not and the code might be wrong.<br>
<br>
Martin,<br>
<br>
since you made the change, could you please fix it and set the curly brackets<br>
accordingly?<br>
<font color="#888888"><br>
Andi<br>
</font></blockquote></div><br></div>