Commons:User scripts/users namespace/reports/User:Rillke/CodeMirror/addon/merge/merge.js
Jump to navigation
Jump to search
This is the report page for User:Rillke/CodeMirror/addon/merge/merge.js. It is bot-maintained. All manual changes might be overwritten.
jshint
[edit]- ISSUE:
line 41 character 19
: Missing 'new' prefix when invoking a constructor. - Evidence:this.orig = CodeMirror(pane, copyObj({value: orig, readOnly: !this.mv.options.allowEditingOriginals}, copyObj(options)));
- ISSUE:
line 108 character 32
: 'change' is already defined. - Evidence:function change(_cm, change) {
- ISSUE:
line 141 character 35
: Missing '()' invoking a constructor. - Evidence:var editor, other, now = +new Date;
- ISSUE:
line 152 character 26
: A leading decimal point can be confused with a dot: '.5'. - Evidence:var halfScreen = .5 * sInfo.clientHeight, midY = sInfo.top + halfScreen;
- ISSUE:
line 173 character 32
: 'targetPos' used out of scope. - Evidence:other.scrollTo(sInfo.left, targetPos);
- ISSUE:
line 181 character 13
: Use '===' to compare with 'null'. - Evidence:if (bot == null) bot = editor.lastLine() + 1;
- ISSUE:
line 188 character 23
: Use '!==' to compare with 'false'. - Evidence:if (val && action != false) syncScroll(dv, DIFF_INSERT) && makeConnections(dv);
- ISSUE:
line 230 character 15
: Missing 'new' prefix when invoking a constructor. - Evidence:var pos = Pos(0, 0);
- ISSUE:
line 231 character 15
: Missing 'new' prefix when invoking a constructor. - Evidence:var top = Pos(from, 0), bot = editor.clipPos(Pos(to - 1));
- ISSUE:
line 231 character 50
: Missing 'new' prefix when invoking a constructor. - Evidence:var top = Pos(from, 0), bot = editor.clipPos(Pos(to - 1));
- ISSUE:
line 292 character 60
: 'w' used out of scope. - Evidence:drawConnectorsForChunk(dv, ch, sTopOrig, sTopEdit, w);
- ISSUE:
line 315 character 18
: 'i' is already defined. - Evidence:for (var i = 0; i < other.chunks.length; i++) {
- ISSUE:
line 316 character 19
: 'chunk' is already defined. - Evidence:var chunk = other.chunks[i];
- ISSUE:
line 355 character 16
: 'i' is already defined. - Evidence:for (var i = 0; i < cm.length; i++)
- ISSUE:
line 361 character 16
: 'i' is already defined. - Evidence:for (var i = 0; i < cm.length; i++)
- ISSUE:
line 367 character 54
: Use '!==' to compare with 'null'. - Evidence:for (var i = 0; i < cm.length; i++) if (lines[i] != null) {
- ISSUE:
line 372 character 16
: 'i' is already defined. - Evidence:for (var i = 0; i < cm.length; i++) if (lines[i] != null) {
- ISSUE:
line 372 character 54
: Use '!==' to compare with 'null'. - Evidence:for (var i = 0; i < cm.length; i++) if (lines[i] != null) {
- ISSUE:
line 400 character 27
: 'tmp' is already defined. - Evidence:if (flip) { var tmp = botLpx; botLpx = botRpx; botRpx = tmp; }
- ISSUE:
line 430 character 35
: Missing 'new' prefix when invoking a constructor. - Evidence:to.replaceRange(from.getRange(Pos(chunk.origFrom, 0), Pos(chunk.origTo, 0)),
- ISSUE:
line 430 character 59
: Missing 'new' prefix when invoking a constructor. - Evidence:to.replaceRange(from.getRange(Pos(chunk.origFrom, 0), Pos(chunk.origTo, 0)),
- ISSUE:
line 431 character 26
: Missing 'new' prefix when invoking a constructor. - Evidence:Pos(chunk.editFrom, 0), Pos(chunk.editTo, 0));
- ISSUE:
line 431 character 50
: Missing 'new' prefix when invoking a constructor. - Evidence:Pos(chunk.editFrom, 0), Pos(chunk.editTo, 0));
- ISSUE:
line 440 character 68
: Use '===' to compare with 'null'. - Evidence:var origLeft = options.origLeft, origRight = options.origRight == null ? options.orig : options.origRight;
- ISSUE:
line 442 character 28
: Use '!==' to compare with 'null'. - Evidence:var hasLeft = origLeft != null, hasRight = origRight != null;
- ISSUE:
line 442 character 58
: Use '!==' to compare with 'null'. - Evidence:var hasLeft = origLeft != null, hasRight = origRight != null;
- ISSUE:
line 464 character 17
: 'rightPane' used out of scope. - Evidence:(hasRight ? rightPane : editPane).className += " CodeMirror-merge-pane-rightmost";
- ISSUE:
line 469 character 17
: Missing 'new' prefix when invoking a constructor. - Evidence:this.edit = CodeMirror(editPane, copyObj(options));
- ISSUE:
line 471 character 25
: 'leftPane' used out of scope. - Evidence:if (left) left.init(leftPane, origLeft, options);
- ISSUE:
line 472 character 27
: 'rightPane' used out of scope. - Evidence:if (right) right.init(rightPane, origRight, options);
- ISSUE:
line 523 character 20
: Did you mean to return a conditional instead of an assignment? - Evidence:return dv.gap = elt("div", gapElts, "CodeMirror-merge-gap");
- ISSUE:
line 550 character 17
: A constructor name should start with an uppercase letter. - Evidence:var dmp = new diff_match_patch();
- ISSUE:
line 570 character 16
: Missing 'new' prefix when invoking a constructor. - Evidence:var edit = Pos(0, 0), orig = Pos(0, 0);
- ISSUE:
line 570 character 34
: Missing 'new' prefix when invoking a constructor. - Evidence:var edit = Pos(0, 0), orig = Pos(0, 0);
- ISSUE:
line 604 character 11
: Use '===' to compare with '0'. - Evidence:if (i == 0) return true;
- ISSUE:
line 618 character 18
: Use '===' to compare with 'null'. - Evidence:if (afterE == null) {
- ISSUE:
line 633 character 28
: Missing 'new' prefix when invoking a constructor. - Evidence:var mark = cm.markText(Pos(from, 0), Pos(to - 1), {
- ISSUE:
line 633 character 42
: Missing 'new' prefix when invoking a constructor. - Evidence:var mark = cm.markText(Pos(from, 0), Pos(to - 1), {
- ISSUE:
line 721 character 22
: Missing 'new' prefix when invoking a constructor. - Evidence:var out = copy ? Pos(pos.line, pos.ch) : pos, at = 0;
- ISSUE:
line 760 character 15
: Use '!==' to compare with 'null'. - Evidence:if (pos != null && (found == null || (dir < 0 ? pos > found : pos < found)))
- ISSUE:
line 760 character 33
: Use '===' to compare with 'null'. - Evidence:if (pos != null && (found == null || (dir < 0 ? pos > found : pos < found)))
- ISSUE:
line 763 character 15
: Use '!==' to compare with 'null'. - Evidence:if (found != null)