2011/6/29 Eric Blake <eblake@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