ubik: remove unused code
authorMarc Dionne <marc.c.dionne@gmail.com>
Sat, 5 Mar 2011 16:19:10 +0000 (11:19 -0500)
committerDerrick Brashear <shadow@dementia.org>
Sun, 6 Mar 2011 18:08:09 +0000 (10:08 -0800)
Remove some unused code and associated comment.  rx_Init does
belong earlier in the sequence.

Change-Id: I9db399b8ae3343e0c2c14f64150d55a2411899c3
Reviewed-on: http://gerrit.openafs.org/4145
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>

src/ubik/ubik.c

index c91ce7d..cb9a9c6 100644 (file)
@@ -451,17 +451,6 @@ ubik_ServerInitCommon(afs_uint32 myHost, short myPort,
     }
     /* for backwards compat this should keep working as it does now
        and not host bind */
-#if 0
-    /* This really needs to be up above, where I have put it.  It works
-     * here when we're non-pthreaded, but the code above, when using
-     * pthreads may (and almost certainly does) end up calling on a
-     * pthread resource which gets initialized by rx_Init.  The end
-     * result is that an assert fails and the program dies. -- klm
-     */
-    code = rx_Init(myPort);
-    if (code < 0)
-       return code;
-#endif
 
     tservice =
        rx_NewService(0, VOTE_SERVICE_ID, "VOTE", ubik_sc, 3,