swick/openmw — comment on commit 16c214a17af6c4506d89bad139a4d66e7423f180
File:  line 0
By lgromanowski:

[quote]would you change the code so it would not contain multiple returns?
Can you explane me why? It would add many new levels of indenting.[/quote]

It's easier to follow the code when you have only one place when function exit.

[quote]line.length() check could be moved from two places to just one, i.e
Does not work. If isRegEntry is true and we have an empty line we have to break the loop. It would search threw the entire registry if there is no "Installed Path" entry. [/quote]
Sorry, this was probably not visible on first sight, but I left ellipsis for "else { break; }".

Ok, probably it's nitpicking - if Zini says that code is ok, then it's ok for me too.