On Friday, July 06, 2012 11:34:01 AM Per Bothner wrote:
> On 07/06/2012 10:05 AM, Dan Bernstein wrote:
> > It appears that lately most WebCore change log entires don’t include any
> > comments on individual functions. An overall description of the change at
> > the top of the change log entry is valuable, but it is no substitute for
> > describing the changes to each function. Good function-level comments are
> > useful both while reviewing a patch and while revisiting existing code.
> > Personally, I find that writing the function-level comments helps me a
> > lot in reviewing my own patches before I post them.
> You forget there is a WebKit policy of not writing comments
> or otherwise documenting the code.

I think he meant function level comments on ChangeLogs, not on the code
itself.

> Or at least that's what it looks like. :-(
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.19 (GNU/Linux)

iEYEABECAAYFAk/3MI8ACgkQ+tuOMzntPWmSvwCfZX7mSpblM8xqtVYfMagmkS1V
6U4AoJOVtSW5QtfGk0C10boluUg+jzoe
vPjB
-----END PGP SIGNATURE-----
_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

Reply via email to