diff options
| author | wm4 <wm4@nowhere> | 2012-09-18 11:48:50 +0200 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2012-10-12 10:10:30 +0200 |
| commit | 3d67041089c7945dcea0162c0038e495d588b379 (patch) | |
| tree | f808740db919bae523d45ae19196885ebf651c73 /cfg-mplayer.h | |
| parent | 10437c35df5d3944625fa6cee05e565b0791fe15 (diff) | |
| download | mpv-3d67041089c7945dcea0162c0038e495d588b379.tar.bz2 mpv-3d67041089c7945dcea0162c0038e495d588b379.tar.xz | |
options: remove CONF_TYPE_POSITION
This was the option parser for the off_t C type. These days, off_t is
always int64_t, so replace all its uses by int64_t and CONF_TYPE_INT64.
Fix the --sstep option. It used CONF_TYPE_INT with an off_t variable,
which will result in invalid memory accesses. Make it use type double
instead, which seems to make more sense for this option.
Diffstat (limited to 'cfg-mplayer.h')
| -rw-r--r-- | cfg-mplayer.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/cfg-mplayer.h b/cfg-mplayer.h index 080b04bd63..8dfebd2a5d 100644 --- a/cfg-mplayer.h +++ b/cfg-mplayer.h @@ -381,7 +381,7 @@ const m_option_t common_opts[] = { {"frames", &play_n_frames_mf, CONF_TYPE_INT, CONF_MIN, 0, 0, NULL}, // seek to byte/seconds position - {"sb", &seek_to_byte, CONF_TYPE_POSITION, CONF_MIN, 0, 0, NULL}, + {"sb", &seek_to_byte, CONF_TYPE_INT64, CONF_MIN, 0, 0, NULL}, OPT_TIME("ss", seek_to_sec, 0), // start paused @@ -678,7 +678,7 @@ const m_option_t mplayer_opts[]={ OPT_STRING("vobsub", vobsub_name, 0), {"vobsubid", &vobsub_id, CONF_TYPE_INT, CONF_RANGE, 0, 31, NULL}, - {"sstep", &step_sec, CONF_TYPE_INT, CONF_MIN, 0, 0, NULL}, + {"sstep", &step_sec, CONF_TYPE_DOUBLE, CONF_MIN, 0, 0, NULL}, OPT_CHOICE("framedrop", frame_dropping, 0, ({"no", 0}, |