wqking / eventpp

Event Dispatcher and callback list for C++

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

unused parameter 'args'

bd1es opened this issue · comments

commented

Hello,

One question, is it appropriate to replace the first line with the second one as shown below?
static auto forEach(const Self * /self/, A && ...args)
//static auto forEach(const Self * /self/, A && ...)
It's line 271 of “eventpp/eventdispatcher.h”.

Seems that, this removes a warning “unused parameter 'args'“ emitted from clang if -Wunused-parameter is on.

Thank you.

Thanks for the issue report.
I just fixed it on master, please check.

commented

Cloned, thank you.