treat nulls more carefully in presense of weak/concurrent maps to avoid NPEs, annotate relevant arguments NotNull to make clear they won't cause troubles

GitOrigin-RevId: 50d40addab348d1da6bd77afc76e88cd037e87ef
This commit is contained in:
Alexey Kudravtsev
2021-11-08 16:23:29 +01:00
committed by intellij-monorepo-bot
parent dc46a5bb8c
commit 99caec4aa9
18 changed files with 49 additions and 60 deletions

View File

@@ -234,7 +234,7 @@ public class StackCapturingLineBreakpoint extends SyntheticMethodBreakpoint {
if (evaluator == null) {
@SuppressWarnings("ConstantConditions")
Location location = context.getFrameProxy().location();
evaluator = myEvaluatorCache.get(location);
evaluator = location == null ? null : myEvaluatorCache.get(location);
if (evaluator == null && !StringUtil.isEmpty(myExpression)) {
evaluator = ApplicationManager.getApplication().runReadAction((ThrowableComputable<ExpressionEvaluator, EvaluateException>)() -> {
SourcePosition sourcePosition = ContextUtil.getSourcePosition(context);