Skip to content

Commit

Permalink
fix(vsync): do not use android vsync because of compatibility issue
Browse files Browse the repository at this point in the history
  • Loading branch information
I-m-SuperMan committed Feb 10, 2020
1 parent 0d3db42 commit 4cacc83
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions framework/render/video/vsync/VSyncFactory.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,14 @@

std::unique_ptr<IVSync> VSyncFactory::create(IVSync::Listener &listener, float HZ)
{
#if defined(ANDROID)
std::string version = get_android_property("ro.build.version.sdk");

if (atoi(version.c_str()) >= 16)
return std::unique_ptr<IVSync>(new AndroidVSync(listener));

#endif
//TODO use android vsync: compatibility issue
//#if defined(ANDROID)
// std::string version = get_android_property("ro.build.version.sdk");
//
// if (atoi(version.c_str()) >= 16)
// return std::unique_ptr<IVSync>(new AndroidVSync(listener));
//
//#endif
return std::unique_ptr<IVSync>(new timedVSync(listener, HZ));
// return std::unique_ptr<IVSync>(new CADisplayLinkVSync(listener));
}

0 comments on commit 4cacc83

Please sign in to comment.