There's no need to duplicate the packet as the reference count will be 1
after the av_read_frame call. Duplicating causes heap corruption when a
synthetic clock packet is duplicated and assigned the buffer from the
stack-based temporary packet which is then double-freed by the decoder
thread.

To be able to use index buffers, they must also be bound to a vertex
array object along with the vertex buffers.

Ideally, if there are multiple index buffers for a vertex buffer,
separate VAOs should be created for each combination.

GL_ARRAY_BUFFER is for vertex buffers, GL_ELEMENT_ARRAY_BUFFER is for
index buffers.

The 'tier' parameter is used with the NVENC HEVC encoder, not with the
AVC encoder.

When using QSV is used on a windows 7 machine with a dedicated card, you
have to fake a monitor connection to your Intel graphics to be able to
use QSV.  If you do not, the initialization will fail with an error.
The error for that situation is not handled properly, and a variable
will be used while null.  Instead, the function should safely return
after that error is received.

Also, do not call ClearData in the destructor unless QSV has been
properly initialized (if m_pmfxENC is null).

The if statement erroneously ended with a ';', which means that the code
is always executed, but there's no reason to even have these if checks
in the first place as the functions themselves return safely with null
pointers.

Allows extraction of headers/sei from a frame so that an encoder can
separate those headers and provide them for later use.

If audio buffering is very high, the audio packets built up in the
interleaved buffer would be significantly before the first video packet,
causing the offset between the starting video/audio packet pairs to be
significantly off, leading to desync.

This issue was not spotted until recently because it only happens when
streaming/recording with same encoders while audio buffering is very
high.

When using a chain hook method (forward or reverse), it was unwisely
assumed that the previous hook in the chain would not overwrite new
hooks when it's called.  When the game capture hook calls the previous
hook in the chain, certain other programs that hook (in this case,
rivatuner on-screen display) would overwrite the hook with older data
and erase the game capture hook, causing it to only capture the first
frame and then never capture again.

This patch ensures that the hook is always saved before calling the next
hook in the chain and then restored after the call returns.  It also
preserves any new hooks that may be added on top of it at any point.

UI: Change message regarding vcredist

libobs: remove duplicated determinant calculation

Specify that it's the streaming bitrate and not just "bitrate" so users
hopefully realize that it's not referring to the recording settings.

These command line options allow the user to start streaming/recording
on program startup.

Hides video encoders marked as deprecated if they're not currently in
use.

Allows the ability to mark an encoder as deprecated (such as the media
foundation h264 encoders)

This should make it a bit more clear, that you need both vcredist_x64
and vcredist_x86 on 64 Bit Windows versions.

The source shouldn't be inserted into obs->data.first_audio_source until it's
fully initialized, or other threads will access source->control and
dereference an uninitialized pointer.

Detected by Coverity Scan (CID 92178, 92171)

Detected by Coverity Scan (CID 92187, 92170)

Bitrate limit warnings wouldn't be correctly shown if service settings
are changed but not yet saved before modifying the bitrates/settings.
It would use the currently active service settings instead of the
changed values.

Fixes a bug where audio bitrate would not be unlocked if maximum bitrate
enforcement is disabled.

This is a band-aid solution to be able to create temporary services
without logging them and keep them out of enumeration functions.

This is a band-aid solution -- 'master obs context lists' should not be
kept by the core.  Logging of object creation/destruction should also be
controlled by the front-end instead of the core.

deps/libff: Fix that inputs are not closed when a demuxer is freed

avformat_free_context() only frees the memory used by an AVFormatContext
but it does not close the opened media file. This causes a leaked file
descriptor every time a media source frees a demuxer. Using
avformat_close_input() instead frees the context and closes the media
file.

Allows the user to know that the maximum bitrate limit for a streaming
service is being enforced.

Allows the user to disable streaming bitrate limits in simple output
mode if they so choose.  This option is enabled by default.

A double spin control expects valueChanged(double), not
valueChanged(int).