[libav-devel] [PATCH] rtmpproto: check config before variables

Vittorio Giovara vittorio.giovara at gmail.com
Sun Nov 3 11:45:45 CET 2013


On Sun, Nov 3, 2013 at 11:11 AM, Martin Storsjö <martin at martin.st> wrote:
> On Sun, 3 Nov 2013, Vittorio Giovara wrote:
>
>> This fixes missing symbols when using clang and no optimiziations
>> configuration.
>>
>> Based on a suggestion by Martin Storsjö <martin at martin.st>
>> ---
>> libavformat/rtmpproto.c | 12 ++++++------
>> 1 file changed, 6 insertions(+), 6 deletions(-)
>
>
> The patch itself is ok with me.
>
> What about this as commit message:
>
> ---8<---
> rtmpproto: Reorder conditions to help dead code elimination
>
> This makes sure that these branches are eliminated properly with clang with
> optimizations disabled.
> ---8<---
>

Sure, amended locally.
Vittorio


More information about the libav-devel mailing list