Skip Menu |

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

Report information
The Basics
Id: 3236
Status: resolved
Priority: 0/
Queue: Time-Beat

People
Owner: gro.edolnwod [...] lrep
Requestors: spoon [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Date: Thu, 14 Aug 2003 20:56:14 +1000
To: bug-time-beat [...] rt.cpan.org
Subject: Bug fixes for Time::Beat
From: Iain Truskett <spoon [...] cpan.org>
Heyo, I was looking at Time::Beat, to see if you were doing anything I could nab for DateTime::Format::IBeat and found some things that needed fixing or could be improved. I've attached two things: 1 - diffs from 1.1, which are probably hard to read given the number of changes, so I've also attached: 2 - tar.gz of the modified version. While some of the changes may seem superficial, your core calculation routines were producing incorrect results. Hope you find the attached useful. cheers, -- Iain.

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

Download Time-Beat-1.1_b.tar.gz
application/x-tar-gz 1.9k

Message body not shown because it is not plain text.

[SPOON - Thu Aug 14 06:56:32 2003]: Show quoted text
> I was looking at Time::Beat, to see if you were doing > anything I could nab for DateTime::Format::IBeat
:) I saw your module and was impressed; I haven't got into proper OO code yet, and this was meant to be a quick hack (after taking it over from the previous author who never got around to doing conversions for both ways). Show quoted text
> and found some things that needed fixing or could be improved. > > I've attached two things...
Thanks very much! I've released a new version with your changes. Show quoted text
> While some of the changes may seem superficial, your core > calculation routines were producing incorrect results.
I guess I should have done some testing on the code I inherited from the previous version. I suck. Thanks, Earle.