Yeah, I agree.  Too often TODOs are put there just to tell reviewers that you are doing something short-sighted, but that you recognize it.  Really reviewers should be enforcing that there is a bug to track that item, or requiring it to just be done in the initial version.  There is also a use of a TODO which I would probably refer to as a "FUTURE" tag, which is to say: someday, we may plan on implementing X, and if we do that, the change would go here.  One could argue that those are useful comments to have in the code, but do not require bugs, as they do not actually represent deficiencies that could currently be fixed.  Perhaps we should use something other than TODO in that scenario.<div>

<br></div><div>Dan<br><br><div class="gmail_quote">On Mon, Mar 4, 2013 at 3:26 AM, Thierry Carrez <span dir="ltr"><<a href="mailto:thierry@openstack.org" target="_blank">thierry@openstack.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">Nachi Ueno wrote:<br>
> Hi quantum folks<br>
><br>
> I would like to propose new #TODO format.<br>
><br>
> #TODO(name) bug or lb [launchpad bug id] comment<br>
><br>
> example: #TODO(nati) bug100000  fix me !!<br>
><br>
> There are 114 todos in code. I'm not sure they have bug report<br>
>  but it's worth tracking it.<br>
> <a href="http://paste.openstack.org/show/32495/" target="_blank">http://paste.openstack.org/show/32495/</a><br>
><br>
> I'm not suggesting fix all existing #TODOs ,but how about<br>
> to this format for new #TODOs?<br>
<br>
</div>Ensuring we have properly-prioritized bugs for #TODOs sounds like a<br>
great idea. Not sure that's just a quantulm thing though. I'd love to<br>
see the new format adopted elsewhere.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Thierry Carrez (ttx)<br>
Release Manager, OpenStack<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>Dan Wendlandt <div>Nicira, Inc: <a href="http://www.nicira.com" target="_blank">www.nicira.com</a><br><div>twitter: danwendlandt<br>

~~~~~~~~~~~~~~~~~~~~~~~~~~~<br></div></div>
</div>