[Pkg-xfce-devel] Bug#735478: Bug#735478: this bug is in debian-stable

[Graham]

On Mon, 27 Jan 2014 19:32:44 +0100
Yves-Alexis Perez <corsac at debian.org> wrote:

> > As posted before in message #50, that version from proposed is
> > missing *all* maximum/high temperatures in the forecast tab. First
> > patch from Christoph Biedl works smoothly here.
> > 
> Well, the initial patch just ignores if hi pointer is NULL, so I fail
> to see how you can have a value there if weather.com doesn't provide
> it?

Thing is, there's a problem with both Christoph Biedl's and the
proposed patches, but I'd argue that the patch from Christoph Biedl is
preferable to the one in proposed, causing less of an issue.

Thing is, for me with the proposed patch, the forecast for all the
upcoming days are null. The nights are fine, they show temperatures,
but there is no information about the coming days, for they're *all*
shown as null. What's the point of a forecast if only the upcoming
nights are on offer? The patch by Christoph does have a problem with
showing "99" during the current day after it's reached its highpoint,
but at least the forecast for the other days are there and serves some
use. I'd argue that the forecast tab showing the high and low
temperatures for both the days and the nights, respectively, despite
the issue, is far more useful than showing "Null" on the current day
and *all* the days to come in the forecast, which is what happens for me
with the proposed version. With Christoph's patch, at least I can get a
forecast for the coming days, while with the one in proposed I can only
get the forecast for the nights.

I've been using Christoph's patch for a few days without issue, bar the
"99" thing. The "99" is a problem, but more an irritant than being
mostly broken like the one in proposed, which doesn't even give me a
forecast for daytimes to come.

Still, despite all the above, may I thank you for maintaining this
package despite its ongoing problems.

Regards,


Graham