Flavio S. Glock wrote:
> Hill, Ronald wrote:
>> 
>> Hi All,
>> 
>> I have just updated the DateTime::Event::Sunrise package on
>> CVS. I have added a new method sunrise_sunset (and a new method).

[snipped]

> 
> Is this too verbose?
> 
>   my $dt_span = $sunrise->sunrise_sunset_span( datetime => $dt );
>   print "Sunrise is:" , $dt_span->start->datetime , "\n";
>   print "Sunset is:" ,  $dt_span->end->datetime;

Ok, easy enought to fix.

> 
>   my $dt_set =  $sunrise->sunrise_set( %optional_parameters );
>   my $dt_set2 = $sunrise->sunset_set( %optional_parameters );

I don't understand this?

> 
>   my $dt =  $sunrise->sunrise_dt( datetime => $dt );
>   my $dt2 = $sunrise->sunset_dt( datetime => $dt );

Ok, I will add another method for this.

Thanks Flavio

Ron Hill

Reply via email to