Re: switch_audio on *.ts files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]<

 



On Fri, Jun 19, 2009 at 01:05:30PM +0200, rvm wrote:
> (Don't know why gmail didn't send the whole text, 2nd try):
> 
> Both -aid and -vid require the ID printed by mplayer, but then the
> slave commands don't accept those IDs, they expect other values that
> mplayer doesn't print anywhere... :(
> 
> The video IDs from a ts:
> ID_VIDEO_ID=1001
> ID_VIDEO_ID=1501
> ID_VIDEO_ID=301
> ID_VIDEO_ID=501
> 
> mplayer video.ts -vid 301 works as expected but
> 
> set_property switch_video 301
> The switch_video option must be <= 255: 301
> Failed to set property 'switch_video' to '301'
> 
> Is this a bug or am I doing something wrong?

Probably, there are two kinds on stream IDs in MPlayer and -vid
originally selected one that made no sense.
They are still confused in some places and this is probably one.
See if this patch fixes it for you:
Index: command.c
===================================================================
--- command.c   (revision 29371)
+++ command.c   (working copy)
@@ -2029,7 +2029,7 @@
     { "channels", mp_property_channels, CONF_TYPE_INT,
      0, 0, 0, NULL },
     { "switch_audio", mp_property_audio, CONF_TYPE_INT,
-     CONF_RANGE, -2, MAX_A_STREAMS - 1, NULL },
+     CONF_RANGE, -2, 65535, NULL },
     { "balance", mp_property_balance, CONF_TYPE_FLOAT,
      M_OPT_RANGE, -1, 1, NULL },
 
@@ -2075,7 +2075,7 @@
     { "aspect", mp_property_aspect, CONF_TYPE_FLOAT,
      0, 0, 0, NULL },
     { "switch_video", mp_property_video, CONF_TYPE_INT,
-     CONF_RANGE, -2, MAX_V_STREAMS - 1, NULL },
+     CONF_RANGE, -2, 65535, NULL },
     { "switch_program", mp_property_program, CONF_TYPE_INT,
      CONF_RANGE, -1, 65535, NULL },
 
_______________________________________________
MPlayer-users mailing list
MPlayer-users@xxxxxxxxxxxx
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-users

[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux