While being fixing compiler warnings about unused variables (bug96901) I found it useful to have UNUSED_PARAMS macro accepting a list of unused variables in order to safe some lines of code in case when we have to mark several arguments as unused (for instance in case of empty implementation of a method).
Created attachment 164505 [details] patch
Comment on attachment 164505 [details] patch Attachment 164505 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13898087
Comment on attachment 164505 [details] patch This is not needed. Our policy is to not put the argument names when they are not used. There is also no need to update URLParser.h, that code will either be implemented or removed entirely.
(In reply to comment #3) > (From update of attachment 164505 [details]) > This is not needed. > Our policy is to not put the argument names when they are not used. There is also no need to update URLParser.h, that code will either be implemented or removed entirely. That is not entirely true for headers through. They are normally added when they add more information.
(In reply to comment #3) > (From update of attachment 164505 [details]) > This is not needed. > Our policy is to not put the argument names when they are not used. There is also no need to update URLParser.h, that code will either be implemented or removed entirely. Should than UNUSED_PARAM macro exist in WebKit in principal? or it should be considered as deprecated and will be deleted eventually?
Created attachment 164526 [details] patch v2 Should build on gtk and mac
Comment on attachment 164526 [details] patch v2 Attachment 164526 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/13904118
Unless you have a good use case, I don't think we should add this. Please _do not_ modify URLParser.h. My local version differ significantly.
I agree with Benjamin. It's unlikely that we have many places to deploy this.
close based on previous comments