Skip to content

Commit 94b95e8

Browse files
author
Chris K
committed
Improved fix for DFL-3333 and DFL-3328.
1 parent 0c1ca74 commit 94b95e8

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

src/ecma-debugger/jssourcetooltip.js

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ cls.JSSourceTooltip = function(view)
2626
var TOOLTIP_NAME = cls.JSInspectionTooltip.tooltip_name;
2727
var MAX_MOUSE_POS_COUNT = 2;
2828
var FILTER_HANDLER = "js-tooltip-filter";
29-
var CONTROL_KEYWORD = ["while", "for", "if", "switch"];
29+
var KEYWORD_BEFORE_BARCETS_BLACKLIST = ["while", "for", "if", "switch", "return"];
3030

3131
var _tooltip = null;
3232
var _view = null;
@@ -492,7 +492,8 @@ cls.JSSourceTooltip = function(view)
492492
}
493493
case IDENTIFIER:
494494
{
495-
if (previous_token[VALUE] == "(" && CONTROL_KEYWORD.contains(token[VALUE]))
495+
if (previous_token[VALUE] == "(" &&
496+
KEYWORD_BEFORE_BARCETS_BLACKLIST.contains(token[VALUE]))
496497
break;
497498
}
498499
case STRING:
@@ -600,9 +601,7 @@ cls.JSSourceTooltip = function(view)
600601

601602
while (bracket_stack.length || (shift_key && parens_stack.length))
602603
{
603-
for (var i = match_index + 1, token = null, break_loop = false;
604-
!break_loop && (token = tokens[i]);
605-
i++)
604+
for (var i = match_index + 1, token = null; token = tokens[i]; i++)
606605
{
607606
// consume everything between parentheses if shiftKey is pressed
608607
if (shift_key && parens_stack.length)
@@ -781,10 +780,10 @@ cls.JSSourceTooltip = function(view)
781780
}
782781
}
783782
}
784-
break_loop = true;
785783
break;
786784
}
787785
}
786+
break;
788787
}
789788

790789
if (i == tokens.length && bracket_stack.length)

0 commit comments

Comments
 (0)