On Oct 1, 2012, at 8:47 AM, Ryosuke Niwa <rn...@webkit.org> wrote: > On Mon, Oct 1, 2012 at 7:07 AM, Dirk Schulze <k...@webkit.org> wrote: > > > On Monday, October 1, 2012, Gyuyoung Kim wrote: > Hello WebKit folks, > > There were build warning related to unused parameter nowadays. I think there > are three solutions. One is to remove parameter, > another is to use UNUSED_PARAM macro and the other is to use /* */ in > parameters. > > I like to use UNUSED_PARAM macro except for primitive parameters personally, > for example > > void foo(RenderObject* object, int /*width*/, int /*height*/) { > UNUSED_PARAM(object); > } > > I'd like to know what webkittens think about this. > > 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. > > or only used in ASSERTs.
That's what the ASSERT_UNUSED(variable, assertion) macro is for :D --Oliver > > - Ryosuke > > _______________________________________________ > webkit-dev mailing list > webkit-dev@lists.webkit.org > http://lists.webkit.org/mailman/listinfo/webkit-dev
_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo/webkit-dev