Re: [PATCH] virthreadpthread: don't try to cast pthread_t to void*

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



At Thu, 02 May 2013 07:54:47 -0600,
Eric Blake wrote:
> 
> [1  <multipart/signed (7bit)>]
> [1.1  <text/plain; UTF-8 (quoted-printable)>]
> On 05/02/2013 06:05 AM, Guido Günther wrote:
> > This fixes the build on kFreeBSD that otherwise fails with:
> > 
> > util/virthreadpthread.c: In function 'virThreadSelfID':
> > util/virthreadpthread.c:222:27: error: cast from function call of type 'pthread_t' to non-matching type 'void *' [-Werror=bad-function-cast]
> > ---
> > I'm unsure why we'd need the cast to (void*) first.
> 
> EVERYTHING should be castable to void*.  That's lame that your compiler
> is refusing to cast pthread_t.
> 
> I'm thinking that the solution will require writing a union, assign to
> the pthread_t member of the union, and read out the int member, as the
> only way we're going to shut up compilers.  We KNOW that casting
> pthread_t isn't portable; but all we really want is a few bits that are
> hopefully likely enough to be unique to make the use of virThreadSelfID
> useful in debug messages.  We aren't relying on the conversion being a
> 1:1 mapping nor on trying to reverse the mapping.

Since the warning only ever refers to directly casting a function
return value, wouldn't it suffice to introduce a (properly typed)
variable first assigning the function result?

pthread_t self = pthread_self();
return (int)(intptr_t)self;

Just my 2¢
-- 
AV-Test GmbH, Henricistraße 20, 04155 Leipzig, Germany
Phone: +49 341 265 310 19
Web:<http://www.av-test.org>

Eingetragen am / Registered at: Amtsgericht Stendal (HRB 114076)
Geschaeftsfuehrer (CEO): Andreas Marx, Guido Habicht, Maik Morgenstern

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux