[MPlayer-dev-eng] [PATCH] EDL changes: loadfile and adjust pts functionality

Reynaldo H. Verdejo Pinochet reynaldo at opendot.cl
Thu Sep 2 02:24:20 CEST 2010


Hi again  Vlad

On Wed, Sep 01, 2010 at 07:45:22PM -0400, Vlad Seryakov wrote:
> [...] 
> 
> i only add to help_mp-en.h, right?

Yes, thanks.

> [...]
>    int brk_cmd = 0;
>    while( !brk_cmd && (cmd = mp_input_get_cmd(0,0,0)) != NULL) {
>        brk_cmd = run_command(mpctx, cmd);
> +      if (cmd->id == MP_CMD_EDL_LOADFILE) {
> +          if (edl_filename) free(edl_filename);
> +          edl_filename = strdup(cmd->args[0].v.s);
> +          if (!edl_filename) {
> +              mp_msg(MSGT_CPLAYER, MSGL_ERR, MSGTR_EdlOutOfMemFile, cmd->args[0].v.s);
> +          }
> +          edl_loadfile();

Please drop the pointless braces and consider making it an
if/else clause. I know edl_loadfile() won't do a thing
with a NULL edl_filename but there is no need to call
it in that case and you already have the if so...

Best regards

--
Reynaldo


More information about the MPlayer-dev-eng mailing list