-
Notifications
You must be signed in to change notification settings - Fork 261
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Last update for warp 3.3.30 #947
Last update for warp 3.3.30 #947
Conversation
Vlix
commented
Oct 13, 2023
•
edited
Loading
edited
- Update the Changelog.md file with a link to your PR
- Check that CI passes (or if it fails, for reasons unrelated to your change, like CI timeouts)
…count up to 2.1GB on 32-bit machines)
@@ -10,7 +8,7 @@ import Network.Wai.Handler.Warp.Buffer | |||
import Network.Wai.Handler.Warp.Imports | |||
import Network.Wai.Handler.Warp.Types | |||
|
|||
toBufIOWith :: Int -> IORef WriteBuffer -> (ByteString -> IO ()) -> Builder -> IO Int | |||
toBufIOWith :: Int -> IORef WriteBuffer -> (ByteString -> IO ()) -> Builder -> IO Integer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't Int64
good enough?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would also be fine, but then we'd have to fromIntegral
it to Integer
at the next step anyway. So thought, why not just do Integer
immediately.
EDIT: This reminded me I hadn't actually removed the fromIntegral
, I've done that now.
Removed the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now LGTM!
Merged. Thanks! |
Tagged commit as |