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.
Dirk
> Cheers,
> Gyuyoung
>
_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-dev
