libobs,plugins: Remove new obs_output_*2 functions

Effectively reverting parts of d314d47, this commit removes the new
functions that got added to remove the flags parameter. Instead, it just
marks the parameter as unused and documents this. Having what is
effectively an API break just to remove a parameter is a bit overkill.
The other parts of d314d47 which cleaned up the usage of the flags
parameter are untouched here.
This commit is contained in:
gxalpha
2023-06-04 14:00:37 +02:00
committed by Lain
parent 5cc319d97f
commit 79822a58c3
17 changed files with 60 additions and 70 deletions

View File

@@ -882,32 +882,38 @@ Functions used by outputs
---------------------
.. function:: bool obs_output_can_begin_data_capture2(const obs_output_t *output)
.. function:: bool obs_output_can_begin_data_capture(const obs_output_t *output, int flags)
Determines whether video/audio capture (encoded or raw) is able to
start. Call this before initializing any output state to ensure that
the output can start.
:param output: The output
:param flags: Reserved. Set this to 0.
:return: *true* if data capture can begin
---------------------
.. function:: bool obs_output_initialize_encoders2(obs_output_t *output)
.. function:: bool obs_output_initialize_encoders(obs_output_t *output, int flags)
Initializes any encoders/services associated with the output. This
must be called for encoded outputs before calling
:c:func:`obs_output_begin_data_capture2()`.
:c:func:`obs_output_begin_data_capture()`.
:param output: The output
:param flags: Reserved. Set this to 0.
:return: *true* if successful, *false* otherwise
---------------------
.. function:: bool obs_output_begin_data_capture2(obs_output_t *output)
.. function:: bool obs_output_begin_data_capture(obs_output_t *output, int flags)
Begins data capture from raw media or encoders. This is typically
when the output actually activates (starts) internally. Video/audio
data will start being sent to the callbacks of the output.
:param output: The output
:param flags: Reserved. Set this to 0.
:return: *true* if successful, *false* otherwise. Typically the
return value does not need to be checked if
:c:func:`obs_output_can_begin_data_capture2()` was

View File

