Support mplayer2 and mpv

[Nikoli]

Did not find anything about mplayer2 and mpv in git log -p, please add support for them.

[kokoko3k]

As far as i tested, mplayer2 works good.
Could it be that some extra options are not handled directly by xt7, but you can use them by using custom filters and custom options.
For mpv, i didn't even knew it existed; and from what i've read, mpv changes some options naming.
I'm sorry, but i'm not going to add support for mpv.
if you find something specific to mplayer2 that is blocking xt7 from running fine, feel free to open a bug report.

[Nikoli]

Now mpv has libmpv shared library, what is your opinion about API?
mpv-player/mpv#97 (comment)

[kokoko3k]

Sorry for the late answer.
Xt7 uses slave mode to talk to mplayer and mplayer2, which is actually
as simple as writing and reading strings from a pipe.
Actually mpv has a broken slave mode; so i'll consider to support it
when it will work.
Using a shared library would mean to completely rewrite a large
portion of code, and it is something i've no time to do right now,
sorry.

2014-02-10 13:51 GMT+01:00, Nikoli notifications@github.com:

Now mpv has libmpv shared library, what is your opinion about API?
mpv-player/mpv#97 (comment)

---

Reply to this email directly or view it on GitHub:
#2 (comment)

[Nikoli]

Actually mpv has a broken slave mode; so i'll consider to support it when it will work.

Broken? What do you mean? It works fine, there is even GTK gui using it:
https://github.com/sebadoom/mpvguihs

[kokoko3k]

Check by yourself in the man page:
https://github.com/mpv-player/mpv/blob/master/DOCS/man/en/options.rst
search for --slave-broken

2014-02-18 13:21 GMT+01:00, Nikoli notifications@github.com:

Actually mpv has a broken slave mode; so i'll consider to support it
when it will work.

Broken? What do you mean? It works fine, there is even GTK gui using it:
https://github.com/sebadoom/mpvguihs

---

Reply to this email directly or view it on GitHub:
#2 (comment)

[Nikoli]

They just tell that current mpv slave mode is not as perfect as they desire, it does not mean, that slave mode in mplayer/mplayer2 is somehow better.

[kokoko3k]

Please, read better, they say it is currently not compatible with the
old slave mode and that they need a new better one.
They also explicitely state: "This is for testing only"; i fear in
the future they will just drop it and more work will need to be made
to use the new one.

If you really care as it seems, you should really ask mpv devs about
their intentions about that question, i'm certainly not going to
implement something they suggest me not to do.

2014-02-18 15:06 GMT+01:00, Nikoli notifications@github.com:

They just tell that current mpv slave mode is not as perfect as they desire,
it does not mean, that slave mode in mplayer/mplayer2 is somehow better.

---

Reply to this email directly or view it on GitHub:
#2 (comment)

[kokoko3k]

Just to let you know that i'm working on the port since some weeks.
I'm not going to use the slave mode but something near (--input switch)
When ready, i think i'll open a new github page for it (no fork or whatever).
Something like xt7-mpv...

[kokoko3k]

https://github.com/kokoko3k/xt7-player-mpv