audio player was initialized as having audio out by default, even if no audio output had beed created. This caused to always try to decode audio even when not required (like during commflagging)

Also, exit early in AVFD::ProcessAudioPacket if no one is listening.

Fixes #11616
Similar to c9d8533fe8fc19fead0ad896f37bb8f07367cc57, no need to attempt to decode audio if it's not going to be played
Update schema version in Perl and Python bindings (missing from 4f023d7222)
RSSEditPopup: Fix Coverity ID  1037043 Unchecked return value

In RSSEditPopup::slotSave(): check the download completed OK
Initialize _tune_mode to hdhrTuneModeNone in ctor

Fixes coverity 700909. Closes #11615.

Signed-off-by: Paul Harrison <pharrison@mythtv.org>
Disable minimum file size check for .srt files.  Refs #11618
Support .srt files for in-progress recordings.  Refs #11618

Allow a .srt file to be reloaded for an in-progress recording after
the last subtitle is reached.  The backend is queried at most one a
second for an updated .srt file.

Still to do:

1. Do the update check in a separate thread to avoid stuttering.

2. Currently the REC_STARTED_WRITING event is not sent when the HD-PVR
switches to a new program during live TV, so the third-party script
ends up not generating .srt files for subsequent programs in a live TV
session.

3. ccextractor buffers its output, so one needs to be at least 30
seconds behind real-time to get .srt captions.  A better solution
would be to add the required functionality to mythccextractor.
Fix transcoding to high resolutions.

When the transcoder loaded the recording profile, the max width and
height fields were not being set properly so transcoding profiles
with high resolutions were instead limited to 768x576.  Since these
max values are set the ImageSize ctor, changing them after the fact
is a bit invasive.  Rather than using a RecordingProfile variable
in Transcode and trying to change the values after the fact, this patch
fixes the issue by using a RecordingProfile pointer and creating an
instance allowing us to tell the constructor that the profile we're
going to load is a transcoder profile and the max resolution values
(and any other transcoder-specific values or limits) should be
set correctly.

Fixes #11348.
IPTVSignalMonitor: Fix Coverity ID 746869 & 746790

m_streamHandlerStarted was never initialised in the ctor so the random value
was used later (looks like a genuine bug).

Comment out for now the dead code which will be used once danielk fixes the
TODO and initialises isLocked to the correct value :)
CetonStreamHandler: Fix Coverity ID 700906 Uninitialized scalar field

In CetonStreamHandler::CetonStreamHandler(): Several scalar fields are not
initialized by the constructor.
AnalogSignalMonitor: Fix Coverity ID 700903 Uninitialized scalar field

m_version can be left uninitialized by the constructor.
NuppelDecoder: Fix Coverity ID 700917 Uninitialized pointer field

strm_buf was left uninitialized by the constructor.
DataDirectProcessor: Fix Coverity ID  700255 Unchecked return value

In DataDirectProcessor::GrabData(): Check the data file opened OK.
Use RemoteFile::isOpen() instead of Open() to check for success.

Calling Open() was unnecessarily creating extra socket connections to
the backend, and the original socket connections from calling Open()
in the constructor were leaking and accumulating in both the frontend
and backend (refs #11341).  Refs #11618
RemoteFile leak fix and simplification

RemoteFile::Open() would leak file descriptors if Open() was explicitly
called again after passing a url to the contructor which itself called
Open().

This commit does several things:

- Moves Open() private since RemoteFile users shouldn't need to Open()
- Moves Close() private since there's no need to Close() if you can't Open().
  Close() is already called in the destructor.
- Convert the remaining uses of Open() to isOpen() since the connection is
  already opened in all these cases.
- Remove a few Close() uses since they were were redundant since we already
  Close() in the destructor.
- Removes unused SetURL() functionality.  If you can't Open() or Close(),
  then you need to pass the URL in to the constructor so there's no need.
- Fixes the actual leak in Open() by checking to see if the sockets are
  already open.  Just in case....  Closes #11341.

NOTE: This does modify the binary ABI version number due to the remotefile.h
      changes, so make clean, etc..
Consistency with mythtv/.gitignore

Instead of explicitly listing config.ep, .log, .pro, just use
config.* as we do in .gitignore so they're consistent.
If we didn't actually handle LEFT/RIGHT/UP/DOWN in  MythScreenType::keyPressEvent() then return false.
Fix whitespace error in EITFixUp::FixNL()
Remove unused member variable m_imr in MBroadcastSocketDevice

The member variable m_imr appears to be a carryover
from a code copy/paste from the multicast socket
class (where it is used for the group address).
It is unused in a broadcast socket.  Remove it.

Fixes coverity 746874
Fixes #11622

NOTE: This modifies the binary ABI version number
      due to the mbroadcastsocket.h changes, so
      make distclean, etc. required.
(cherry picked from commit 5e15740501d9e89c0eba71d8ae97c16c3b737495)

Signed-off-by: Stuart Morgan <smorgan@mythtv.org>
Default language for DVD menus, audio and subtitle is now set based on the frontend language setting.