From 5c4623e9f77e7cbaea11eb5393c3cadafdcd0c3e Mon Sep 17 00:00:00 2001 From: Thomas Harte Date: Tue, 28 Jan 2020 20:27:24 -0500 Subject: [PATCH] Adds a sequence-point test for 72Hz mode. Which immediately appears to trigger the hsync issue I'm also seeing in manual testing. --- .../Mac/Clock SignalTests/AtariSTVideoTests.mm | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/OSBindings/Mac/Clock SignalTests/AtariSTVideoTests.mm b/OSBindings/Mac/Clock SignalTests/AtariSTVideoTests.mm index 7bba7b73e..8749a3b78 100644 --- a/OSBindings/Mac/Clock SignalTests/AtariSTVideoTests.mm +++ b/OSBindings/Mac/Clock SignalTests/AtariSTVideoTests.mm @@ -95,11 +95,22 @@ struct VideoTester { // MARK: - Sequence Point Prediction Tests /// Tests that no events occur outside of the sequence points the video predicts. -- (void)testSequencePoints { +- (void)testSequencePoints50 { // Set 4bpp, 50Hz. _video->write(0x05, 0x0200); _video->write(0x30, 0x0000); + [self runSequencePointsTest]; +} + +- (void)testSequencePoints72 { + // Set 1bpp, 72Hz. + _video->write(0x30, 0x0200); + + [self runSequencePointsTest]; +} + +- (void)runSequencePointsTest { // Run for [more than] a whole frame making sure that no observeable outputs // change at any time other than a sequence point. HalfCycles next_event;