@@ -157,10 +157,10 @@ bool obs_output_delay_start(obs_output_t *output)
};
if (!delay_active(output)) {
bool can_begin = obs_output_can_begin_data_capture2(output);
bool can_begin = obs_output_can_begin_data_capture(output, 0);
if (!can_begin)
return false;
if (!obs_output_initialize_encoders2(output))
if (!obs_output_initialize_encoders(output, 0))
return false;
}
@@ -175,7 +175,7 @@ bool obs_output_delay_start(obs_output_t *output)
return true;
}
if (!obs_output_begin_data_capture2(output)) {
if (!obs_output_begin_data_capture(output, 0)) {
obs_output_cleanup_delay(output);
return false;
}

View File

@@ -2107,8 +2107,11 @@ static inline void signal_stop(struct obs_output *output)
calldata_free(&params);
}
bool obs_output_can_begin_data_capture2(const obs_output_t *output)
bool obs_output_can_begin_data_capture(const obs_output_t *output,
uint32_t flags)
{
UNUSED_PARAMETER(flags);
if (!obs_output_valid(output, "obs_output_can_begin_data_capture"))
return false;
@@ -2123,13 +2126,6 @@ bool obs_output_can_begin_data_capture2(const obs_output_t *output)
return can_begin_data_capture(output);
}
bool obs_output_can_begin_data_capture(const obs_output_t *output,
uint32_t flags)
{
UNUSED_PARAMETER(flags);
return obs_output_can_begin_data_capture2(output);
}
static inline bool initialize_audio_encoders(obs_output_t *output)
{
for (size_t i = 0; i < MAX_OUTPUT_AUDIO_ENCODERS; i++) {
@@ -2179,8 +2175,10 @@ static inline void pair_encoders(obs_output_t *output)
}
}
bool obs_output_initialize_encoders2(obs_output_t *output)
bool obs_output_initialize_encoders(obs_output_t *output, uint32_t flags)
{
UNUSED_PARAMETER(flags);
if (!obs_output_valid(output, "obs_output_initialize_encoders"))
return false;
if (!log_flag_encoded(output, __FUNCTION__, false))
@@ -2201,12 +2199,6 @@ bool obs_output_initialize_encoders2(obs_output_t *output)
return true;
}
bool obs_output_initialize_encoders(obs_output_t *output, uint32_t flags)
{
UNUSED_PARAMETER(flags);
return obs_output_initialize_encoders2(output);
}
static bool begin_delayed_capture(obs_output_t *output)
{
if (delay_capturing(output))
@@ -2263,8 +2255,10 @@ static void reset_raw_output(obs_output_t *output)
pause_reset(&output->pause);
}
bool obs_output_begin_data_capture2(obs_output_t *output)
bool obs_output_begin_data_capture(obs_output_t *output, uint32_t flags)
{
UNUSED_PARAMETER(flags);
if (!obs_output_valid(output, "obs_output_begin_data_capture"))
return false;
@@ -2307,12 +2301,6 @@ bool obs_output_begin_data_capture2(obs_output_t *output)
return true;
}
bool obs_output_begin_data_capture(obs_output_t *output, uint32_t flags)
{
UNUSED_PARAMETER(flags);
return obs_output_begin_data_capture2(output);
}
static inline void stop_audio_encoders(obs_output_t *output,
encoded_callback_t encoded_callback)
{

View File

@@ -2249,15 +2249,13 @@ EXPORT void
obs_output_set_audio_conversion(obs_output_t *output,
const struct audio_convert_info *conversion);
/** Returns whether data capture can begin with the specified flags */
EXPORT bool obs_output_can_begin_data_capture2(const obs_output_t *output);
OBS_DEPRECATED EXPORT bool
obs_output_can_begin_data_capture(const obs_output_t *output, uint32_t flags);
/** Returns whether data capture can begin */
EXPORT bool obs_output_can_begin_data_capture(const obs_output_t *output,
uint32_t flags);
/** Initializes encoders (if any) */
EXPORT bool obs_output_initialize_encoders2(obs_output_t *output);
OBS_DEPRECATED EXPORT bool obs_output_initialize_encoders(obs_output_t *output,
uint32_t flags);
EXPORT bool obs_output_initialize_encoders(obs_output_t *output,
uint32_t flags);
/**
* Begins data capture from media/encoders.
@@ -2265,9 +2263,7 @@ OBS_DEPRECATED EXPORT bool obs_output_initialize_encoders(obs_output_t *output,
* @param output Output context
* @return true if successful, false otherwise.
*/
EXPORT bool obs_output_begin_data_capture2(obs_output_t *output);
OBS_DEPRECATED EXPORT bool obs_output_begin_data_capture(obs_output_t *output,
uint32_t flags);
EXPORT bool obs_output_begin_data_capture(obs_output_t *output, uint32_t flags);
/** Ends data capture from media/encoders */
EXPORT void obs_output_end_data_capture(obs_output_t *output);

View File

@@ -1229,7 +1229,7 @@ static bool aja_output_start(void *data)
obs_output_set_audio_conversion(ajaOutput->GetOBSOutput(), &conversion);
if (!obs_output_begin_data_capture2(ajaOutput->GetOBSOutput())) {
if (!obs_output_begin_data_capture(ajaOutput->GetOBSOutput(), 0)) {
blog(LOG_ERROR,
"aja_output_start: Begin OBS data capture failed!");
return false;

View File

@@ -110,7 +110,7 @@ static bool decklink_output_start(void *data)
obs_output_set_audio_conversion(decklink->GetOutput(), &conversion);
if (!obs_output_begin_data_capture2(decklink->GetOutput()))
if (!obs_output_begin_data_capture(decklink->GetOutput(), 0))
return false;
return true;

View File

@@ -173,7 +173,7 @@ static bool try_connect(void *data, const char *device)
}
blog(LOG_INFO, "Virtual camera started");
obs_output_begin_data_capture2(vcam->output);
obs_output_begin_data_capture(vcam->output, 0);
return true;

View File

@@ -523,7 +523,7 @@ static bool virtualcam_output_start(void *data)
[vcam->machServer run];
}
if (!obs_output_begin_data_capture2(vcam->output)) {
if (!obs_output_begin_data_capture(vcam->output, 0)) {
return false;
}

View File

@@ -118,9 +118,9 @@ bool ffmpeg_hls_mux_start(void *data)
obs_data_t *settings;
int keyint_sec;
if (!obs_output_can_begin_data_capture2(stream->output))
if (!obs_output_can_begin_data_capture(stream->output, 0))
return false;
if (!obs_output_initialize_encoders2(stream->output))
if (!obs_output_initialize_encoders(stream->output, 0))
return false;
service = obs_output_get_service(stream->output);
@@ -170,7 +170,7 @@ bool ffmpeg_hls_mux_start(void *data)
stream->dropped_frames = 0;
stream->min_priority = 0;
obs_output_begin_data_capture2(stream->output);
obs_output_begin_data_capture(stream->output, 0);
dstr_copy(&stream->printable_path, path_str);
info("Writing to path '%s'...", stream->printable_path.array);

View File

@@ -1025,9 +1025,9 @@ static bool set_config(struct ffmpeg_output *stream)
}
av_dump_format(ff_data->output, 0, NULL, 1);
}
if (!obs_output_can_begin_data_capture2(stream->output))
if (!obs_output_can_begin_data_capture(stream->output, 0))
return false;
if (!obs_output_initialize_encoders2(stream->output))
if (!obs_output_initialize_encoders(stream->output, 0))
return false;
ret = pthread_create(&stream->write_thread, NULL, write_thread, stream);
@@ -1042,7 +1042,7 @@ static bool set_config(struct ffmpeg_output *stream)
os_atomic_set_bool(&stream->active, true);
stream->write_thread_active = true;
stream->total_bytes = 0;
obs_output_begin_data_capture2(stream->output);
obs_output_begin_data_capture(stream->output, 0);
return true;
fail:

