On Tue, 25 Mar 2003, Hill, Ronald wrote: > my $sunrise = DateTime::Event::Sunrise ->new( > Longitude =>'-118' , > Latitude => '33', > );
For consistency with other DateTime modules, it'd be good if the parameter names were lower case. A couple others nits: $VERSION = qw($Revision: 0.01 $) [1]; That's not CVS, is it? If it is, please don't use it's revision numbers directly (this is discussed in the dev docs I wrote). Otherwise it looks good to me. -dave /*======================= House Absolute Consulting www.houseabsolute.com =======================*/
