From ccf428b1dfeeaf9db66aaa2d5fe23a12768562c9 Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Fri, 5 Feb 2021 14:02:36 +0100 Subject: [PATCH] Remove an obsolete comment The caller codepaths have evolved to the point where copySubstatement() is again always called only when we are truly materializing statements. This renders the comment obsolete, remove it. JIRA: YANGTOOLS-1195 Change-Id: I913a6685b712b008ea21597e2341d8c7c96c24d6 Signed-off-by: Robert Varga --- .../yang/parser/stmt/reactor/InferredStatementContext.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/yang/yang-parser-reactor/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/InferredStatementContext.java b/yang/yang-parser-reactor/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/InferredStatementContext.java index 95e68c9172..c5eeb87286 100644 --- a/yang/yang-parser-reactor/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/InferredStatementContext.java +++ b/yang/yang-parser-reactor/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/InferredStatementContext.java @@ -538,10 +538,6 @@ final class InferredStatementContext, E extend } private Optional> copySubstatement(final Mutable substatement) { - // FIXME: YANGTOOLS-1195: this is not exactly what we want to do here, because we are dealing with two different - // requests: copy for inference purposes (this method), while we also copy for purposes - // of buildEffective() -- in which case we want to probably invoke asEffectiveChildOf() - // or similar return substatement.copyAsChildOf(this, childCopyType, targetModule); } -- 2.36.6