View File

@@ -411,9 +411,9 @@ static inline bool ffmpeg_mux_start_internal(struct ffmpeg_muxer *stream,
update_encoder_settings(stream, path);
if (!obs_output_can_begin_data_capture2(stream->output))
if (!obs_output_can_begin_data_capture(stream->output, 0))
return false;
if (!obs_output_initialize_encoders2(stream->output))
if (!obs_output_initialize_encoders(stream->output, 0))
return false;
if (stream->is_network) {
@@ -468,7 +468,7 @@ static inline bool ffmpeg_mux_start_internal(struct ffmpeg_muxer *stream,
os_atomic_set_bool(&stream->active, true);
os_atomic_set_bool(&stream->capturing, true);
stream->total_bytes = 0;
obs_output_begin_data_capture2(stream->output);
obs_output_begin_data_capture(stream->output, 0);
info("Writing file '%s'...", stream->path.array);
return true;
@@ -1039,9 +1039,9 @@ static bool replay_buffer_start(void *data)
{
struct ffmpeg_muxer *stream = data;
if (!obs_output_can_begin_data_capture2(stream->output))
if (!obs_output_can_begin_data_capture(stream->output, 0))
return false;
if (!obs_output_initialize_encoders2(stream->output))
if (!obs_output_initialize_encoders(stream->output, 0))
return false;
obs_data_t *s = obs_output_get_settings(stream->output);
@@ -1052,7 +1052,7 @@ static bool replay_buffer_start(void *data)
os_atomic_set_bool(&stream->active, true);
os_atomic_set_bool(&stream->capturing, true);
stream->total_bytes = 0;
obs_output_begin_data_capture2(stream->output);
obs_output_begin_data_capture(stream->output, 0);
return true;
}

View File

@@ -1209,7 +1209,7 @@ static bool try_connect(struct ffmpeg_output *output)
output->active = true;
if (!obs_output_can_begin_data_capture2(output->output))
if (!obs_output_can_begin_data_capture(output->output, 0))
return false;
ret = pthread_create(&output->write_thread, NULL, write_thread, output);
@@ -1223,7 +1223,7 @@ static bool try_connect(struct ffmpeg_output *output)
obs_output_set_video_conversion(output->output, NULL);
obs_output_set_audio_conversion(output->output, &aci);
obs_output_begin_data_capture2(output->output);
obs_output_begin_data_capture(output->output, 0);
output->write_thread_active = true;
return true;
}

View File

@@ -154,9 +154,9 @@ static bool flv_output_start(void *data)
obs_data_t *settings;
const char *path;
if (!obs_output_can_begin_data_capture2(stream->output))
if (!obs_output_can_begin_data_capture(stream->output, 0))
return false;
if (!obs_output_initialize_encoders2(stream->output))
if (!obs_output_initialize_encoders(stream->output, 0))
return false;
stream->got_first_video = false;
@@ -177,7 +177,7 @@ static bool flv_output_start(void *data)
/* write headers and start capture */
os_atomic_set_bool(&stream->active, true);
obs_output_begin_data_capture2(stream->output);
obs_output_begin_data_capture(stream->output, 0);
info("Writing FLV file '%s'...", stream->path.array);
return true;

View File

@@ -593,7 +593,7 @@ static int init_send(struct ftl_stream *stream)
os_atomic_set_bool(&stream->active, true);
obs_output_begin_data_capture2(stream->output);
obs_output_begin_data_capture(stream->output, 0);
return OBS_OUTPUT_SUCCESS;
}
@@ -649,10 +649,10 @@ static bool ftl_stream_start(void *data)
obs_data_set_int(video_settings, "bf", 0);
obs_data_release(video_settings);
if (!obs_output_can_begin_data_capture2(stream->output)) {
if (!obs_output_can_begin_data_capture(stream->output, 0)) {
return false;
}
if (!obs_output_initialize_encoders2(stream->output)) {
if (!obs_output_initialize_encoders(stream->output, 0)) {
return false;
}

View File

@@ -51,15 +51,15 @@ static bool null_output_start(void *data)
{
struct null_output *context = data;
if (!obs_output_can_begin_data_capture2(context->output))
if (!obs_output_can_begin_data_capture(context->output, 0))
return false;
if (!obs_output_initialize_encoders2(context->output))
if (!obs_output_initialize_encoders(context->output, 0))
return false;
if (context->stop_thread_active)
pthread_join(context->stop_thread, NULL);
obs_output_begin_data_capture2(context->output);
obs_output_begin_data_capture(context->output, 0);
return true;
}

View File

@@ -1071,7 +1071,7 @@ static int init_send(struct rtmp_stream *stream)
return OBS_OUTPUT_DISCONNECTED;
}
obs_output_begin_data_capture2(stream->output);
obs_output_begin_data_capture(stream->output, 0);
return OBS_OUTPUT_SUCCESS;
}
@@ -1373,9 +1373,9 @@ static bool rtmp_stream_start(void *data)
{
struct rtmp_stream *stream = data;
if (!obs_output_can_begin_data_capture2(stream->output))
if (!obs_output_can_begin_data_capture(stream->output, 0))
return false;
if (!obs_output_initialize_encoders2(stream->output))
if (!obs_output_initialize_encoders(stream->output, 0))
return false;
os_atomic_set_bool(&stream->connecting, true);

View File

@@ -68,7 +68,7 @@ static bool virtualcam_start(void *data)
os_atomic_set_bool(&vcam->active, true);
os_atomic_set_bool(&vcam->stopping, false);
blog(LOG_INFO, "Virtual output started");
obs_output_begin_data_capture2(vcam->output);
obs_output_begin_data_capture(vcam->output, 0);
return true;
}