Minor comments in HttpServer.ts.

Author: JeremyTCD

src/NodeJS/Javascript/Servers/OutOfProcess/Http/HttpServer.ts

@@ -278,9 +278,11 @@ function demarcateMessageEndings(outputStream: NodeJS.WritableStream) {
         if (typeof value === 'string') {
             // Node appends a new line character at the end of the message. This facilitates reading of the stream: the process reading it reads messages line by line -
             // characters stay in its buffer until a new line character is received. This means that the null terminating character must appear before the last
-            // new line character of the message. This approach is slightly heavy handed in that it removes all whitespace at the end of the message, generally,
-            // such whitespace is pointless.
-            arguments[0] = value.trimRight() + '\0\n';
+            // new line character of the message. This approach is inefficent since it allocates 2 new strings.
+            //
+            // TODO consider sending '\0\n' as a demarcator after sending value (profile). Also need to check if logging from worker threads might affect 
+            // such an approach.
+            arguments[0] = value.slice(0, value.length - 1) + '\0\n';
         }
 
         origWriteFunction.apply(this, arguments);