Mir

Code review comment for lp:~vanvugt/mir/timerless-multimonitor-frame-sync

  1. timerless-multimonitor-frame-sync
  2. Merge into trunk
Revision history for this message
Chris Halse Rogers (raof) wrote :

Mostly looks good to me, only nitpick:

160 + local_frame_count <= 0) // Wrap around, unlikely, but handle it...

I'm not a huge fan of testing unsigned values for <= 0. Maybe make it == 0?

« Back to merge proposal