[FFmpeg-devel] [PATCH] avdevice/decklink_enc: fix operator precedence in create_s337_payload

[FFmpeg-devel] [PATCH] avdevice/decklink_enc: fix operator precedence in create_s337_payload

[Marton Balint]

Signed-off-by: Marton Balint <cus@passwd.hu>
---
 libavdevice/decklink_enc.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavdevice/decklink_enc.cpp b/libavdevice/decklink_enc.cpp
index 92bfdb279f..9f1a8df438 100644
--- a/libavdevice/decklink_enc.cpp
+++ b/libavdevice/decklink_enc.cpp
@@ -318,7 +318,7 @@ static int create_s337_payload(AVPacket *pkt, uint8_t **outbuf, int *outsize)
     /* Ensure final payload is aligned on 4-byte boundary */
     if (pkt->size & 1)
         bytestream2_put_le16u(&pb, pkt->data[pkt->size - 1] << 8);
-    if ((pkt->size & 3 == 1) || (pkt->size & 3 == 2))
+    if ((pkt->size & 3) == 1 || (pkt->size & 3) == 2)
         bytestream2_put_le16u(&pb, 0);
 
     *outsize = payload_size;
-- 
2.35.3

_______________________________________________
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".

Re: [FFmpeg-devel] [PATCH] avdevice/decklink_enc: fix operator precedence in create_s337_payload

[Marton Balint]

On Sun, 16 Apr 2023, Marton Balint wrote:

> Signed-off-by: Marton Balint <cus@passwd.hu>
> ---
> libavdevice/decklink_enc.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Will apply.

Regards,
Marton

>
> diff --git a/libavdevice/decklink_enc.cpp b/libavdevice/decklink_enc.cpp
> index 92bfdb279f..9f1a8df438 100644
> --- a/libavdevice/decklink_enc.cpp
> +++ b/libavdevice/decklink_enc.cpp
> @@ -318,7 +318,7 @@ static int create_s337_payload(AVPacket *pkt, uint8_t **outbuf, int *outsize)
>     /* Ensure final payload is aligned on 4-byte boundary */
>     if (pkt->size & 1)
>         bytestream2_put_le16u(&pb, pkt->data[pkt->size - 1] << 8);
> -    if ((pkt->size & 3 == 1) || (pkt->size & 3 == 2))
> +    if ((pkt->size & 3) == 1 || (pkt->size & 3) == 2)
>         bytestream2_put_le16u(&pb, 0);
>
>     *outsize = payload_size;
> -- 
> 2.35.3
>
> _______________________________________________
> 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".