Correctly override size_hint and is_end_stream for Empty (#42)

I noticed this while working on hyperium/tonic#622 as it broke some interop tests.
hyperium · May 8, 2021 · 7b811bb · 7b811bb
1 parent e5bd0d1
commit 7b811bb
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,6 @@
 # Unreleased
 
-Nothing.
+- Correctly override `Body::size_hint` and `Body::is_end_stream` for `Empty`.
 
 # 0.4.1 (March 18, 2021)
 

diff --git a/src/empty.rs b/src/empty.rs
@@ -1,4 +1,4 @@
-use super::Body;
+use super::{Body, SizeHint};
 use bytes::Buf;
 use http::HeaderMap;
 use std::{
@@ -40,6 +40,14 @@ impl<D: Buf> Body for Empty<D> {
     ) -> Poll<Result<Option<HeaderMap>, Self::Error>> {
         Poll::Ready(Ok(None))
     }
+
+    fn is_end_stream(&self) -> bool {
+        true
+    }
+
+    fn size_hint(&self) -> SizeHint {
+        SizeHint::with_exact(0)
+    }
 }
 
 impl<D> fmt::Debug for Empty<D> {