From d0128787e74c36d589f00ac89ab17902dd8e0de0 Mon Sep 17 00:00:00 2001
From: Ryan Dahl <ry@tinyclouds.org>
Date: Wed, 12 May 2010 12:15:58 -0700
Subject: Deprecate sendHeader() and writeHeader(), ppl should use writeHead()

---
 lib/http.js | 28 ++++++++++++++++++++++++----
 1 file changed, 24 insertions(+), 4 deletions(-)

diff --git a/lib/http.js b/lib/http.js
index 633bb86e9..352a05ed3 100644
--- a/lib/http.js
+++ b/lib/http.js
@@ -183,8 +183,15 @@ IncomingMessage.prototype._parseQueryString = function () {
   throw new Error("_parseQueryString is deprecated. Use require(\"querystring\") to parse query strings.\n");
 };
 
+var setBodyEncodingWarning;
+
 IncomingMessage.prototype.setBodyEncoding = function (enc) {
-  // TODO deprecation message?
+  // deprecation message
+  if (!setBodyEncodingWarning) {
+    setBodyEncodingWarning = "setBodyEncoding has been renamed to setEncoding, please update your code.";
+    sys.error(setBodyEncodingWarning);
+  }
+
   this.setEncoding(enc);
 };
 
@@ -446,9 +453,22 @@ ServerResponse.prototype.writeHead = function (statusCode) {
   this.headWritten = true;
 };
 
-// TODO eventually remove sendHeader(), writeHeader()
-ServerResponse.prototype.sendHeader = ServerResponse.prototype.writeHead;
-ServerResponse.prototype.writeHeader = ServerResponse.prototype.writeHead;
+// TODO Eventually remove
+var sendHeaderWarning, writeHeaderWarning;
+ServerResponse.prototype.sendHeader = function () {
+  if (!sendHeaderWarning) {
+    sendHeaderWarning = "sendHeader() has been renamed to writeHead()";
+    sys.error(sendHeaderWarning);
+  }
+  this.writeHead.apply(this, arguments);
+};
+ServerResponse.prototype.writeHeader = function () {
+  if (!writeHeaderWarning) {
+    writeHeaderWarning = "writeHeader() has been renamed to writeHead()";
+    sys.error(writeHeaderWarning);
+  }
+  this.writeHead.apply(this, arguments);
+};
 
 function ClientRequest (socket, method, url, headers) {
   OutgoingMessage.call(this, socket);
-- 
cgit v1.2.3