It's never acceptable to use comments to "hide" code from the compiler,
particularly when the compiler provides constructs which permit one to
indicate intent (eg, ifdef and the pragma "unused").
On Oct 1, 2012 9:47 AM, "Gyuyoung Kim" <gyuyo...@gmail.com> 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.
>
> Cheers,
> Gyuyoung
>
> _______________________________________________
> 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
