[prev in list] [next in list] [prev in thread] [next in thread] 

List:       ros-diffs
Subject:    [ros-diffs] [ea] 17577: Autoregister Debug internal subsystem too.
From:       <ea () svn ! reactos ! com>
Date:       2005-08-28 7:34:45
Message-ID: 000001c5aba2$f6593420$6b01a8c0 () PENELOPE
[Download RAW message or body]

Autoregister Debug internal subsystem too. Now 'sm info' displays
meaningful information.

NOTE: It seems the new client management introduced a stupid bug that
shows up on system shutdown (smss crashes calling an LPC related API). I
can't locate it now (of course, that happens for every 'stupid' bug, but
will tray again later).
Modified: trunk/reactos/subsys/smss/debug.c
  _____  

Modified: trunk/reactos/subsys/smss/debug.c
--- trunk/reactos/subsys/smss/debug.c	2005-08-28 05:34:22 UTC (rev
17576)
+++ trunk/reactos/subsys/smss/debug.c	2005-08-28 07:34:35 UTC (rev
17577)
@@ -140,7 +140,6 @@

 	DPRINT("SM: %s called\n", __FUNCTION__);
 
 	/* Self register */
-#if 0
 	Status = SmRegisterInternalSubsystem (L"Debug",
 						(USHORT)-1,
 						& hSmDbgApiPort);
@@ -150,7 +149,6 @@
 			__FUNCTION__, Status);
 		return Status;
 	}
-#endif	
 	/* Create the \DbgSsApiPort object (LPC) */
 	Status = SmpCreatePT(& DbgSsApiPort,
 			     SM_DBGSS_PORT_NAME,

[Attachment #3 (text/html)]

<html>
<head>
<style>
<!--
body { background-color:#ffffff }
.file { border:1px solid #eeeeee; margin-top:1em; margin-bottom:1em }
.pathname { font-family:monospace; float:right }
.fileheader { margin-bottom:.5em }
.diff { margin:0 }
.tasklist { padding:4px; border:1px dashed #000000; margin-top:1em }
.tasklist ul { margin-top:0; margin-bottom:0 }
tr.alt { background-color:#eeeeee }
#added { background-color:#ddffdd }
#addedchars { background-color:#99ff99; font-weight:bolder }
tr.alt #added { background-color:#ccf7cc }
#removed { background-color:#ffdddd }
#removedchars { background-color:#ff9999; font-weight:bolder }
tr.alt #removed { background-color:#f7cccc }
#info { color:#888888 }
#context { background-color:#eeeeee }
td {padding-left:.3em; padding-right:.3em }
tr.head { border-bottom-width:1px; border-bottom-style:solid }
tr.head td { padding:0; padding-top:.2em }
.task { background-color:#ffff00 }
.comment { padding:4px; border:1px dashed #000000; background-color:#ffffdd }
.error { color:red }
hr { border-width:0px; height:2px; background:black }
-->
</style>
</head>
<body>
<pre class="comment">Autoregister Debug internal subsystem too. Now 'sm info' \
displays meaningful information.

NOTE: It seems the new client management introduced a stupid bug that shows up on \
system shutdown (smss crashes calling an LPC related API). I can't locate it now (of \
course, that happens for every 'stupid' bug, but will tray again later).</pre><pre \
class="diff" id="context">Modified: trunk/reactos/subsys/smss/debug.c </pre><hr \
/><div class="file"> <div class="fileheader"><big><b>Modified: \
trunk/reactos/subsys/smss/debug.c</b></big></div> <pre class="diff"><small \
                id="info">--- trunk/reactos/subsys/smss/debug.c	2005-08-28 05:34:22 \
                UTC (rev 17576)
+++ trunk/reactos/subsys/smss/debug.c	2005-08-28 07:34:35 UTC (rev 17577)
@@ -140,7 +140,6 @@
</small></pre><pre class="diff" id="context"> 	DPRINT(&quot;SM: %s called\n&quot;, \
__FUNCTION__);  
 	/* Self register */
</pre><pre class="diff" id="removed">-#if 0
</pre><pre class="diff" id="context"> 	Status = SmRegisterInternalSubsystem \
(L&quot;Debug&quot;,  (USHORT)-1,
 						&amp; hSmDbgApiPort);
@@ -150,7 +149,6 @@
</pre><pre class="diff" id="context"> 			__FUNCTION__, Status);
 		return Status;
 	}
</pre><pre class="diff" id="removed">-#endif	
</pre><pre class="diff" id="context"> 	/* Create the \DbgSsApiPort object (LPC) */
 	Status = SmpCreatePT(&amp; DbgSsApiPort,
 			 &nbsp; &nbsp; SM_DBGSS_PORT_NAME,
</pre>
</div>

</body>
</html>



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic