For the usage in StartRPCThreads this is ok (testnet is parsed way before calling StartRPCThreads), but in the CommandLineRPC case this is incorrect, as fTestNet is never set.