RepoMirrors
/
mpv
mirror of https://github.com/mpv-player/mpv
1
0
Fork 0
Code Issues Releases Wiki Activity

cocoa: improve refresh rate fallback code 

Apparently CoreGraphics reports the actual refresh rate. DisplayLink can also
query the nominal refresh rate of the display so we use that as fallback
instead of the fugly 60fps hardcode added in aeb1fca0d.

Props to people on https://github.com/glfw/glfw/issues/137
This commit is contained in:
Stefano Pigozzi 2015-02-03 22:59:54 +01:00
parent aeb1fca0d4
commit eacf22e42a
1 changed files with 14 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
video/out/cocoa_common.m
View File

@ -267,14 +267,17 @@ static void vo_cocoa_update_screen_fps(struct vo *vo)
CGDisplayModeRef mode = CGDisplayCopyDisplayMode(did);
s->screen_fps = CGDisplayModeGetRefreshRate(mode);
CGDisplayModeRelease(mode);
if (s->screen_fps == 0.0) {
// Most internal Apple monitors and laptop monitors report 0 instead
// of 60fps. Assume them to be 60hz. This is technically incorrect but
// works most of the time, and seems to be used in most open source
// software for lack of a better Apple API.
MP_VERBOSE(vo, "CoreGraphics reports a 0fps display. Assuming internal "
"Apple monitor @ 60fps instead.\n");
s->screen_fps = 60.0;
// Fallback to using Nominal refresh rate from DisplayLink,
// CVDisplayLinkGet *Actual* OutputVideoRefreshPeriod seems to
// return 0 as well if CG returns 0
CVDisplayLinkRef link;
CVDisplayLinkCreateWithCGDisplay(did, &link);
const CVTime t = CVDisplayLinkGetNominalOutputVideoRefreshPeriod(link);
if (!(t.flags & kCVTimeIsIndefinite))
s->screen_fps = (t.timeScale / (double) t.timeValue);
CVDisplayLinkRelease(link);
}
}
@ -673,8 +676,10 @@ int vo_cocoa_control(struct vo *vo, int *events, int request, void *arg)
vo_cocoa_control_get_icc_profile(vo, arg);
return VO_TRUE;
case VOCTRL_GET_DISPLAY_FPS:
*(double *)arg = vo->cocoa->screen_fps;
return VO_TRUE;
if (vo->cocoa->screen_fps > 0.0) {
*(double *)arg = vo->cocoa->screen_fps;
return VO_TRUE;
}
}
return VO_NOTIMPL;
}