<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
<blockquote type="cite">
<pre wrap="">for phpdoc // @todo wouldn't work... /** @todo would word */</pre>
</blockquote>
Most IDEs recognize <br>
<br>
// TODO: Whatevs. <br>
<br>
as valid todos. So for inline todos, those seem the way to go. For
Doxygen, you're right that we should definitely use @todo. Are you
suggesting that we use /* @todo */ for inline todos as well and Doxygen
will pick up on those?<br>
<pre class="moz-signature" cols="72">Rob Roy Barreca
Founder and COO
Electronic Insight Corporation
<a class="moz-txt-link-freetext" href="http://www.electronicinsight.com">http://www.electronicinsight.com</a>
<a class="moz-txt-link-abbreviated" href="mailto:rob@electronicinsight.com">rob@electronicinsight.com</a></pre>
<br>
<br>
Darrel O'Pry wrote:
<blockquote cite="mid1168281116.32537.95.camel@localhost.localdomain"
type="cite">
<pre wrap="">On Mon, 2007-01-08 at 10:43 -0500, Angela Byron wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Excellent! Thanks for the summary, Derek!!
</pre>
</blockquote>
<pre wrap=""><!---->
<snip />
</pre>
<blockquote type="cite">
<pre wrap="">Now, just one (technical) question.
Does this:
</pre>
<blockquote type="cite">
<pre wrap=""> // @todo These queries are very expensive, see <a class="moz-txt-link-freetext" href="http://drupal.org/">http://drupal.org/</a>
node/105639.
</pre>
</blockquote>
<pre wrap="">...actually work and get parsed by doxygen?? I thought doxygen only
worked on function headers. This would make @todo less useful, imo,
since:
// TODO: this can be an expensive query. Perhaps only execute it
every x minutes. Requires investigation into cache expiration.
if ($user->uid) {
db_query("UPDATE {users} SET access = %d WHERE uid = %d", time
(), $user->uid);
}
Tells me that the queries immediately following that line are
expensive, vs.
/**
* Reads a session variable.
*
* @todo the update query used here can be expensive....
*/
function sess_read($key) {
Anyone know?
</pre>
</blockquote>
<pre wrap=""><!---->
for phpdoc // @todo wouldn't work... /** @todo would word */
</pre>
<blockquote type="cite">
<pre wrap="">Also, as Robert brought up, // TODO, etc. have support in IDEs (Zend,
etc.). Can someone check if @todo does as well?
-Angie
</pre>
</blockquote>
<pre wrap=""><!---->
</pre>
</blockquote>
</body>
</html>