<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><br></div><div><br></div><div><div><div>On Apr 29, 2010, at 1:05 PM, Adam Barth wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>On Thu, Apr 29, 2010 at 12:43 PM, Maciej Stachowiak &lt;<a href="mailto:mjs@apple.com">mjs@apple.com</a>&gt; wrote:<br><font class="Apple-style-span" color="#540000"><br></font><blockquote type="cite">It seems to me a better model would be to regenerate the bindings test file<br></blockquote><blockquote type="cite">automatically as part of the build. Then the changes can still be reviewed<br></blockquote><blockquote type="cite">by you, or as part of a diff, but there would be no extra manual steps<br></blockquote><blockquote type="cite">involved. And people making behaviorally transparent changes to codegen<br></blockquote><blockquote type="cite">output would perhaps feel a little less burdened.<br></blockquote><br>That sounds like a good improvement. &nbsp;I think it would be fine to<br>regenerate the output as part of the build. &nbsp;However, I think we<br>should still preserve the ability to run the script along it "test"<br>mode because that's about three orders of magnitude faster than<br>performing a build after touching CodeGeneratorJS.<br></div></blockquote></div></div><div><br></div><div>Alexey (or others who don't like the new tests), do you think this change would address your concerns?</div><br><div><div>On Apr 29, 2010, at 1:05 PM, Adam Barth wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div><br>What I hear from this conversation is the following:<br><br>1) A bunch of people who've used the tool saying that they've found it useful.<br>2) A bunch of people who haven't used the tool suggesting improvements.<br><br>This tool impacts the handful of people who work on<br>CodeGeneratorJS.pm. &nbsp;Everyone else in the project can safely ignore<br>it. &nbsp;I'm all for improvements, and I invite anyone interested to<br>either improve the tool or write a new tool that does the job better.<br></div></blockquote></div><br><div>If everyone has to use the tool for the tool to be useful, then ideally we want a system where the people who change the bindings frequently mostly buy into.&nbsp;Here is the list of people with more than 5 all-time commits in the WebCore/bindings/scripts directory. Ideally I'd like to hear from more of these what they think would be helpful and not burdensome.</div><div><br></div><div><div>&nbsp;&nbsp;59 &nbsp;<a href="mailto:weinig@apple.com">weinig@apple.com</a></div><div>&nbsp;&nbsp;46 &nbsp;<a href="mailto:eric@webkit.org">eric@webkit.org</a></div><div>&nbsp;&nbsp;35 &nbsp;<a href="mailto:darin@apple.com">darin@apple.com</a></div><div>&nbsp;&nbsp;32 &nbsp;<a href="mailto:japhet@chromium.org">japhet@chromium.org</a></div><div>&nbsp;&nbsp;29 &nbsp;<a href="mailto:oliver@apple.com">oliver@apple.com</a></div><div>&nbsp;&nbsp;26 &nbsp;<a href="mailto:ggaren@apple.com">ggaren@apple.com</a></div><div>&nbsp;&nbsp;26 &nbsp;<a href="mailto:dglazkov@chromium.org">dglazkov@chromium.org</a></div><div>&nbsp;&nbsp;16 &nbsp;<a href="mailto:abarth@webkit.org">abarth@webkit.org</a></div><div>&nbsp;&nbsp;14 &nbsp;<a href="mailto:zimmermann@webkit.org">zimmermann@webkit.org</a></div><div>&nbsp;&nbsp;12 &nbsp;<a href="mailto:ap@webkit.org">ap@webkit.org</a></div><div>&nbsp;&nbsp;10 &nbsp;<a href="mailto:aroben@apple.com">aroben@apple.com</a></div><div>&nbsp;&nbsp; 8 &nbsp;<a href="mailto:levin@chromium.org">levin@chromium.org</a></div><div>&nbsp;&nbsp; 7 &nbsp;<a href="mailto:mjs@apple.com">mjs@apple.com</a></div><div>&nbsp;&nbsp; 7 &nbsp;<a href="mailto:darin@chromium.org">darin@chromium.org</a></div><div>&nbsp;&nbsp; 6 &nbsp;<a href="mailto:timothy@apple.com">timothy@apple.com</a></div><div>&nbsp;&nbsp; 6 &nbsp;<a href="mailto:snej@chromium.org">snej@chromium.org</a></div><div>&nbsp;&nbsp; 6 &nbsp;<a href="mailto:jianli@chromium.org">jianli@chromium.org</a></div><div>&nbsp;&nbsp; 6 &nbsp;<a href="mailto:ddkilzer@apple.com">ddkilzer@apple.com</a></div><div>&nbsp;&nbsp; 6 &nbsp;<a href="mailto:cwzwarich@webkit.org">cwzwarich@webkit.org</a></div><div><br></div><div>Here is the command anyone can run to see the full list (assuming you have an SVN checkout):</div><div><div>$ svn log WebCore/bindings/scripts | grep '|.*@' | sed -e 's/^[^|]* |//g; s/ | .*$//g;' | sort | uniq -c | sort -rn</div><div><br></div><div>The long tail of people who have made only a few bindings changes is rather large, so I suspect this tool affects more than a handful people, if it becomes a mandatory part of the process.</div><div><br></div><div>Regards,</div><div>Maciej</div><div><br></div></div></div></body></html>