Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Time-HiRes CPAN distribution.

Report information
The Basics
Id:
100751
Status:
resolved
Priority:
Low/Low
Queue:

People
Owner:
Nobody in particular
Requestors:
JHI [...] cpan.org
Cc:
AdminCc:

BugTracker
Severity:
(no value)
Broken in:
(no value)
Fixed in:
(no value)



Subject: gcc 4.9.2 warnings
HiRes.c: In function 'XS_Time__HiRes_clock_gettime': HiRes.c:1671:6: warning: variable 'clock_id' set but not used [-Wunused-but-set-variable] int clock_id; ^ HiRes.c: In function 'XS_Time__HiRes_clock_getres': HiRes.c:1741:6: warning: variable 'clock_id' set but not used [-Wunused-but-set-variable] int clock_id; ^ HiRes.xs: In function 'XS_Time__HiRes_clock': HiRes.xs:1250:18: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] RETVAL = clocks == -1 ? -1 : (NV)clocks / (NV)CLOCKS_PER_SEC; ^
Is this also fixed by your fix for #102718? -- rjbs
Subject: Re: [rt.cpan.org #100751] gcc 4.9.2 warnings
Date: Sat, 29 Aug 2015 19:27:16 -0400
To: bug-Time-HiRes@rt.cpan.org, JHI@cpan.org
From: Jarkko Hietaniemi <jhi@iki.fi>
On Saturday-201508-29 18:10, Ricardo Signes via RT wrote:
Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=100751 > > > Is this also fixed by your fix for #102718? >
It might be. Is there a repo I could check?
CC: ;
Subject: Re: [rt.cpan.org #100751] gcc 4.9.2 warnings
Date: Sat, 29 Aug 2015 19:37:11 -0400
To: "jhi@iki.fi via RT" <bug-Time-HiRes@rt.cpan.org>
From: Ricardo Signes <rjbs@cpan.org>
* "jhi@iki.fi via RT" <bug-Time-HiRes@rt.cpan.org> [2015-08-29T19:27:40]
Show quoted text
> It might be. Is there a repo I could check?
https://github.com/rjbs/Time-HiRes has the lastest, as does https://metacpan.org/release/RJBS/Time-HiRes-1.9727_02 -- rjbs

Message body not shown because it is not plain text.

Subject: Re: [rt.cpan.org #100751] gcc 4.9.2 warnings
Date: Sat, 29 Aug 2015 19:59:50 -0400
To: bug-Time-HiRes@rt.cpan.org, JHI@cpan.org
From: Jarkko Hietaniemi <jhi@iki.fi>
On Saturday-201508-29 19:37, Ricardo Signes via RT wrote:
Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=100751 > > > * "jhi@iki.fi via RT" <bug-Time-HiRes@rt.cpan.org> [2015-08-29T19:27:40]
>> It might be. Is there a repo I could check?
> > https://github.com/rjbs/Time-HiRes > > has the lastest, as does > https://metacpan.org/release/RJBS/Time-HiRes-1.9727_02 >
The #102718 fix indeed works, but in OS X I noticed a new (unrelated to this bug) warning, because of unimplemented interfaces. Patch attached.

Message body is not shown because sender requested not to inline it.

Subject: Re: [rt.cpan.org #100751] gcc 4.9.2 warnings
Date: Sat, 29 Aug 2015 20:08:49 -0400
To: bug-Time-HiRes@rt.cpan.org
From: Jarkko Hietaniemi <jhi@iki.fi>
On Saturday-201508-29 20:00, jhi@iki.fi via RT wrote:
Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=100751 > > > On Saturday-201508-29 19:37, Ricardo Signes via RT wrote:
>> <URL: https://rt.cpan.org/Ticket/Display.html?id=100751 > >> >> * "jhi@iki.fi via RT" <bug-Time-HiRes@rt.cpan.org> [2015-08-29T19:27:40]
>>> It might be. Is there a repo I could check?
>> >> https://github.com/rjbs/Time-HiRes >> >> has the lastest, as does >> https://metacpan.org/release/RJBS/Time-HiRes-1.9727_02 >>
> > The #102718 fix indeed works, but in OS X I noticed a new (unrelated to > this bug) warning, because of unimplemented interfaces. Patch attached.
So the patch was against the blead T::H, sorry about that. Now testing with the repo.
Show quoted text
> > >
Subject: Re: [rt.cpan.org #100751] gcc 4.9.2 warnings
Date: Sat, 29 Aug 2015 20:19:31 -0400
To: bug-Time-HiRes@rt.cpan.org
From: Jarkko Hietaniemi <jhi@iki.fi>
On Saturday-201508-29 20:09, jhi@iki.fi via RT wrote:
Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=100751 > > > On Saturday-201508-29 20:00, jhi@iki.fi via RT wrote:
>> <URL: https://rt.cpan.org/Ticket/Display.html?id=100751 > >> >> On Saturday-201508-29 19:37, Ricardo Signes via RT wrote:
>>> <URL: https://rt.cpan.org/Ticket/Display.html?id=100751 > >>> >>> * "jhi@iki.fi via RT" <bug-Time-HiRes@rt.cpan.org> [2015-08-29T19:27:40]
>>>> It might be. Is there a repo I could check?
>>> >>> https://github.com/rjbs/Time-HiRes >>> >>> has the lastest, as does >>> https://metacpan.org/release/RJBS/Time-HiRes-1.9727_02 >>>
>> >> The #102718 fix indeed works, but in OS X I noticed a new (unrelated to >> this bug) warning, because of unimplemented interfaces. Patch attached.
> > So the patch was against the blead T::H, sorry about that. Now testing > with the repo.
I'm confused about the results... and right now out of time. So please ignore the patch. But there's definitely some "unused" sprinkling needed around the clock_id when the APIs with clock_id args are not available, and we have just a croak() instead.
This was resolved by the "rjbs" releases.


This service runs on Request Tracker, is sponsored by The Perl Foundation, and maintained by Best Practical Solutions.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.