[ros-diffs] [gschneider] 42886: Fix multiple threads listings 1/x: Reset receive variables when reaching the block end, continue with the loop as soon as ListEvents were created

Author: gschneider
Date: Sun Aug 23 23:37:35 2009
New Revision: 42886

URL: http://svn.reactos.org/svn/reactos?rev=42886&view=rev
Log:
Fix multiple threads listings 1/x:
Reset receive variables when reaching the block end, continue with the loop as soon as
ListEvents were created

Modified:
    trunk/tools/reactosdbg/DebugProtocol/KDBG.cs
    trunk/tools/reactosdbg/RosDBG/Properties/AssemblyInfo.cs

Modified: trunk/tools/reactosdbg/DebugProtocol/KDBG.cs
URL:
http://svn.reactos.org/svn/reactos/trunk/tools/reactosdbg/DebugProtocol/KDB…
==============================================================================
--- trunk/tools/reactosdbg/DebugProtocol/KDBG.cs [iso-8859-1] (original)
+++ trunk/tools/reactosdbg/DebugProtocol/KDBG.cs [iso-8859-1] Sun Aug 23 23:37:35 2009
@@ -257,12 +257,16 @@
                                 if (ProcessListEvent != null)
                                     ProcessListEvent(this, new
ProcessListEventArgs(ulong.Parse(pidEntryMatch.Groups["pid"].ToString(),
NumberStyles.HexNumber), pidEntryMatch.Groups["cur"].Length > 0, 
                                        
pidEntryMatch.Groups["state"].ToString(),
pidEntryMatch.Groups["name"].ToString()));
+                                continue;
                             }
                             else
                             {
-                                /* TODO: this is called by far too often, results in
several "thread list xx" commands */
                                 if ((mReceivingProcs || cleanedLine.Contains("No
processes")) && ProcessListEvent != null)
+                                {
                                     ProcessListEvent(this, new
ProcessListEventArgs(true));
+                                    mReceivingProcs = false;
+                                    continue;
+                                }
                             }
                         }
 
@@ -282,11 +286,16 @@
                             {
                                 if (ThreadListEvent != null)
                                     ThreadListEvent(this, new
ThreadListEventArgs(ulong.Parse(tidEntryMatch.Groups["tid"].ToString(),
NumberStyles.HexNumber), tidEntryMatch.Groups["cur"].Length > 0,
ulong.Parse(tidEntryMatch.Groups["eip"].ToString(), NumberStyles.HexNumber)));
+                                continue;
                             }
                             else
                             {
                                 if (mReceivingThreads && ThreadListEvent !=
null)
+                                {
                                     ThreadListEvent(this, new
ThreadListEventArgs(true));
+                                    mReceivingThreads = false;
+                                    continue;
+                                }
                             }
                         }
                     }

Modified: trunk/tools/reactosdbg/RosDBG/Properties/AssemblyInfo.cs
URL:
http://svn.reactos.org/svn/reactos/trunk/tools/reactosdbg/RosDBG/Properties…
==============================================================================
--- trunk/tools/reactosdbg/RosDBG/Properties/AssemblyInfo.cs [iso-8859-1] (original)
+++ trunk/tools/reactosdbg/RosDBG/Properties/AssemblyInfo.cs [iso-8859-1] Sun Aug 23
23:37:35 2009
@@ -39,5 +39,5 @@
 // will be increased as well. MSI installers must not be generated with the same Build
Number
 // otherwise they won't upgrade the old installation!
 
-[assembly: AssemblyVersion("1.0.2.72")]
-[assembly: AssemblyFileVersion("1.0.2.72")]
+[assembly: AssemblyVersion("1.0.2.73")]
+[assembly: AssemblyFileVersion("1.0.2.73")]