On 10/25/2021 10:20 AM, Michael Niedermayer wrote:
On Mon, Oct 25, 2021 at 10:13:32AM -0300, James Almer wrote:
On 10/23/2021 5:12 AM, Michael Niedermayer wrote:
Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc>
---
libswscale/swscale.h | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/libswscale/swscale.h b/libswscale/swscale.h
index 59610d03426..daa53dc01ee 100644
--- a/libswscale/swscale.h
+++ b/libswscale/swscale.h
@@ -318,14 +318,22 @@ unsigned int sws_receive_slice_alignment(const struct
SwsContext *c);
* @param brightness 16.16 fixed point brightness correction
* @param contrast 16.16 fixed point contrast correction
* @param saturation 16.16 fixed point saturation correction
+#if LIBSWSCALE_VERSION_MAJOR > 6
+ * @return negative error code on error, non negative otherwise
+#else
* @return -1 if not supported
+#endif
*/
int sws_setColorspaceDetails(struct SwsContext *c, const int inv_table[4],
int srcRange, const int table[4], int dstRange,
int brightness, int contrast, int saturation);
/**
+#if LIBSWSCALE_VERSION_MAJOR > 6
+ * @return negative error code on error, non negative otherwise
Strictly speaking, this has been the behavior for a while (Since the
av_image_alloc() call was added, at least).
IMO just change the line and don't wrap it like this with a preprocessor
check, especially if you're not also adding a relevant line to APIChanges.
didnt see your message before pushing
feel free to change it any way you prefer
I think you pushed the patch while i was writing it, so i was simply late.
thx
[...]
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@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-requ...@ffmpeg.org with subject "unsubscribe".
