Skip to content

Commit

Permalink
Obsolete TODOs
Browse files Browse the repository at this point in the history
  • Loading branch information
edsko committed Oct 25, 2024
1 parent c501e76 commit 8ee46de
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 5 deletions.
2 changes: 0 additions & 2 deletions grapesy/src/Network/GRPC/Client/Session.hs
Original file line number Diff line number Diff line change
Expand Up @@ -109,8 +109,6 @@ instance SupportsClientRpc rpc => InitiateSession (ClientSession rpc) where
case verifyAllIf connVerifyHeaders responseHeaders of
Left err -> throwIO $ CallSetupInvalidResponseHeaders err
Right hdrs -> do
-- TODO: <https://github.com/well-typed/grapesy/issues/203>
-- If we omit this call to 'updateConnectionMeta', no tests fail.
Connection.updateConnectionMeta conn responseHeaders
cIn <- getCompression $ requiredResponseCompression hdrs
return $ FlowStartRegular $ InboundHeaders {
Expand Down
3 changes: 0 additions & 3 deletions grapesy/src/Network/GRPC/Util/Session/Channel.hs
Original file line number Diff line number Diff line change
Expand Up @@ -141,9 +141,6 @@ data RegularFlowState flow = RegularFlowState {
-- This TMVar is written to for incoming messages ('recvMessageLoop') and
-- read from for outgoing messages ('sendMessageLoop'). It acts as a
-- one-place buffer, providing backpressure in both directions.
--
-- TODO: <https://github.com/well-typed/grapesy/issues/118>.
-- It might make sense to generalize this to an @N@-place buffer.
, flowMsg :: TMVar (StreamElem (Trailers flow) (Message flow))

-- | Trailers
Expand Down

0 comments on commit 8ee46de

Please sign in to comment.