rpc_N00

[Gregory M. Turner]

Odds and ends...

License: bugroff

Changelog:

* dlls/rpcrt4: rpc_server.c
  Greg Turner <gmturner007 at ameritech.net>
- Trace the argument to RPCRT4_server_thread
- ^null implies two indirections, we have one
- Ove's intent doesn't matter, the current implementation is right

--
diff -ur --minimal --exclude-from=/home/greg/bin/winetreediff_excl ../wine.vanilla/dlls/rpcrt4/rpc_server.c ./dlls/rpcrt4/rpc_server.c
--- ../wine.vanilla/dlls/rpcrt4/rpc_server.c	2003-10-14 21:15:56.000000000 -0500
+++ ./dlls/rpcrt4/rpc_server.c	2003-11-20 13:11:53.000000000 -0600
@@ -375,6 +375,8 @@
   RpcConnection* conn;
   RpcConnection* cconn;
 
+  TRACE("(the_arg == ^%p)\n", the_arg);
+
   for (;;) {
     EnterCriticalSection(&server_cs);
     /* open and count connections */
@@ -516,7 +518,7 @@
   if (BindingVector)
     TRACE("(*BindingVector == ^%p)\n", *BindingVector);
   else
-    ERR("(BindingVector == ^null!!?)\n");
+    ERR("(BindingVector == NULL!!?)\n");
 
   EnterCriticalSection(&server_cs);
   /* count connections */
@@ -625,7 +627,6 @@
 
   ps = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(RpcServerProtseq));
   ps->MaxCalls = MaxCalls;
-  /* FIXME: Did Ove have these next two as RPCRT4_strdupW for a reason? */
   ps->Protseq = RPCRT4_strdupWtoA(Protseq);
   ps->Endpoint = RPCRT4_strdupWtoA(Endpoint);
 
-- 
gmt

"It is to be the assent and ratification of the several States,
derived from the supreme authority in each State, the authority
of the people themselves.  The act, therefore, establishing the
Constitution, will not be a NATIONAL, but a FEDERAL act." --James
Madison, Federalist No. 39