Sounds like the right fix. If we were preferring the latest buffer returned then we were accidentally double buffering instead of triple. Which would quantize to half frame rate more easily.
What concerns me though is that you've broken one aspect of the old test cases. In that you no longer expect minimal buffer allocation [1]:
Sounds like the right fix. If we were preferring the latest buffer returned then we were accidentally double buffering instead of triple. Which would quantize to half frame rate more easily.
What concerns me though is that you've broken one aspect of the old test cases. In that you no longer expect minimal buffer allocation [1]:
88 - EXPECT_ THAT(unique_ ids_in( log), Eq(2)); 1>(GetParam( )) == TestType: :SubmitSemantic s) 1>(GetParam( )) == TestType: :ExchangeSemant ics) THAT(unique_ ids_in( log), Eq(2));
89 + if (std::get<
90 + EXPECT_THAT(log, NeverBlocks());
91 + if (std::get<
92 + EXPECT_
[1] https:/ /docs.google. com/spreadsheet s/d/1qf3IssN_ sujygMPK2sTX2AU hGiDUBF3TMYYcSR SaSy0/pubhtml
That's a slight regression compared to BufferQueue. Although BufferQueue did not do it well to start with.