Tony Wu wrote:
It's not a public API and I can not imagine that it will be used by other classes/methods in the fufture. If the second condition were reached one day, there might be something wrong with its caller rather than this method. IMO I would remove the redundant condition in the loop.
Agree .. and throw in a small comment for good measure. Endre.
