On Mon, Jan 27, 2014 at 08:29:32PM +0000, Graham wrote: > On Mon, 27 Jan 2014 19:32:44 +0100 > Yves-Alexis Perez <cor...@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. > That doesn't actually reply to my question way above. -- Yves-Alexis Perez
signature.asc
Description: Digital signature
