[MPlayer-dev-eng] [PATCH] Use unrar for open vobsubs if available

Diego Biurrun diego at biurrun.de
Fri Nov 30 19:41:05 CET 2007


On Sat, Dec 01, 2007 at 01:49:31AM +0800, Ulion wrote:
> 2007/11/27, Diego Biurrun <diego at biurrun.de>:
> >
> > What I am trying to say is that external unrar should replace the
> > internal unrarlib, not coexist with it.
> 
> Oh, unrarlib can be removed, if you want, but may after add unrar_exec
> support, make it be an 'add' and a 'remove' should be better than a
> 'replace' if finally decide to remove it.

Our internal copy of unrarlib should go, I think there is consensus
about this.

> Here's slight update patch, make sure the cmdline buffer is large
> enough. Is this commitable? If no objects, I will apply this in 5
> days.

Build system part is OK, some nits below, I did not look at the rest in
detail.  Removing trailing whitespace would be a plus.

> --- unrar_exec.c	(revision 0)
> +++ unrar_exec.c	(revision 0)
> @@ -0,0 +1,293 @@
> +
> +        // ./executable_name is not recognised by cmd.exe without quote.

quotes

> +     * If not use popen, the only other way is to use CreateProcess, that also
> +     * need a cmdline as parameter, so we use popen on win32 and escape cmdline.
> +     * On win32, popen equals 'cmd.exe /C cmdline'.
> +     * Check the output of 'cmd.exe /?' for how quotes in cmdline is processed.

     * If not using popen, the only other way is to use CreateProcess.
     * This needs a command line as parameter, so we use popen on Win32
     * and escape the command line.
     * On win32, popen equals 'cmd.exe /C cmdline'.
     * Check the output of 'cmd.exe /?' to see how quotes on the command
     * line are processed.

> +    mp_msg(MSGT_GLOBAL, MSGL_V, "UnRAR: call unrar with cmdline: %s\n",cmdline);

command line, same below

Diego



More information about the MPlayer-dev-eng mailing list