Re: problem case with recurrence in DateTime::Event::ICal

2005-09-30 Thread Flavio S. Glock
DateTime-Event-ICal-0.09 was uploaded to CPAN - it fixes the warnings
while calculating BYSETPOS.

Mark D. Anderson wrote:
 Could you try again without the dtend and until args?
 Combining with bysetpos and dtstart being set still gives me problems.

I was not able to replicate the error message - would you send me a
small runnable program?

 btw, the RRULE processing seems rather inefficient.

BYSETPOS calculates the whole set, and then discards the uninteresting
items. The code is in DateTime/Event/ICal.pm line 361.
BYSETPOS=1 could be special-cased to calculate only the first instance
(patches are welcome).

- Flavio S. Glock


Re: problem case with recurrence in DateTime::Event::ICal

2005-09-30 Thread Mark D. Anderson

On Fri, 30 Sep 2005 14:07:55 -0300, Flavio S. Glock [EMAIL PROTECTED]
said:
 DateTime-Event-ICal-0.09 was uploaded to CPAN - it fixes the warnings
 while calculating BYSETPOS.
 
 Mark D. Anderson wrote:
  Could you try again without the dtend and until args?
  Combining with bysetpos and dtstart being set still gives me problems.
 
 I was not able to replicate the error message - would you send me a
 small runnable program?

You can find one at 
http://discerning.com/hacks/bugs/dtbug.pl

It is still failing for me, after the latest changes.

 
  btw, the RRULE processing seems rather inefficient.
 
 BYSETPOS calculates the whole set, and then discards the uninteresting
 items. The code is in DateTime/Event/ICal.pm line 361.
 BYSETPOS=1 could be special-cased to calculate only the first instance
 (patches are welcome).

I will be immensely more motivated to take a crack at
working on optimization once i get this to work at all :)

-mda