[MPlayer-dev-eng] [PATCH] build: Check if SCTP is available
Alexander Strasser
eclipse7 at gmx.net
Sun May 13 22:04:56 CEST 2012
Hi Reimar,
Reimar Döffinger wrote:
> On 13 May 2012, at 18:33, Alexander Strasser <eclipse7 at gmx.net> wrote:
[...]
> > I did miss it is available in MPlayer's configure. But I found it
> > now under the name header_check.
>
> IMO please commit.
> Though you could replace the innermost if by &&, i.e.
> header_check ... && sctp=yes
Committed.
I did see the opportunity to get rid of this "if" but as it is
a matter of personal taste I wasn't sure what to do; so I left it
as it was before in Johan's patch. I was under the impression his
patch was strongly modelled after another existing check.
As the matter is quite annoying and multiple users (rightly)
complained until now I did not want to edit the tested patch.
If you insist on avoiding the "if" I can change it locally and
commit after I tested it.
Alexander
More information about the MPlayer-dev-eng
mailing list