* [FFmpeg-devel] [PATCH 4/7] avutil: Fix mismatching argument names
@ 2022-09-15 1:52 Marvin Scholz
2022-09-15 1:52 ` [FFmpeg-devel] [PATCH 5/7] swresample/swresample: " Marvin Scholz
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Marvin Scholz @ 2022-09-15 1:52 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Marvin Scholz
---
libavutil/hwcontext.h | 2 +-
libavutil/mathematics.h | 6 ++++--
libavutil/mem.h | 3 ++-
libavutil/rational.h | 3 ++-
4 files changed, 9 insertions(+), 5 deletions(-)
diff --git a/libavutil/hwcontext.h b/libavutil/hwcontext.h
index c18b7e1e8b..efca17585e 100644
--- a/libavutil/hwcontext.h
+++ b/libavutil/hwcontext.h
@@ -249,7 +249,7 @@ const char *av_hwdevice_get_type_name(enum AVHWDeviceType type);
/**
* Iterate over supported device types.
*
- * @param type AV_HWDEVICE_TYPE_NONE initially, then the previous type
+ * @param prev AV_HWDEVICE_TYPE_NONE initially, then the previous type
* returned by this function in subsequent iterations.
* @return The next usable device type from enum AVHWDeviceType, or
* AV_HWDEVICE_TYPE_NONE if there are no more.
diff --git a/libavutil/mathematics.h b/libavutil/mathematics.h
index 64d4137a60..e4aff1e973 100644
--- a/libavutil/mathematics.h
+++ b/libavutil/mathematics.h
@@ -111,7 +111,8 @@ enum AVRounding {
/**
* Compute the greatest common divisor of two integer operands.
*
- * @param a,b Operands
+ * @param a Operand
+ * @param b Operand
* @return GCD of a and b up to sign; if a >= 0 and b >= 0, return value is >= 0;
* if a == 0 and b == 0, returns 0.
*/
@@ -186,7 +187,8 @@ int av_compare_ts(int64_t ts_a, AVRational tb_a, int64_t ts_b, AVRational tb_b);
* av_compare_mod(0x11, 0x02, 0x20) > 0 // since 0x11 % 0x20 (0x11) > 0x02 % 0x20 (0x02)
* @endcode
*
- * @param a,b Operands
+ * @param a Operand
+ * @param b Operand
* @param mod Divisor; must be a power of 2
* @return
* - a negative value if `a % mod < b % mod`
diff --git a/libavutil/mem.h b/libavutil/mem.h
index d91174196c..c9c4fcf1ff 100644
--- a/libavutil/mem.h
+++ b/libavutil/mem.h
@@ -667,7 +667,8 @@ void *av_dynarray2_add(void **tab_ptr, int *nb_ptr, size_t elem_size,
/**
* Multiply two `size_t` values checking for overflow.
*
- * @param[in] a,b Operands of multiplication
+ * @param[in] a Operand of multiplication
+ * @param[in] b Operand of multiplication
* @param[out] r Pointer to the result of the operation
* @return 0 on success, AVERROR(EINVAL) on overflow
*/
diff --git a/libavutil/rational.h b/libavutil/rational.h
index cbb08a0baf..8cbfc8e066 100644
--- a/libavutil/rational.h
+++ b/libavutil/rational.h
@@ -179,7 +179,8 @@ AVRational av_d2q(double d, int max) av_const;
* Find which of the two rationals is closer to another rational.
*
* @param q Rational to be compared against
- * @param q1,q2 Rationals to be tested
+ * @param q1 Rational to be tested
+ * @param q2 Rational to be tested
* @return One of the following values:
* - 1 if `q1` is nearer to `q` than `q2`
* - -1 if `q2` is nearer to `q` than `q1`
--
2.37.0 (Apple Git-136)
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
* [FFmpeg-devel] [PATCH 5/7] swresample/swresample: Fix mismatching argument names
2022-09-15 1:52 [FFmpeg-devel] [PATCH 4/7] avutil: Fix mismatching argument names Marvin Scholz
@ 2022-09-15 1:52 ` Marvin Scholz
2022-09-15 2:12 ` James Almer
2022-09-15 1:53 ` [FFmpeg-devel] [PATCH 6/7] avformat/avformat: Improve doxy style Marvin Scholz
2022-09-15 1:53 ` [FFmpeg-devel] [PATCH 7/7] avcodec/avcodec: Escape Doxygen reference Marvin Scholz
2 siblings, 1 reply; 6+ messages in thread
From: Marvin Scholz @ 2022-09-15 1:52 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Marvin Scholz
---
libswresample/swresample.h | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/libswresample/swresample.h b/libswresample/swresample.h
index 980be65783..83821a0930 100644
--- a/libswresample/swresample.h
+++ b/libswresample/swresample.h
@@ -354,8 +354,8 @@ int swr_convert(struct SwrContext *s, uint8_t **out, int out_count,
* in this case the output timestamps will match output sample numbers.
* See ffmpeg-resampler(1) for the two modes of compensation.
*
- * @param s[in] initialized Swr context
- * @param pts[in] timestamp for the next input sample, INT64_MIN if unknown
+ * @param[in] s initialized Swr context
+ * @param[in] pts timestamp for the next input sample, INT64_MIN if unknown
* @see swr_set_compensation(), swr_drop_output(), and swr_inject_silence() are
* function used internally for timestamp compensation.
* @return the output timestamp for the next output sample
@@ -636,8 +636,8 @@ int swr_convert_frame(SwrContext *swr,
* @see swr_close();
*
* @param swr audio resample context
- * @param output output AVFrame
- * @param input input AVFrame
+ * @param output out AVFrame
+ * @param input in AVFrame
* @return 0 on success, AVERROR on failure.
*/
int swr_config_frame(SwrContext *swr, const AVFrame *out, const AVFrame *in);
--
2.37.0 (Apple Git-136)
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
* [FFmpeg-devel] [PATCH 6/7] avformat/avformat: Improve doxy style
2022-09-15 1:52 [FFmpeg-devel] [PATCH 4/7] avutil: Fix mismatching argument names Marvin Scholz
2022-09-15 1:52 ` [FFmpeg-devel] [PATCH 5/7] swresample/swresample: " Marvin Scholz
@ 2022-09-15 1:53 ` Marvin Scholz
2022-09-15 1:53 ` [FFmpeg-devel] [PATCH 7/7] avcodec/avcodec: Escape Doxygen reference Marvin Scholz
2 siblings, 0 replies; 6+ messages in thread
From: Marvin Scholz @ 2022-09-15 1:53 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Marvin Scholz
Mostly re-indenting and adding some missing references.
---
libavformat/avformat.h | 180 +++++++++++++++++++++++------------------
1 file changed, 103 insertions(+), 77 deletions(-)
diff --git a/libavformat/avformat.h b/libavformat/avformat.h
index c695cfc6de..1d97d56ac5 100644
--- a/libavformat/avformat.h
+++ b/libavformat/avformat.h
@@ -1898,8 +1898,8 @@ const AVOutputFormat *av_muxer_iterate(void **opaque);
/**
* Iterate over all registered demuxers.
*
- * @param opaque a pointer where libavformat will store the iteration state. Must
- * point to NULL to start the iteration.
+ * @param opaque a pointer where libavformat will store the iteration state.
+ * Must point to NULL to start the iteration.
*
* @return the next registered demuxer or NULL when the iteration is
* finished
@@ -1957,12 +1957,13 @@ AVStream *avformat_new_stream(AVFormatContext *s, const AVCodec *c);
/**
* Wrap an existing array as stream side data.
*
- * @param st stream
+ * @param st stream
* @param type side information type
* @param data the side data array. It must be allocated with the av_malloc()
* family of functions. The ownership of the data is transferred to
* st.
* @param size side information size
+ *
* @return zero on success, a negative AVERROR code on failure. On failure,
* the stream is unchanged and the data remains owned by the caller.
*/
@@ -1973,8 +1974,9 @@ int av_stream_add_side_data(AVStream *st, enum AVPacketSideDataType type,
* Allocate new information from stream.
*
* @param stream stream
- * @param type desired side information type
- * @param size side information size
+ * @param type desired side information type
+ * @param size side information size
+ *
* @return pointer to fresh allocated data or NULL otherwise
*/
uint8_t *av_stream_new_side_data(AVStream *stream,
@@ -1983,9 +1985,10 @@ uint8_t *av_stream_new_side_data(AVStream *stream,
* Get side information from stream.
*
* @param stream stream
- * @param type desired side information type
- * @param size If supplied, *size will be set to the size of the side data
- * or to zero if the desired side data is not present.
+ * @param type desired side information type
+ * @param size If supplied, *size will be set to the size of the side data
+ * or to zero if the desired side data is not present.
+ *
* @return pointer to data if present or NULL otherwise
*/
uint8_t *av_stream_get_side_data(const AVStream *stream,
@@ -2003,16 +2006,17 @@ AVProgram *av_new_program(AVFormatContext *s, int id);
* avformat_free_context() can be used to free the context and
* everything allocated by the framework within it.
*
- * @param ctx pointee is set to the created format context, or to NULL in
- * case of failure
- * @param oformat format to use for allocating the context, if NULL
- * format_name and filename are used instead
- * @param format_name the name of output format to use for allocating the
- * context, if NULL filename is used instead
- * @param filename the name of the filename to use for allocating the
- * context, may be NULL
- * @return >= 0 in case of success, a negative AVERROR code in case of
- * failure
+ * @param ctx pointee is set to the created format context,
+ * or to NULL in case of failure
+ * @param oformat format to use for allocating the context, if NULL
+ * format_name and filename are used instead
+ * @param format_name the name of output format to use for allocating the
+ * context, if NULL filename is used instead
+ * @param filename the name of the filename to use for allocating the
+ * context, may be NULL
+ *
+ * @return >= 0 in case of success, a negative AVERROR code in case of
+ * failure
*/
int avformat_alloc_output_context2(AVFormatContext **ctx, const AVOutputFormat *oformat,
const char *format_name, const char *filename);
@@ -2067,15 +2071,16 @@ const AVInputFormat *av_probe_input_format3(const AVProbeData *pd,
* attempt is made. When the maximum probe size is reached, the input format
* with the highest score is returned.
*
- * @param pb the bytestream to probe
- * @param fmt the input format is put here
- * @param url the url of the stream
- * @param logctx the log context
- * @param offset the offset within the bytestream to probe from
+ * @param pb the bytestream to probe
+ * @param fmt the input format is put here
+ * @param url the url of the stream
+ * @param logctx the log context
+ * @param offset the offset within the bytestream to probe from
* @param max_probe_size the maximum probe buffer size (zero for default)
+ *
* @return the score in case of success, a negative value corresponding to an
* the maximal score is AVPROBE_SCORE_MAX
- * AVERROR code otherwise
+ * AVERROR code otherwise
*/
int av_probe_input_buffer2(AVIOContext *pb, const AVInputFormat **fmt,
const char *url, void *logctx,
@@ -2092,16 +2097,19 @@ int av_probe_input_buffer(AVIOContext *pb, const AVInputFormat **fmt,
* Open an input stream and read the header. The codecs are not opened.
* The stream must be closed with avformat_close_input().
*
- * @param ps Pointer to user-supplied AVFormatContext (allocated by avformat_alloc_context).
- * May be a pointer to NULL, in which case an AVFormatContext is allocated by this
- * function and written into ps.
- * Note that a user-supplied AVFormatContext will be freed on failure.
- * @param url URL of the stream to open.
- * @param fmt If non-NULL, this parameter forces a specific input format.
- * Otherwise the format is autodetected.
- * @param options A dictionary filled with AVFormatContext and demuxer-private options.
- * On return this parameter will be destroyed and replaced with a dict containing
- * options that were not found. May be NULL.
+ * @param ps Pointer to user-supplied AVFormatContext (allocated by
+ * avformat_alloc_context). May be a pointer to NULL, in
+ * which case an AVFormatContext is allocated by this
+ * function and written into ps.
+ * Note that a user-supplied AVFormatContext will be freed
+ * on failure.
+ * @param url URL of the stream to open.
+ * @param fmt If non-NULL, this parameter forces a specific input format.
+ * Otherwise the format is autodetected.
+ * @param options A dictionary filled with AVFormatContext and demuxer-private
+ * options.
+ * On return this parameter will be destroyed and replaced with
+ * a dict containing options that were not found. May be NULL.
*
* @return 0 on success, a negative AVERROR on failure.
*
@@ -2140,6 +2148,7 @@ int avformat_find_stream_info(AVFormatContext *ic, AVDictionary **options);
* @param last the last found program, the search will start after this
* program, or from the beginning if it is NULL
* @param s stream index
+ *
* @return the next program which belongs to s, NULL if no program is found or
* the last program is not among the programs of ic.
*/
@@ -2164,10 +2173,12 @@ void av_program_add_stream_index(AVFormatContext *ac, int progid, unsigned int i
* @param decoder_ret if non-NULL, returns the decoder for the
* selected stream
* @param flags flags; none are currently defined
+ *
* @return the non-negative stream number in case of success,
* AVERROR_STREAM_NOT_FOUND if no stream with the requested type
* could be found,
* AVERROR_DECODER_NOT_FOUND if streams were found but no decoder
+ *
* @note If av_find_best_stream returns successfully and decoder_ret is not
* NULL, then *decoder_ret is guaranteed to be set to a valid AVCodec.
*/
@@ -2211,13 +2222,14 @@ int av_read_frame(AVFormatContext *s, AVPacket *pkt);
* Seek to the keyframe at timestamp.
* 'timestamp' in 'stream_index'.
*
- * @param s media file handle
- * @param stream_index If stream_index is (-1), a default
- * stream is selected, and timestamp is automatically converted
- * from AV_TIME_BASE units to the stream specific time_base.
- * @param timestamp Timestamp in AVStream.time_base units
- * or, if no stream is specified, in AV_TIME_BASE units.
- * @param flags flags which select direction and seeking mode
+ * @param s media file handle
+ * @param stream_index If stream_index is (-1), a default stream is selected,
+ * and timestamp is automatically converted from
+ * AV_TIME_BASE units to the stream specific time_base.
+ * @param timestamp Timestamp in AVStream.time_base units or, if no stream
+ * is specified, in AV_TIME_BASE units.
+ * @param flags flags which select direction and seeking mode
+ *
* @return >= 0 on success
*/
int av_seek_frame(AVFormatContext *s, int stream_index, int64_t timestamp,
@@ -2239,12 +2251,12 @@ int av_seek_frame(AVFormatContext *s, int stream_index, int64_t timestamp,
* keyframes (this may not be supported by all demuxers).
* If flags contain AVSEEK_FLAG_BACKWARD, it is ignored.
*
- * @param s media file handle
+ * @param s media file handle
* @param stream_index index of the stream which is used as time base reference
- * @param min_ts smallest acceptable timestamp
- * @param ts target timestamp
- * @param max_ts largest acceptable timestamp
- * @param flags flags
+ * @param min_ts smallest acceptable timestamp
+ * @param ts target timestamp
+ * @param max_ts largest acceptable timestamp
+ * @param flags flags
* @return >=0 on success, error code otherwise
*
* @note This is part of the new seek API which is still under construction.
@@ -2308,16 +2320,22 @@ void avformat_close_input(AVFormatContext **s);
* Allocate the stream private data and write the stream header to
* an output media file.
*
- * @param s Media file handle, must be allocated with avformat_alloc_context().
- * Its oformat field must be set to the desired output format;
- * Its pb field must be set to an already opened AVIOContext.
- * @param options An AVDictionary filled with AVFormatContext and muxer-private options.
- * On return this parameter will be destroyed and replaced with a dict containing
- * options that were not found. May be NULL.
- *
- * @return AVSTREAM_INIT_IN_WRITE_HEADER on success if the codec had not already been fully initialized in avformat_init,
- * AVSTREAM_INIT_IN_INIT_OUTPUT on success if the codec had already been fully initialized in avformat_init,
- * negative AVERROR on failure.
+ * @param s Media file handle, must be allocated with
+ * avformat_alloc_context().
+ * Its \ref AVFormatContext.oformat "oformat" field must be set
+ * to the desired output format;
+ * Its \ref AVFormatContext.pb "pb" field must be set to an
+ * already opened ::AVIOContext.
+ * @param options An ::AVDictionary filled with AVFormatContext and
+ * muxer-private options.
+ * On return this parameter will be destroyed and replaced with
+ * a dict containing options that were not found. May be NULL.
+ *
+ * @retval AVSTREAM_INIT_IN_WRITE_HEADER On success, if the codec had not already been
+ * fully initialized in avformat_init_output().
+ * @retval AVSTREAM_INIT_IN_INIT_OUTPUT On success, if the codec had already been fully
+ * initialized in avformat_init_output().
+ * @retval AVERROR A negative AVERROR on failure.
*
* @see av_opt_find, av_dict_set, avio_open, av_oformat_next, avformat_init_output.
*/
@@ -2326,20 +2344,26 @@ int avformat_write_header(AVFormatContext *s, AVDictionary **options);
/**
* Allocate the stream private data and initialize the codec, but do not write the header.
- * May optionally be used before avformat_write_header to initialize stream parameters
+ * May optionally be used before avformat_write_header() to initialize stream parameters
* before actually writing the header.
- * If using this function, do not pass the same options to avformat_write_header.
- *
- * @param s Media file handle, must be allocated with avformat_alloc_context().
- * Its oformat field must be set to the desired output format;
- * Its pb field must be set to an already opened AVIOContext.
- * @param options An AVDictionary filled with AVFormatContext and muxer-private options.
- * On return this parameter will be destroyed and replaced with a dict containing
- * options that were not found. May be NULL.
- *
- * @return AVSTREAM_INIT_IN_WRITE_HEADER on success if the codec requires avformat_write_header to fully initialize,
- * AVSTREAM_INIT_IN_INIT_OUTPUT on success if the codec has been fully initialized,
- * negative AVERROR on failure.
+ * If using this function, do not pass the same options to avformat_write_header().
+ *
+ * @param s Media file handle, must be allocated with
+ * avformat_alloc_context().
+ * Its \ref AVFormatContext.oformat "oformat" field must be set
+ * to the desired output format;
+ * Its \ref AVFormatContext.pb "pb" field must be set to an
+ * already opened ::AVIOContext.
+ * @param options An ::AVDictionary filled with AVFormatContext and
+ * muxer-private options.
+ * On return this parameter will be destroyed and replaced with
+ * a dict containing options that were not found. May be NULL.
+ *
+ * @retval AVSTREAM_INIT_IN_WRITE_HEADER On success, if the codec requires
+ * avformat_write_header to fully initialize.
+ * @retval AVSTREAM_INIT_IN_INIT_OUTPUT On success, if the codec has been fully
+ * initialized.
+ * @retval AVERROR Anegative AVERROR on failure.
*
* @see av_opt_find, av_dict_set, avio_open, av_oformat_next, avformat_write_header.
*/
@@ -2484,11 +2508,11 @@ int av_write_trailer(AVFormatContext *s);
* there is no match.
*
* @param short_name if non-NULL checks if short_name matches with the
- * names of the registered formats
- * @param filename if non-NULL checks if filename terminates with the
- * extensions of the registered formats
- * @param mime_type if non-NULL checks if mime_type matches with the
- * MIME type of the registered formats
+ * names of the registered formats
+ * @param filename if non-NULL checks if filename terminates with the
+ * extensions of the registered formats
+ * @param mime_type if non-NULL checks if mime_type matches with the
+ * MIME type of the registered formats
*/
const AVOutputFormat *av_guess_format(const char *short_name,
const char *filename,
@@ -2512,9 +2536,11 @@ enum AVCodecID av_guess_codec(const AVOutputFormat *fmt, const char *short_name,
* time_base units
* @param[out] wall absolute time when that packet whas output,
* in microsecond
- * @return 0 if OK, AVERROR(ENOSYS) if the format does not support it
- * Note: some formats or devices may not allow to measure dts and wall
- * atomically.
+ * @retval 0 Success
+ * @retval AVERROR(ENOSYS) The format does not support it
+ *
+ * @note Some formats or devices may not allow to measure dts and wall
+ * atomically.
*/
int av_get_output_timestamp(struct AVFormatContext *s, int stream,
int64_t *dts, int64_t *wall);
--
2.37.0 (Apple Git-136)
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
* [FFmpeg-devel] [PATCH 7/7] avcodec/avcodec: Escape Doxygen reference
2022-09-15 1:52 [FFmpeg-devel] [PATCH 4/7] avutil: Fix mismatching argument names Marvin Scholz
2022-09-15 1:52 ` [FFmpeg-devel] [PATCH 5/7] swresample/swresample: " Marvin Scholz
2022-09-15 1:53 ` [FFmpeg-devel] [PATCH 6/7] avformat/avformat: Improve doxy style Marvin Scholz
@ 2022-09-15 1:53 ` Marvin Scholz
2 siblings, 0 replies; 6+ messages in thread
From: Marvin Scholz @ 2022-09-15 1:53 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Marvin Scholz
The # is interpreted as explicit reference request by Doxygen
which is not desired here.
Additionally use a typewriter font to make the inline code
easier to distinguish.
---
libavcodec/avcodec.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
index 665a5eadaf..f0904520b1 100644
--- a/libavcodec/avcodec.h
+++ b/libavcodec/avcodec.h
@@ -1518,7 +1518,7 @@ typedef struct AVCodecContext {
* libavcodec will behave as if this field was always set to 1.
* Callers that want to be forward compatible with future libavcodec
* versions should wrap access to this field in
- * #if LIBAVCODEC_VERSION_MAJOR < 60
+ * <tt>\#if LIBAVCODEC_VERSION_MAJOR < 60</tt>
*/
attribute_deprecated
int thread_safe_callbacks;
--
2.37.0 (Apple Git-136)
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [FFmpeg-devel] [PATCH 5/7] swresample/swresample: Fix mismatching argument names
2022-09-15 1:52 ` [FFmpeg-devel] [PATCH 5/7] swresample/swresample: " Marvin Scholz
@ 2022-09-15 2:12 ` James Almer
2022-09-15 7:24 ` Marvin Scholz
0 siblings, 1 reply; 6+ messages in thread
From: James Almer @ 2022-09-15 2:12 UTC (permalink / raw)
To: ffmpeg-devel
On 9/14/2022 10:52 PM, Marvin Scholz wrote:
> ---
> libswresample/swresample.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/libswresample/swresample.h b/libswresample/swresample.h
> index 980be65783..83821a0930 100644
> --- a/libswresample/swresample.h
> +++ b/libswresample/swresample.h
> @@ -354,8 +354,8 @@ int swr_convert(struct SwrContext *s, uint8_t **out, int out_count,
> * in this case the output timestamps will match output sample numbers.
> * See ffmpeg-resampler(1) for the two modes of compensation.
> *
> - * @param s[in] initialized Swr context
> - * @param pts[in] timestamp for the next input sample, INT64_MIN if unknown
> + * @param[in] s initialized Swr context
> + * @param[in] pts timestamp for the next input sample, INT64_MIN if unknown
> * @see swr_set_compensation(), swr_drop_output(), and swr_inject_silence() are
> * function used internally for timestamp compensation.
> * @return the output timestamp for the next output sample
> @@ -636,8 +636,8 @@ int swr_convert_frame(SwrContext *swr,
> * @see swr_close();
> *
> * @param swr audio resample context
> - * @param output output AVFrame
> - * @param input input AVFrame
> + * @param output out AVFrame
> + * @param input in AVFrame
@param out output AVFrame
@param in input AVFrame
> * @return 0 on success, AVERROR on failure.
> */
> int swr_config_frame(SwrContext *swr, const AVFrame *out, const AVFrame *in);
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [FFmpeg-devel] [PATCH 5/7] swresample/swresample: Fix mismatching argument names
2022-09-15 2:12 ` James Almer
@ 2022-09-15 7:24 ` Marvin Scholz
0 siblings, 0 replies; 6+ messages in thread
From: Marvin Scholz @ 2022-09-15 7:24 UTC (permalink / raw)
To: FFmpeg development discussions and patches
On 15 Sep 2022, at 4:12, James Almer wrote:
> On 9/14/2022 10:52 PM, Marvin Scholz wrote:
>> ---
>> libswresample/swresample.h | 8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/libswresample/swresample.h b/libswresample/swresample.h
>> index 980be65783..83821a0930 100644
>> --- a/libswresample/swresample.h
>> +++ b/libswresample/swresample.h
>> @@ -354,8 +354,8 @@ int swr_convert(struct SwrContext *s, uint8_t **out, int out_count,
>> * in this case the output timestamps will match output sample numbers.
>> * See ffmpeg-resampler(1) for the two modes of compensation.
>> *
>> - * @param s[in] initialized Swr context
>> - * @param pts[in] timestamp for the next input sample, INT64_MIN if unknown
>> + * @param[in] s initialized Swr context
>> + * @param[in] pts timestamp for the next input sample, INT64_MIN if unknown
>> * @see swr_set_compensation(), swr_drop_output(), and swr_inject_silence() are
>> * function used internally for timestamp compensation.
>> * @return the output timestamp for the next output sample
>> @@ -636,8 +636,8 @@ int swr_convert_frame(SwrContext *swr,
>> * @see swr_close();
>> *
>> * @param swr audio resample context
>> - * @param output output AVFrame
>> - * @param input input AVFrame
>> + * @param output out AVFrame
>> + * @param input in AVFrame
>
> @param out output AVFrame
> @param in input AVFrame
>
Whoops, thanks. Fixed, will wait a bit in case there are other
reviews before I send a v2.
>> * @return 0 on success, AVERROR on failure.
>> */
>> int swr_config_frame(SwrContext *swr, const AVFrame *out, const AVFrame *in);
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-09-15 7:24 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-15 1:52 [FFmpeg-devel] [PATCH 4/7] avutil: Fix mismatching argument names Marvin Scholz
2022-09-15 1:52 ` [FFmpeg-devel] [PATCH 5/7] swresample/swresample: " Marvin Scholz
2022-09-15 2:12 ` James Almer
2022-09-15 7:24 ` Marvin Scholz
2022-09-15 1:53 ` [FFmpeg-devel] [PATCH 6/7] avformat/avformat: Improve doxy style Marvin Scholz
2022-09-15 1:53 ` [FFmpeg-devel] [PATCH 7/7] avcodec/avcodec: Escape Doxygen reference Marvin Scholz
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git