summaryrefslogtreecommitdiffstats
path: root/cfg-mplayer.h
diff options
context:
space:
mode:
authorUoti Urpala <uau@glyph.nonexistent.invalid>2011-01-31 21:53:05 +0200
committerUoti Urpala <uau@glyph.nonexistent.invalid>2011-01-31 21:53:05 +0200
commitfad3585ef4861b227c0a7f577556e3918a46137e (patch)
tree3cd42ee65b2de8de63adf47a84b90188e4c9bdd3 /cfg-mplayer.h
parent4ea60a3d0ec4f01f3c319d6f155653f4f890632f (diff)
downloadmpv-fad3585ef4861b227c0a7f577556e3918a46137e.tar.bz2
mpv-fad3585ef4861b227c0a7f577556e3918a46137e.tar.xz
options: remove CONF_OLD option flag
The resulting semantics of this flag are weird enough that they're unlikely to be what is wanted in any situation. Remove the flag and convert the two options using it, -screenw and -screenh, to use CONF_NOSAVE instead. I'm not sure why those specific options had the flag and if any flag is really needed, but I don't want to check in detail now and using CONF_NOSAVE should keep about the same behavior in practice. A bit more detail about the weird behavior this flag had: When not using file groups, the flag had the same behavior as CONF_NOSAVE, namely that when switching files the option would not be reset to the global value (only possible file-specific settings were applied). When using file groups, group-specific options would apply to the _first two_ files in the group, but for the rest after the first two, settings would not be reset when changing files (wtf?). This was a result of the following sequence: 1) push higher-level settings, enter group 2) apply group-specific settings 3) push settings before applying ones specific to file 1 in group 4) apply file 1 settings, play file 1 5) pop settings to return to group settings 6) push settings before applying ones specific to file 2 7) apply file 2 settings 8) pop settings Here the option was set at 2). 3) saved it because it had been set after last push, so 5) restored the setting and it was used for file 2 too. However 6) no longer saved it because there had been pushes after the original setting in 2), thus 8) no longer restored the setting and the option was no longer forced to any particular value when playing further files after that.
Diffstat (limited to 'cfg-mplayer.h')
-rw-r--r--cfg-mplayer.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/cfg-mplayer.h b/cfg-mplayer.h
index 391f833e05..e2a52ee637 100644
--- a/cfg-mplayer.h
+++ b/cfg-mplayer.h
@@ -136,8 +136,8 @@ const m_option_t mplayer_opts[]={
OPT_INTRANGE("x", screen_size_x, 0, 0, 4096),
OPT_INTRANGE("y", screen_size_y, 0, 0, 4096),
// set screen dimensions (when not detectable or virtual!=visible)
- OPT_INTRANGE("screenw", vo_screenwidth, CONF_OLD, 0, 4096),
- OPT_INTRANGE("screenh", vo_screenheight, CONF_OLD, 0, 4096),
+ OPT_INTRANGE("screenw", vo_screenwidth, CONF_NOSAVE, 0, 4096),
+ OPT_INTRANGE("screenh", vo_screenheight, CONF_NOSAVE, 0, 4096),
// Geometry string
{"geometry", &vo_geometry, CONF_TYPE_STRING, 0, 0, 0, NULL},
OPT_MAKE_FLAGS("force-window-position", force_window_position, 0),