commit | 09940bbc0e7eb687c1b08c1ba0252852752f68ab | [log] [tgz] |
---|---|---|
author | Dichen Zhang <dichenzhang@google.com> | Wed Apr 24 14:58:51 2019 -0700 |
committer | Dichen Zhang <dichenzhang@google.com> | Thu Apr 25 16:23:41 2019 +0000 |
tree | 75269457bc7c5acad8c83b8e48c9a67f8c804b65 | |
parent | a66ea4b93b988942828e60a320a469a5a0f3d674 [diff] |
Add comments This follows Id9b4bc0d2e53c1b4c239027c8b087020019a0147 Bug: 129491498 Test: TBD Change-Id: I8070836b24d46b1152c0af3584ffd444008632f6
diff --git a/media/libstagefright/VideoFrameSchedulerBase.cpp b/media/libstagefright/VideoFrameSchedulerBase.cpp index 912dcf3..0d1517b 100644 --- a/media/libstagefright/VideoFrameSchedulerBase.cpp +++ b/media/libstagefright/VideoFrameSchedulerBase.cpp
@@ -115,6 +115,9 @@ #endif +// If overflow happens, the value is already incorrect, and no mater what value we get is OK. +// And this part of calculation is not important, so it's OK to simply disable overflow check +// instead of using double which makes code more complicated. __attribute__((no_sanitize("integer"))) bool VideoFrameSchedulerBase::PLL::fit( nsecs_t phase, nsecs_t period, size_t numSamplesToUse,