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.
Merge remote-tracking branch 'upstream/master'
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

NOTE: This modifies the binary ABI version number
      due to the mbroadcastsocket.h changes, so
      make distclean, etc. required.