Remove an obsolete comment 46/95046/1
authorRobert Varga <robert.varga@pantheon.tech>
Fri, 5 Feb 2021 13:02:36 +0000 (14:02 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Fri, 5 Feb 2021 13:04:15 +0000 (14:04 +0100)
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 <robert.varga@pantheon.tech>
yang/yang-parser-reactor/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/InferredStatementContext.java

index 95e68c91727d027236dac2283f7086ec58bb8c08..c5eeb87286bb70a14ef33b0ba226aa829cfc9311 100644 (file)
@@ -538,10 +538,6 @@ final class InferredStatementContext<A, D extends DeclaredStatement<A>, E extend
     }
 
     private Optional<? extends Mutable<?, ?, ?>> 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);
     }