I just learned about CaptureListExpr when working on some diagnostics. Is
there a particular reason that its member "closureBody" is an Expr* and not
a ClosureExpr*? There seems to be only one place it's built
<https://github.com/apple/swift/blob/1e46f13184d7256c991b2ba9424af9efe3cd860f/lib/Parse/ParseExpr.cpp#L2425>,
and the body is always a ClosureExpr.

I can see one minor place
<https://github.com/apple/swift/blob/1e46f13184d7256c991b2ba9424af9efe3cd860f/lib/AST/ASTWalker.cpp#L663-L665>
where it might be less convenient to have a ClosureExpr, but otherwise
there doesn't seem to be much of a reason to keep it generalized to Expr*.

Jacob
_______________________________________________
swift-dev mailing list
swift-dev@swift.org
https://lists.swift.org/mailman/listinfo/swift-dev

Reply via email to