Re: [PATCH] haiku support

Reimar Döffinger <[email protected]>
Newsgroups gmane.comp.video.mplayer.devel
Message-ID <[email protected]>
On 02.09.2017, at 22:28, Compn <[email protected]> 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).
_______________________________________________
MPlayer-dev-eng mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.