amend comments.

vesoft-inc · Dec 15, 2022 · 616b7f1 · 616b7f1
1 parent ad9c822
commit 616b7f1
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/common/expression/AttributeExpression.cpp b/src/common/expression/AttributeExpression.cpp
@@ -24,9 +24,8 @@ const Value &AttributeExpression::eval(ExpressionContext &ctx) {
       return lvalue.getMap().at(rvalue.getStr());
     case Value::Type::VERTEX: {
       /*
-       * WARNING(Xuntao): this entire case shall be removed, in theory.
-       * Vertices shall not be evaluated as AttributeExpressions, since
-       * there shall always be a tag specified in the format of:
+       * WARNING(Xuntao): Vertices shall not be evaluated as AttributeExpressions,
+       * since there shall always be a tag specified in the format of:
        * var.tag.property. Due to design flaws, however, we have to keep
        * this case segment.
        */
@@ -35,7 +34,7 @@ const Value &AttributeExpression::eval(ExpressionContext &ctx) {
         return result_;
       }
       /*
-       * WARNING(Xuntao): the following code snippet is a dedicated to address the legacy
+       * WARNING(Xuntao): the following code snippet is dedicated to address the legacy
        * problem of treating LabelTagProperty expressions as Attribute expressions.
        * This snippet is necessary to allow users to write var.tag.prop in
        * ListComprehensionExpression without making structural changes to the implementation.