Opened 4 years ago

Closed 4 years ago

#736 closed defect (fixed)

ffplay fullscreen toggle issue - fix included

Reported by: glangshaw Owned by:
Priority: normal Component: ffplay
Version: git-master Keywords: fullscreen
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Incorrect condition match and return in video_open()
when toggling to fullscreen a video window which
already matches the display resolution.

e.g. a 1920x1080 HD stream in a -geometry 1920x1080+0+22 window

on a 1920x1080 display.

diff attached

Attachments (1)

ffplay-fix-fullscreen_toggle.diff (1.6 KB) - added by glangshaw 4 years ago.

Download all attachments as: .zip

Change History (5)

Changed 4 years ago by glangshaw

comment:1 Changed 4 years ago by cehoyos

  • Keywords ffplay removed

Please consider posting patches to ffmpeg-devel where they usually receive more attraction.

comment:2 Changed 4 years ago by glangshaw

Thanks for the suggestion, but I don't want to get involved on the dev list - I'm just an end user who knows a little C making a one-off submission. The contacts page of the website clearly says that list is not for bug reports so this seemed the appropriate place for it.

comment:3 Changed 4 years ago by cus

Should be fixed in git master now.

comment:4 Changed 4 years ago by cehoyos

  • Resolution set to fixed
  • Status changed from new to closed
  • Version changed from 0.9 to git-master
Note: See TracTickets for help on using tickets.