[TS-2808] Remove comments about deprecated TSVConnCreate API

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 5.0.0
• Component/s: TS API
• Labels: api-change
• Assignee: Leif Hedstrom
• Reporter: Leif Hedstrom
• Created: 14/May/14 04:09
• Updated: 18/Dec/14 05:06
• Resolved: 23/May/14 17:11

Description

There seems to be no use of this API, no documentations, and the code seems to imply (as pointed out by Peach), that it can never work. We should nuke it for v5.0.0.

/* Deprectated.
   Do not use this API.
   The reason is even if VConn is created using this API, it is still useless.
   For example, if we do TSVConnRead, the read operation returns read_vio, if
   we do TSVIOReenable (read_vio), it actually calls:
   void VIO::reenable()
   {
       if (vc_server) vc_server->reenable(this);
   }
   vc_server->reenable calls:
   VConnection::reenable(VIO)

   this function is virtual in VConnection.h. It is defined separately for
   UnixNet, NTNet and CacheVConnection.

   Thus, unless VConn is either NetVConnection or CacheVConnection, it can't
   be instantiated for functions like reenable.

   Meanwhile, this function has never been used.
   */
TSVConn
TSVConnCreate(TSEventFunc event_funcp, TSMutex mutexp)