Skip to content

Commit

Permalink
buffer: fix comments in bidirectionalIndexOf
Browse files Browse the repository at this point in the history
PR-URL: #10162
Fixes: #9801
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
dcposch authored and silverwind committed Jan 24, 2017
1 parent d0112fc commit 39f65e6
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions lib/buffer.js
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -598,9 +598,10 @@ function bidirectionalIndexOf(buffer, val, byteOffset, encoding, dir) {
} else if (byteOffset < -0x80000000) { } else if (byteOffset < -0x80000000) {
byteOffset = -0x80000000; byteOffset = -0x80000000;
} }
byteOffset = +byteOffset; // Coerce to Number. // Coerce to Number. Values like null and [] become 0.
if (isNaN(byteOffset)) { byteOffset = +byteOffset;
// If the offset is undefined, null, NaN, "foo", etc, search whole buffer. // If the offset is undefined, "foo", {}, coerces to NaN, search whole buffer.
if (Number.isNaN(byteOffset)) {
byteOffset = dir ? 0 : (buffer.length - 1); byteOffset = dir ? 0 : (buffer.length - 1);
} }
dir = !!dir; // Cast to bool. dir = !!dir; // Cast to bool.
Expand Down

0 comments on commit 39f65e6

Please sign in to comment.