2011/6/29 Eric Blake <eblake(a)redhat.com>:
s/compliation/compilation/ in the subject
On 06/28/2011 04:34 PM, Matthias Bolte wrote:
> Version 1.3 uses this macro
version 1.3 of <sys/sdt.h>
>
> #define STAP_CAST(t) (size_t)t
>
> that breaks if t is a function. For that to work it should probably
> look like this
>
> #define STAP_CAST(t) ((size_t)(t))
For the record, this is what Matthias posted on IRC:
remote.c:1775: error: cast from function call of type 'const char *' to
non-matching type 'long unsigned int' [-Wbad-function-cast]
>
> In systemtap 1.4 this was completely rewritten.
>
> Anyway, before commit df0b57a95a767c t was always a variable, but now
> also a function is used here, namely virNetSASLSessionGetIdentity.
>
> Use an intermediate variable to avoid this problem.
> ---
> daemon/remote.c | 17 +++++++++--------
> 1 files changed, 9 insertions(+), 8 deletions(-)
ACK.
I improved the commit message and pushed the result.
--
Matthias Bolte
http://photron.blogspot.com