trinity-users@lists.pearsoncomputing.net

Message: previous - next
Month: January 2021

Re: [tde-users] Re: kweather does not update weather data [SOLVED]

From: deloptes <deloptes@...>
Date: Thu, 07 Jan 2021 14:37:28 +0100
Stefan Krusche via tde-users wrote:

>> The service runs independently and can update multiple stations. The
>> applet then displays just one of these stations. Therefore, there are
>> two separate configuration files.
> 
> Good. That makes technical sense and I don't need to bother about it
> anymore.

This is quite OK. What I mean is that you have the entries hardcoded in
those files and not generated by the same source. This leads to problems
like the one we have that someone should update all the files if something
changes. It would be better if we generate all those files from the same
source.

I also noticed other duplicates in the code and despite I am not a
professional C++ developer, I still had to shake my head.
I was thinking there should be one service and library doing all the work,
no?