Remove socket sendfile wrong implementation

rodrigocam · rodrigocam · commit f4b9bd66cbf3 · 2020-10-27T17:28:38.000-03:00
diff --git a/extra_tests/snippets/stdlib_socket.py b/extra_tests/snippets/stdlib_socket.py
@@ -23,6 +23,14 @@
 assert recv_a == MESSAGE_A
 assert recv_b == MESSAGE_B
 
+fd = open('README.md', 'rb')
+connector.sendfile(fd)
+recv_readme = connection.recv(os.stat('README.md').st_size)
+# need this because sendfile leaves the cursor at the end of the file
+fd.seek(0)
+assert recv_readme == fd.read()
+fd.close()
+
 # fileno
 if os.name == "posix":
 	connector_fd = connector.fileno()
diff --git a/vm/src/stdlib/socket.rs b/vm/src/stdlib/socket.rs
@@ -21,7 +21,6 @@ use crate::pyobject::{
     BorrowValue, Either, IntoPyObject, PyClassImpl, PyObjectRef, PyRef, PyResult, PyValue,
     StaticType, TryFromObject,
 };
-use crate::stdlib::os::rust_file;
 use crate::vm::VirtualMachine;
 
 #[cfg(unix)]
@@ -221,31 +220,6 @@ impl PySocket {
         Ok(())
     }
 
-    #[pymethod]
-    fn sendfile(&self, fd: i64, offset: OptionalArg<u64>, count: OptionalArg<u64>, vm: &VirtualMachine) -> PyResult<usize> {
-        let mut file = rust_file(fd);
-        let mut bufsize: u64 = 0;
-
-        if let Ok(metadata) = file.metadata() {
-            bufsize = metadata.len();
-        }
-
-        if let OptionalArg::Present(c) = count {
-            bufsize = c;
-        }
-
-        let mut buffer = vec![0u8; bufsize as usize];
-
-        let n = file
-            .read(&mut buffer)
-            .map_err(|err| err.into_pyexception(vm))?;
-
-        buffer.truncate(n);
-
-        let bytes = PyBytesLike::try_from_object(vm, vm.ctx.new_bytes(buffer))?;
-        self.send(bytes, vm)
-    }
-
     #[pymethod]
     fn close(&self) {
         *self.sock_mut() = invalid_sock();
