From: Moshe Zadka <moshez@math.huji.ac.il>
Date: Sat, 29 Jul 2000 05:15:56 +0000 (+0000)
Subject: Using shutil's function to copy data between file objects instead
X-Git-Tag: v2.0b1~691
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=37c03ff3a95d5518d54364e232c35c50ab469edb;p=thirdparty%2FPython%2Fcpython.git

Using shutil's function to copy data between file objects instead
of rolling our own.

Note:
This is my first sourceforge checkin. If you see anything funny about
this patch, please let me know.
---

diff --git a/Lib/SimpleHTTPServer.py b/Lib/SimpleHTTPServer.py
index 71254131202b..8a7775856c96 100644
--- a/Lib/SimpleHTTPServer.py
+++ b/Lib/SimpleHTTPServer.py
@@ -15,6 +15,7 @@ import posixpath
 import BaseHTTPServer
 import urllib
 import cgi
+import shutil
 from StringIO import StringIO
 
 
@@ -151,12 +152,7 @@ class SimpleHTTPRequestHandler(BaseHTTPServer.BaseHTTPRequestHandler):
         to copy binary data as well.
 
         """
-
-        BLOCKSIZE = 8192
-        while 1:
-            data = source.read(BLOCKSIZE)
-            if not data: break
-            outputfile.write(data)
+        shutil.copyfileobj(source, outputfile)
 
     def guess_type(self, path):
         """Guess the type of a file.