tfinnell/bit561 — comment on commit 1c9a74baca26386a00cb0ccba5c6c726f2161386
File:  line 0
By Mutate:

Duly noted....will edit it tonight. Thanks for the feedback.
On Feb 8, 2012 5:26 PM, "Tim Finnell" <
reply@reply.github.com>
wrote:

> The comments at the head of your blocks are great, but if I may suggest
> that you do away with the line comments at the ends of your lines that are
> describing what the line does. For example:
> `$this->lineString = trim($this->lineString); //Trim any whitespace
> from the file line string.`
> This line reads redundantly since there is nothing unique going on in
> here. You might consider adding a line comment for something more complex
> or give a reason why you were adding the particular line rather than
> explaining what it does, but much of Reader.php would actually read
> *easier* without the the multitude of comments.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/tfinnell/bit561/commit/1c9a74baca26386a00cb0ccba5c6c726f2161386#commitcomment-947832
>