<div dir="ltr">Hi,<div><br></div><div>Thank you to develop and share the Lemon graph library.</div><div><br></div><div>By curiosity I have ran the code analyzer from Visual Studio on the Lemon library.</div><div><br></div><div>And it found some strange things in lemon/list_graph.hpp:</div><div><br></div><div>In all the ObserverProxy implementations we can see something like:</div><div>"</div><div><div>virtual void add(const std::vector<Node>& nodes) {</div><div>          for (int i = nodes.size() - 1; i >= 0; ++i) {</div><div>            snapshot.addNode(nodes[i]);</div><div>          }</div><div>        }</div></div><div>"</div><div>The increment step of the loop must be -- and not ++ !!</div><div><br></div><div>This error appears 18 times and must be fixed!</div><div><br></div><div>I'm using Lemon 1.3 and I have checked on the current mercurials files, the error is still there in the trunk.</div><div><br></div><div><div><div class="gmail_signature"><div><font size="2">Regards/Cordialement,</font></div><div><font size="2">Pierre M</font></div></div></div>
</div></div>