diff options
| author |  wm4 <wm4@nowhere> | 2016-08-30 23:52:16 +0200 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2016-08-30 23:52:16 +0200 |
| commit | 4d755143216e4d8dec22b4918c1dd290a5efe814 (patch) | |
| tree | 177ef8dba13d14853fb79175d9dc8370d4823fbd /video/out/vo_rpi.c | |
| parent | af1379c43d6a5274b75bce0adeef9e3a9ce87bdf (diff) | |
vo: change messy handling of fullscreen and other flags
Before this commit, all VOs had to toggle the option flag themselves,
now command.c does it.
I can't really comprehend why it required every VO to do this manually.
Maybe it was for rejecting the property/option change if the VO didn't
support a specific capability. But then it could have checked the VOCTRL
result. In any case, I don't care, and successfully changing the
property without doing anything (With some VOs) is fine too. Many things
work this way now, and it's simpler overall.
This change will be useful for cleaning up VO option handling.
Diffstat (limited to 'video/out/vo_rpi.c')
| -rw-r--r-- | video/out/vo_rpi.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/video/out/vo_rpi.c b/video/out/vo_rpi.c index 947e63037e..cbbd781f0b 100644 --- a/video/out/vo_rpi.c +++ b/video/out/vo_rpi.c @@ -612,7 +612,6 @@ static int control(struct vo *vo, uint32_t request, void *data) switch (request) { case VOCTRL_FULLSCREEN: - vo->opts->fullscreen = !vo->opts->fullscreen; if (p->renderer_enabled) set_geometry(vo); vo->want_redraw = true; |