[MPlayer-dev-eng] [PATCH] haiku support

Reimar Döffinger Reimar.Doeffinger at gmx.de
Fri Sep 8 09:10:48 EEST 2017


On 02.09.2017, at 22:28, Compn <tempn at mi.rr.com> wrote:

> patch looks ok to me.
> 
> maintained by miqlas.
> 
> any haiku users out there? :)

Some of the files lack license headers, I don't think the ao/vo modules should be enabled unconditionally,
the libraries should be linked against depending on enabed modules, not depending on OS,
the header include guards don't have the right format (especially should not start with _),
several files miss license headers (and I don't like the MIT license one either, because then I have to check of it's compatible with GPL, plus the actual license needs to be included as well).


More information about the MPlayer-dev-eng mailing list