Vote Details for "PEAR_Size" by izi

» Details
  • Voter: David Jean Louis 
  • Vote: +1 (not conditional)
  • Reviews: Deep source review
» Comment
I also get the notices reported by Christian.

Some notes:
- I don't like very much these setters that just assign the value to a public var (setVerbose, setHumanReadable...etc), they are quite useless imho;
- you should declare these public class variables at class level (eg. public $verbose = false;);
- imho HTML_Table and Console_Table should be optional requirements (if Console_Table is installed use it and if not the result is tableless).

Otherwise nice package.