diff mbox

[FFmpeg-devel] ffserver: set format bitexact flag, eliminate warnings about it not being set

Message ID 20161201170251.28717-1-michael@niedermayer.cc
State Accepted
Commit fbdf8f176e0fbf48132e45445f538d74c74107a4
Headers show

Commit Message

Michael Niedermayer Dec. 1, 2016, 5:02 p.m. UTC 
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 ffserver.c | 3 +++
 1 file changed, 3 insertions(+)

Comments

Reynaldo H. Verdejo Pinochet Dec. 1, 2016, 5:43 p.m. UTC | #1 
Looks good & works correctly Michael. Please feel free to push. Thank you.

On 12/01/2016 09:02 AM, Michael Niedermayer wrote:
> if (src->codec->flags & AV_CODEC_FLAG_BITEXACT)
> +                c->pfmt_ctx->flags |= AVFMT_FLAG_BITEXACT;
Reto Kromer Dec. 1, 2016, 5:50 p.m. UTC | #2 
Michael Niedermayer wrote:

>+
>+            if (src->codec->flags & AV_CODEC_FLAG_BITEXACT)
>+                c->pfmt_ctx->flags |= AVFMT_FLAG_BITEXACT;

Works fine here. Best regards, Reto
Michael Niedermayer Dec. 1, 2016, 7:08 p.m. UTC | #3 
On Thu, Dec 01, 2016 at 09:43:34AM -0800, Reynaldo H. Verdejo Pinochet wrote:
> Looks good & works correctly Michael. Please feel free to push. Thank you.

applied

thx

[...]
diff mbox

Patch

diff --git a/ffserver.c b/ffserver.c
index 4e5ce1f..19c3ced 100644
--- a/ffserver.c
+++ b/ffserver.c
@@ -2323,6 +2323,9 @@  static int http_prepare_data(HTTPContext *c)
 
             unlayer_stream(c->pfmt_ctx->streams[i], src); //TODO we no longer copy st->internal, does this matter?
             av_assert0(!c->pfmt_ctx->streams[i]->priv_data);
+
+            if (src->codec->flags & AV_CODEC_FLAG_BITEXACT)
+                c->pfmt_ctx->flags |= AVFMT_FLAG_BITEXACT;
         }
         /* set output format parameters */
         c->pfmt_ctx->oformat = c->stream->fmt;