Clear additional field when resetting DVD VM to ensure playback doesn't abort with "Error reading NAV packet".
Add methods to display ourselves should the current window not handle screen stack (e.g. TV Playback)

During video playback OSD elements are directly displayed by the appropriate videoout method. So the Notification Center will have to piggy back on top of those to be displayed.
Display Notification Center cards during video playback

We do so by manually drawing the notification cards within the OSD.

Only tested with OpenGL rendering
Display notification cards even if there's no OSD element displayed
Fix whitespace
Install MythUINotificationCenter public headers
Restore content of registered application's deleted screen.

Should the screen get deleted for whatever reason (theme reload, user pressing ESC): we want to restore the original content of the screen as we may only receive updates later.
Subtitles: Reset the external subtitle filename.  Refs #11618

This ensures that during Live TV, .srt subtitles from the previous
program are not accidentally used for the next program.
Force mythfilldatabase to run from install PREFIX.

This reverts behavior following 6122951dae46d96 to force the full path
of mythfilldatabase to be absolute, based off the stored install PREFIX,
rather than relying on the binary to show up in the environment's binary
path.

Fixes #11633
Put notification card in suspended mode if user delete it with ESC

Each notification update would make the notification card reappear; which could make it a tad invasive... So suspend the card, ignoring all future updates; it will be active again when a "New" notification is received
Merge branch 'master' of code.mythtv.org:mythtv
Tweak the Get[Filtered]RecordedList() services API calls.

Change GetRecordedList() to return recordings from the Deleted group.
This is consistent with the equivalent Myth protocol functionality.
Remove GetFilteredRecordedList() and add the optional filter
parameters to GetRecordedList().  They already used the same function
internally.
Remove GetFilteredLiveStreamList() and add the filtering parameter to
GetLiveStreamList().

They used the same funtion internally.  This also makes it consistent
with all of the other Get*List() calls after GetFilteredRecordedList()
was similarly removed.
Check the return value from posix_fadvise

Static analysis determined that the return value
from posix_fadvise was not checked.  Check the
return, and log if failed (only debug level
logging, since fadvise should never fail in the
first place, and if it does fail it should not
matter (it is, after all, only a hint) but if
debugging, the failure might tell one something).

Fixes coverity 746748 and 746747

Fixes #11612.

Signed-off-by: Paul Harrison <pharrison@mythtv.org>
Initialize m_bProtected in HTTPRequest ctor

m_bProtected will be set to the appropriate value
when the header is fully received.

Fixes coverity 700944

Fixes #11621.

Signed-off-by: Paul Harrison <pharrison@mythtv.org>
Initialize fd_set in MythSystemLegacyIOHandler ctor

Static analysis detected that the fd_set was not
being initialized in the ctor.  While correct
from a static analysis POV, the structure was
being initialized elsewhere before use.  To
make static analysis happy, initialize the fd_set.

Fixes coverity 1028698

Note that coverity may not properly understand
that the use of the provided FD_ZERO macro will
properly initialize the structure since on
some platforms (i.e. x86) the macro turns into
inline assembly code.  In the case that adding
the patch does not eliminate the coverity issue,
one will need to manually acknowledge the defect.

Fixes #11625.

Signed-off-by: Paul Harrison <pharrison@mythtv.org>
Copy m_nice and m_ioprio values from other in ctor

It appears that when commit 201407310b183210ab58b5030abf656e797bae39
added the m_ioprio and m_nice member variables that the
constructor was not updated to copy those values.

Fixes Coverity 1028696

Fixes #11626.

Signed-off-by: Paul Harrison <pharrison@mythtv.org>
Properly handle fd number 0 in signal manager

Visual analysis (IEAEYEBALL) detected that the
check for the file descriptor value should be
"fd >= 0" and not just "fd > 0".  While a fd of
zero is commonly stdin, do the right check just
in case.

The CLOSE macro properly performs the check
for greater than or equal to zero.  This
patch just does the same thing.

Fixes #11630.

Signed-off-by: Paul Harrison <pharrison@mythtv.org>
Check the return value from setsockopt and log errors

Fixes coverty 700269

Fixes #11611.

Signed-off-by: Paul Harrison <pharrison@mythtv.org>
Subtitles: Reload .srt files in the background with a helper thread.

This avoids a brief stutter in the video thread when subtitles are
reloaded.  Refs #11618.