On 07/06/2012 01:02 PM, Eric Seidel wrote:
You began by thread-jacking Dan's discussion of ChangeLogs, and now
appear to be attacking a seasoned contributors polite response with
sarcasm and derision.
Sorry - it wasn't meant to be a thread-jacking, but the issue of
commenting in WebKit hit a ne
Per:
You began by thread-jacking Dan's discussion of ChangeLogs, and now
appear to be attacking a seasoned contributors polite response with
sarcasm and derision.
I recommend you soften your words, and try again on another thread.
Commenting, or lack there of, has been discussed at great length
On 07/06/2012 11:41 AM, Ryosuke Niwa wrote:
Indeed, we try to avoid adding comments as much as possible since
comments tend to get out-of-date very quickly, we don't want to be
spending all our time updating comments.
Heavens forbid that someone who actually understands the code should have
to
On Fri, Jul 6, 2012 at 11:42 AM, Dana Jansens wrote:
> On Fri, Jul 6, 2012 at 1:05 PM, 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 entr
: Friday, July 06, 2012 02:42 PM
To: Dan Bernstein
Cc: WebKit Development
Subject: Re: [webkit-dev] Please include function-level comments in change log
entries
On Fri, Jul 6, 2012 at 1:05 PM, Dan Bernstein
mailto:m...@apple.com>> wrote:
It appears that lately most WebCore change log entires
On Fri, Jul 6, 2012 at 1:05 PM, 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 c
On Fri, Jul 6, 2012 at 11:34 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 valua
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
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 eac
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 us
10 matches
Mail list logo