From 3be727073c11d204664a3090967dea7c35c6d160 Mon Sep 17 00:00:00 2001
From: Marcus10110 <markgarrison@saleae.com>
Date: Thu, 28 Jan 2021 10:02:59 -0800
Subject: [PATCH] added code to detect the last disable event when there are no
 more transitions on the clock channel.

---
 src/SpiAnalyzer.cpp | 33 +++++++++++++++++++++++++++------
 1 file changed, 27 insertions(+), 6 deletions(-)

diff --git a/src/SpiAnalyzer.cpp b/src/SpiAnalyzer.cpp
index 983a86a..7ed8a8b 100644
--- a/src/SpiAnalyzer.cpp
+++ b/src/SpiAnalyzer.cpp
@@ -177,12 +177,7 @@ bool SpiAnalyzer::WouldAdvancingTheClockToggleEnable( bool add_disable_frame, U6
     if( mEnable == NULL )
         return false;
 
-    U64 next_edge = mClock->GetSampleOfNextEdge();
-    bool enable_will_toggle = mEnable->WouldAdvancingToAbsPositionCauseTransition( next_edge );
-
-    if( enable_will_toggle )
-    {
-        U64 enable_edge = mEnable->GetSampleOfNextEdge();
+    auto log_disable_event = [&]( U64 enable_edge ) {
         if( add_disable_frame )
         {
             FrameV2 frame_v2_end_of_transaction;
@@ -192,6 +187,32 @@ bool SpiAnalyzer::WouldAdvancingTheClockToggleEnable( bool add_disable_frame, U6
         {
             *disable_frame = enable_edge;
         }
+    };
+
+    // if the enable is currently active, and there are no more clock transitions in the capture, attempt to capture the final disable event
+    if( !mClock->DoMoreTransitionsExistInCurrentData() && mEnable->GetBitState() == mSettings->mEnableActiveState )
+    {
+        if( mEnable->DoMoreTransitionsExistInCurrentData() )
+        {
+            U64 next_enable_edge = mEnable->GetSampleOfNextEdge();
+            // double check that the clock line actually processed all samples up to the next enable edge.
+            // double check is required becase data is getting processed while we're running, it's possible more has already become
+            // available.
+            if( !mClock->WouldAdvancingToAbsPositionCauseTransition( next_enable_edge ) )
+            {
+                log_disable_event( next_enable_edge );
+                return true;
+            }
+        }
+    }
+
+    U64 next_edge = mClock->GetSampleOfNextEdge();
+    bool enable_will_toggle = mEnable->WouldAdvancingToAbsPositionCauseTransition( next_edge );
+
+    if( enable_will_toggle )
+    {
+        U64 enable_edge = mEnable->GetSampleOfNextEdge();
+        log_disable_event( enable_edge );
     }
 
     if( enable_will_toggle == false )