On 28-2-2018 10:35, Willem Jan Withagen wrote:
>
> Hi Radoslow,
>
> Clang is unhappy about:
```
/home/jenkins/workspace/ceph-master/src/tools/rbd_ggate/debug.cc:20:5:
error: no matching member function for call to 'should_gather'
dout(level) << msg << dendl;
^~~~~~~~~~~
/home/jenkins/workspace/ceph-master/src/common/debug.h:23:17: note:
expanded from macro 'dout'
#define dout(v) ldout((dout_context), v)
^~~~~~~~~~~~~~~~~~~~~~~~
/home/jenkins/workspace/ceph-master/src/common/dout.h:98:24: note:
expanded from macro 'ldout'
#define ldout(cct, v) dout_impl(cct, dout_subsys, v) dout_prefix
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/jenkins/workspace/ceph-master/src/common/dout.h:82:44: note:
expanded from macro 'dout_impl'
return (cctX->_conf->subsys.template should_gather<sub, v>()); \
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
/home/jenkins/workspace/ceph-master/src/tools/rbd_ggate/debug.cc:20:5:
note: in instantiation of function template specialization 'debugv(int,
const char *, __va_list_tag *)::(anonymous
class)::operator()<CephContext *>' requested here
/home/jenkins/workspace/ceph-master/src/common/debug.h:23:17: note:
expanded from macro 'dout'
#define dout(v) ldout((dout_context), v)
^
/home/jenkins/workspace/ceph-master/src/common/dout.h:98:24: note:
expanded from macro 'ldout'
#define ldout(cct, v) dout_impl(cct, dout_subsys, v) dout_prefix
^
/home/jenkins/workspace/ceph-master/src/common/dout.h:84:4: note:
expanded from macro 'dout_impl'
}(cct); \
^
/home/jenkins/workspace/ceph-master/src/log/SubsystemMap.h:71:8: note:
candidate template ignored: invalid explicitly-specified argument for
template parameter 'LvlV'
bool should_gather() {
^
1 error generated.
```
And what worries me is the comment in dout.h:
```
} else {
/* The parentheses are **essential** because commas in angle
* brackets are NOT ignored on macro expansion! A language's
* limitation, sorry. */
return (cctX->_conf->subsys.template should_gather<sub, v>());
}
```
Could these 2 be related....
--WjW
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
