<div class="gmail_quote">On Thu, Apr 29, 2010 at 5:44 PM, Alexey Proskuryakov <span dir="ltr">&lt;<a href="mailto:ap@webkit.org">ap@webkit.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="im"><br>
On 26.04.2010, at 22:06, Adam Barth wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
If you make changes to CodeGenerator*.pm, please test your<br>
change using the following command:<br>
<br>
~/git/webkit$ ./WebKitTools/Scripts/run-bindings-tests<br>
</blockquote>
<br>
<br></div>
I don&#39;t understand the utility of such testing. When one edits CodeGenerator*.pm, that&#39;s generally to change its output, so run-bindings-tests tests will of course fail.<br>
<br>
It&#39;s great to test end-to-end behavior, and unit tests can also also useful sometimes, but why test that source code stays byte to byte identical?<br></blockquote><div><br></div><div>When you make a change to the code generator, you should make a corresponding change to the generated test code.  This allows the reviewer to see how the change in CodeGenerator*.pm affects the generated code and documents the change in SVN.  The hope is that it&#39;ll be obvious if your change is going to have unintended consequences on the generated code.  We&#39;ve found this useful when working on the V8 bindings.</div>

<div><br></div><div>J</div></div>