[Issue 15403] New: [internal] FileExp represents ImportExpression, the AST class naming is not intuitive

via Digitalmars-d-bugs digitalmars-d-bugs at puremagic.com
Thu Dec 3 16:51:09 PST 2015


https://issues.dlang.org/show_bug.cgi?id=15403

          Issue ID: 15403
           Summary: [internal] FileExp represents ImportExpression, the
                    AST class naming is not intuitive
           Product: D
           Version: D2
          Hardware: All
                OS: All
            Status: NEW
          Keywords: ice
          Severity: trivial
          Priority: P1
         Component: dmd
          Assignee: nobody at puremagic.com
          Reporter: k.hara.pg at gmail.com

In expression.d:

extern (C++) final class FileExp : UnaExp
{
public:
    extern (D) this(Loc loc, Expression e)
    {
        super(loc, TOKimport, __traits(classInstanceSize, FileExp), e);
    }

By recent issue 15394 fix (PR #5294), it's changed to use TOKimport. So I think
it's a time to rename FileExp to ImportExp.

--


More information about the Digitalmars-d-bugs mailing list