• Mingtao Yang's avatar
    Deprecate AsyncTransportWrapper · 2586be5b
    Mingtao Yang authored
    Summary:
    All implementations of transports in practice were all AsyncTransportWrappers
    (AsyncSocket, AsyncSSLSocket, AsyncFizzBase, etc.).
    
    AsyncTransportWrapper was confusing. The only additional functionality that
    it provided was the ability to query underlying transports, a piece of
    functionality which makes just as much sense on AsyncTransport.
    
    The simplified model:
    
    "AsyncTransport deals with bidirectional I/O. It is both an AsyncReader, and
    an AsyncWriter. It may be nested."
    
    This diff changes AsyncTransportWrapper usages in folly to refer to
    AsyncTransport, and aliases AsyncTransportWrapper for compatibility.
    
    Reviewed By: yfeldblum
    
    Differential Revision: D21915048
    
    fbshipit-source-id: 741fad91b8f7c8080f942168f5513b12602cfe9a
    2586be5b
WriteChainAsyncTransportWrapperTest.cpp 2.72 KB