[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Perlpanel-devel] New applet: WeatherWatch
From: |
Moshe Kaminsky |
Subject: |
Re: [Perlpanel-devel] New applet: WeatherWatch |
Date: |
Sat, 10 Jul 2004 11:32:10 +0300 |
User-agent: |
Mutt/1.5.6i |
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