<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Hi Yaogong,</div><div><br></div><div>There is small redundancy, but it does not introduce any problems. If super::DoPropagateInterest() succeeded, then the processing will not happen in BestRoute at all. If it failed, the foreach loop will try and "skip" (if any exist) the green faces and attempt to send out over an available yellow face.<br><br>This is just a small inefficiency that was made for the sake of simplicity and generality.</div><div><br></div><div>---</div><div>Alex</div><br><div><div>On Apr 12, 2013, at 7:20 AM, Yaogong Wang <<a href="mailto:notifications@github.com">notifications@github.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><p>In BestRoute::DoPropagateInterest(), the green faces are first tried in the call to super::DoPropagateInterest() and tried again in the BOOST_FOREACH loop. The call to super::DoPropagateInterest() appears redundant to me.</p><p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/NDN-Routing/ndnSIM/issues/32">view it on GitHub</a>.<img src="https://github.com/notifications/beacon/-7Cd7fAQSAZl6iIAg3-JrscC6vzH26NO-ATTuvgQBWCdlKwYfDEU_yo42fsYZfZw.gif" height="1" width="1"></p>
</blockquote></div><br></body></html>