- changed the query locking mechanism for MCPConnection to be more thread safe. From now on, always use [self lockConnection] rather than [queryLock lock], independent of what thread you are running on

- A warning is written to the console when the connection is unlocked multiple times (to identify potential race conditions) 
- modified MCPStreamingResult to ensure it only closes the connection once
- added a check to prevent arrow key navigation past the last row

1 files changed, 7 insertions, 0 deletions

diff --git a/Frameworks/MCPKit/MCPFoundationKit/MCPConstants.h b/Frameworks/MCPKit/MCPFoundationKit/MCPConstants.h
index acdeee16..fba7d651 100644
--- a/Frameworks/MCPKit/MCPFoundationKit/MCPConstants.h
+++ b/Frameworks/MCPKit/MCPFoundationKit/MCPConstants.h
@@ -52,6 +52,13 @@ enum
 };
 typedef NSUInteger MCPQueryStreamingType;
 
+// Connection state
+// This is used internally by MCPConnection to prevent simultaneous execution of different queries
+enum {
+    MCPConnectionIdle   = 0,
+    MCPConnectionBusy   = 1
+};
+
 // Charcater set mapping constants
 typedef struct _OUR_CHARSET
 {