List Info

Thread: +1 for PHP::PHP_GenDocBlock




+1 for PHP::PHP_GenDocBlock
user name
2007-05-20 07:12:25
Mark Wiesemann (http://pear.php.ne
t/user/wiesemann) has voted +1 on the proposal for
PHP::PHP_GenDocBlock.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=481

Vote information:
http://pear.php.net/pepr/pepr-vote-show.p
hp?id=481&handle=wiesemann

This vote is conditional. The condition is:

(comments based on the gendocblock.0.3.tgz file)
- linebreak between "class" and "{" is
still missing
- why is GenDocBlockTest.php in the PHP/ directory and not
in the tests/ directory?
- to be honest, the run() method is unreadable:
  * some people already complained about the and's and or's
  * the comments after the lines "hide" the and's
and or's IMHO
  => I would recommend to put the comment into separate
lines and
     to remove the and's and or's
- the "" error suppression calls are still there, but
not needed
  (they can make debugging very complicated)
- comments for run() are also valid for e.g. alignTags();
instead of
---
$this->tagPadding = max(array_map('strlen', $tags)); //
finds out the length of the longest tag
---
I would recommend to use
---
// finds out the length of the longest tag
$this->tagPadding = max(array_map('strlen', $tags));
---

-- 
Sent by PEPr, the automatic proposal system at http://pear.php.net

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub
.php


[1]

about | contact  Other archives ( Real Estate discussion Medical topics )