Tvheadend

One of my recent changes has resulted in a bug:

https://tvheadend.org/issues/6293
https://tvheadend.org/boards/5/topics/50218

I have already completed troubleshooting and have issued a workaround.

This is the same change that causes issues on some systems with the embedded variable declaration within a FOR statement.

https://github.com/tvheadend/tvheadend/pull/1544

Flole Systems - You recently said that you would not merge any new changes until Oliver did his thing. I can see two options: Would you like me to?:

1. Continue to wait for Oliver before proceeding with both fixes.
2. Fix both problems and submit a PR ASAP.

I take full responsibility for this error as it resulted from insufficient system knowledge and testing on my part. I plan to look into the Wiki to see if I can make a note somewhere for future developers so that they can hopefully avoid a repeat of the situation.

The TVH settings subsystem seems to only load settings that are already in the settings file. When my modified version ran on a machine other than mine, it saw no corresponding setting so the internal setting update function was not triggered for that setting and the translation array was not initiated to a 1:1 state.

I know that this project is under-resourced, and I don’t have a solution for that. However, a second set of eyes would have probably caught this. It only would have taken 1 other person with a DVB system to install and run the release to spot the error and stop it. It compiled OK for the most part, but in the end the error was in the logic and I was let down by my insufficient knowledge of the nuances of the TVH internals and insufficient testing.

Luckily, 'only' 2 end users seem to have been impacted and they managed to implement the workaround fairly quickly. Although the impact was relatively small, I am still disappointed that it happened at all.