<br><br>On Monday, October 1, 2012, Gyuyoung Kim  wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>Hello WebKit folks,</div><div><br></div><div>There were build warning related to unused parameter nowadays. I think there are three solutions. One is to remove parameter,</div>
<div>another is to use UNUSED_PARAM macro and the other is to use /* */ in parameters.</div>

<div><br></div><div>I like to use UNUSED_PARAM macro except for primitive parameters personally, for example</div><div><br></div><div>void foo(RenderObject* object, int /*width*/, int /*height*/) {</div><div>    UNUSED_PARAM(object);</div>


<div>}</div><div><br></div><div>I&#39;d like to know what webkittens think about this.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br></blockquote></blockquote>
<div>If it is crystal clear what the parameters stand for, omit them. If not, you find them commented out. UNUSED_PARAM is used when just some platforms of flagged features use parameters, others not.</div><div><br></div>
Dirk<span></span><br><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>Cheers,</div><div>Gyuyoung</div>
</blockquote>