Re: [Perlpanel-devel] New applet: WeatherWatch

[Moshe Kaminsky]

Hi,

Thanks for the applet, I like it. I have some comments, below.

* Thomas <address@hidden> [09/07/04 05:53]:
> Currently it relies on Gtk2::Html2 to display the current weather 
> information, which is not officially release, but you can download a 
> Debian package, or Debian's source backup from 
> http://packages.debian.org/experimental/perl/libgtk2-html2-perl or you 
> can get it out of the CVS server at http://gtk2-perl.sourceforge.net/

For gentoo, there is an ebuild in bugzilla. 
> 
> I am sure there are plenty of C++isms in the code, and places I could have 
> done things in a simpler, more Perlish way, but it works for me.

I didn't have much chance to look at the code. For me, the biggest 
problem was the 'Location code' in the configuration menu, whose meaning 
I didn't know until I looked at the code. You might want to add some 
documentation. I found a module Gtk2::PodViewer, which allows to view 
pod in a gtk window. This allows you to write the docs as pod, and get 
them displayed for free.

Also, if you happen to enter a wrong code, the thing crashes together 
with the rest of the panel. The crashing of the panel is, IMO, a 
perlpanel problem, but you still may want to catch the situation where 
you get an empty whether report.

Other than that, the applet works fine.

Moshe
> 
> Thomas Stephens
> address@hidden
> 
> 
> _______________________________________________
> Perlpanel-devel mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/perlpanel-devel
> 

-- 
I love deadlines. I like the whooshing sound they make as they fly by. 
                                        -- Douglas Adams
    
    Moshe Kaminsky <address@hidden>

pgpuo8qwpvXz2.pgp
Description: PGP signature