Use pass by ref to construct DVDMsgType

This saves doing an extra copy for each call to the constructor, the worst one probably being the PVR channel changes at 168 bytes each.

Fixes coverity CID 719604
[win32] WINFileSMB::Seek() seeking above end of the file is not an error condition
TagLibVFSStream: fix initialization of m_bIsReadOnly
Merge pull request #4784 from cg110/CDVDMsgType_remove_copy

Use pass by ref to construct DVDMsgType
fixed: Default cipherlist fails with CURL NSS backend
[osx] adapt AERingBuffer.h to allow planar formats
[osx] adapt OSX sink to utilise planar output from AE
[osx] drop the bitstreaming buffer - it's not much more costly than multiple small memcpy's and it's not as if we're performance constrained here
[osx] use a separate member for the frames per second in computations rather than m_format.m_sampleRate
[ios/osx] drop unused IsCompatible() functions from darwin audio sinks
TagLibVFSStream::seek: fix parsing files with broken structure (specified length in header is longer than actual file length)
Merge pull request #4792 from Karlson2k/fix_tag_read

Fix tag reading for some broken files, fixes #15239
Merge pull request #4774 from jmarshallnz/planar

[osx] utilize planar support from AE in the OSX sink
videoplayer: adapt lateness detection and dropping to buffering
video player: present correct pts to user for a/v sync (after buffering in renderer)
videoplayer: some rework and documentation
renderer: bump buffers to 5
videoplayer: update frametime, it might change due to fps detection
videoplayer: give streams with invalid fps a chance for fps detection
dvdplayer: allow rewinding at end of stream, do a seek after rewind