Permalink
Browse files

tools: favor === over == in license2rtf.js

PR-URL: https://github.com/nodejs/node/pull/8068.patch
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Prince John Wesley <princejohnwesley@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: targos - Michaël Zasso <mic.besace@gmail.com>
  • Loading branch information...
Trott authored and fhinkel committed Aug 11, 2016
1 parent fce4b98 commit 3fe55f6f91ec106b6f446cf18aee1594d1a8bd78
Showing with 4 additions and 4 deletions.
  1. +4 −4 tools/license2rtf.js
View
@@ -104,7 +104,7 @@ function ParagraphParser() {
if (block_has_c_style_comment) {
var prev = line;
line = line.replace(/^(\s*?)(?:\s?\*\/|\/\*\s|\s\*\s?)/, '$1');
if (prev == line)
if (prev === line)
line = line.replace(/^\s{2}/, '');
if (/\*\//.test(prev))
block_has_c_style_comment = false;
@@ -142,16 +142,16 @@ function ParagraphParser() {
var line_li = result[2];
// Flush the paragraph when there is a li or an indentation jump
if (line_li || (line_indent != paragraph_line_indent &&
paragraph_line_indent != -1)) {
if (line_li || (line_indent !== paragraph_line_indent &&
paragraph_line_indent !== -1)) {
flushParagraph();
paragraph.li = line_li;
}
// Set the paragraph indent that we use to detect indentation jumps. When
// we just detected a list indicator, wait
// for the next line to arrive before setting this.
if (!line_li && paragraph_line_indent != -1) {
if (!line_li && paragraph_line_indent !== -1) {
paragraph_line_indent = line_indent;
}

2 comments on commit 3fe55f6

@MylesBorins

This comment has been minimized.

Member

MylesBorins replied Sep 30, 2016

@Trott PR-URL includes the patch.

@Trott

This comment has been minimized.

Member

Trott replied Sep 30, 2016

Argh, sorry. Insert standard rumination on the benefits of automation here. :-(

Please sign in to comment.