<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">
<font style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">There are some changes which have bug descriptions which are complete enough to not need additional comments and any changes to existing scripts should keep this in mind. One way
 to do this is to check for the number of lines/files the diff has. <br>
Konrad <br>
Sent from my BlackBerry on the Rogers Wireless Network</font><br>
 <br>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<font style="font-size:10.0pt;font-family:"Tahoma","sans-serif""><b>From</b>: Timothy Hatcher [mailto:timothy@apple.com]
<br>
<b>Sent</b>: Wednesday, March 21, 2012 05:29 PM<br>
<b>To</b>: webkit-dev@lists.webkit.org <webkit-dev@lists.webkit.org> <br>
<b>Subject</b>: [webkit-dev] ChangeLogs <br>
</font> <br>
</div>
<div>Lately I have observed <a href="http://trac.webkit.org/changeset/111595">more</a> and <a href="http://trac.webkit.org/changeset/111577">more</a> and <a href="http://trac.webkit.org/changeset/111527">more</a> changes going into WebKit that lack any details
 about why a particular change was made. It is intended that the ChangeLog (and commit message) contain some details about your change, not just the bug title and URL.</div>
<div><br>
</div>
<div>The <a href="http://www.webkit.org/coding/contributing.html#changelogs">contributing information</a> on subject is pretty sparse, which has likely caused this problem to manifest. However, the <a href="http://trac.webkit.org/changeset/43259">example ChangeLog</a> linked
 from that page is prime example of what we all should strive for when describing our changes.</div>
<div><br>
</div>
<div>To help curb this lack of detail in ChangeLogs, I <a href="https://bugs.webkit.org/show_bug.cgi?id=81828">propose we add script</a> (or augment an existing script) to check for this missing information and inform the contributor. It is clear not all reviewers
 are asking patch authors to provide this information when reviewing, and such a tool would help enforce it.</div>
<div><br>
</div>
<div apple-content-edited="true">— Timothy Hatcher<br>
<br>
</div>
--------------------------------------------------------------------- <br>
This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.
</body>
</html>