Move comments out of functions

Author: Ruben Bridgewater

lib/parser.js

@@ -63,13 +63,15 @@ function parseStringNumbers (parser) {
 /**
  * Returns a string or buffer of the provided offset start and
  * end ranges. Checks `optionReturnBuffers`.
+ *
+ * If returnBuffers is active, all return values are returned as buffers besides numbers and errors
+ *
  * @param parser
  * @param start
  * @param end
  * @returns {*}
  */
 function convertBufferRange (parser, start, end) {
-  // If returnBuffers is active, all return values are returned as buffers besides numbers and errors
   parser.offset = end + 2
   if (parser.optionReturnBuffers === true) {
     return parser.buffer.slice(start, end)
@@ -111,13 +113,15 @@ function parseLength (parser) {
 
 /**
  * Parse a ':' redis integer response
+ *
+ * If stringNumbers is activated the parser always returns numbers as string
+ * This is important for big numbers (number > Math.pow(2, 53)) as js numbers
+ * are 64bit floating point numbers with reduced precision
+ *
  * @param parser
  * @returns {*}
  */
 function parseInteger (parser) {
-  // If stringNumbers is activated the parser always returns numbers as string
-  // This is important for big numbers (number > Math.pow(2, 53)) as js numbers
-  // are 64bit floating point numbers with reduced precision
   if (parser.optionStringNumbers) {
     return parseStringNumbers(parser)
   }
@@ -283,13 +287,17 @@ function JavascriptRedisParser (options) {
 
 /**
  * Concat a bulk string containing multiple chunks
+ *
+ * Notes:
+ * 1) The first chunk might contain the whole bulk string including the \r
+ * 2) We are only safe to fully add up elements that are neither the first nor any of the last two elements
+ *
  * @param parser
  * @param buffer
  * @returns {String}
  */
 function concatBulkString (parser) {
   var list = parser.bufferCache
-  // The first chunk might contain the whole bulk string including the \r
   var chunks = list.length
   var offset = parser.bigStrSize - parser.totalChunkSize
   parser.offset = offset
@@ -302,7 +310,6 @@ function concatBulkString (parser) {
   }
   var res = decoder.write(list[0].slice(parser.bigOffset))
   for (var i = 1; i < chunks - 2; i++) {
-    // We are only safe to fully add up elements that are neither the first nor any of the last two elements
     res += decoder.write(list[i])
   }
   res += decoder.end(list[i].slice(0, offset === 1 ? list[i].length - 1 : offset - 2))
@@ -377,7 +384,7 @@ function concatBuffer (parser, length) {
  * @param buffer
  * @returns {undefined}
  */
-JavascriptRedisParser.prototype.execute = function (buffer) {
+JavascriptRedisParser.prototype.execute = function execute (buffer) {
   if (this.buffer === null) {
     this.buffer = buffer
     this.offset = 0
@@ -418,9 +425,9 @@ JavascriptRedisParser.prototype.execute = function (buffer) {
     }
 
     if (type === 45) {
-      this.returnError(response) // Errors -
+      this.returnError(response)
     } else {
-      this.returnReply(response) // Strings + // Integers : // Bulk strings $ // Arrays *
+      this.returnReply(response)
     }
   }