Merge lp://staging/~jesse-barker/glcompbench/dev into lp://staging/glcompbench

Proposed by Jesse Barker
Status: Merged
Merge reported by: Jesse Barker
Merged at revision: not available
Proposed branch: lp://staging/~jesse-barker/glcompbench/dev
Merge into: lp://staging/glcompbench
Diff against target: 98 lines (+30/-29)
1 file modified
src/composite-canvas.cc (+30/-29)
To merge this branch: bzr merge lp://staging/~jesse-barker/glcompbench/dev
Reviewer Review Type Date Requested Status
Alexandros Frantzis Pending
Review via email: mp+51087@code.staging.launchpad.net

Description of the change

These changes add the logic to cycle through the list of tests using a default cadence of 5 iterations, where each iteration is the 3 second interval used for measuring frames per second. There are 2 new options for controlling the number of iterations per test (--num-iters) as well as whether to run continuously (--run-forever) or stop at the end of the test list. The default is to stop as this seemed more immediately useful for integration with test frameworks. Also there is a small bug fix in the handling of the initial window size option (getopt_long was not setting up the option argument which caused a segfault in strtol if an argument was passed).

To post a comment you must log in.
5. By Jesse Barker

Add support for iterating across the list of tests.

6. By Alexandros Frantzis

Whitespace fixes.

7. By Alexandros Frantzis

Use Log::error() instead of writing directly to stderr.

8. By Jesse Barker

Fix CompositeCanvas::handle_xevent() so that it does not violate strict pointer
aliasing rules.

9. By Jesse Barker

Finalize last merges.

Revision history for this message
Jesse Barker (jesse-barker) wrote :

Original changes split up for sanity.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches