[cfe-commits] Patch for review: Parenthesis range in CXXConstructExprs

Daniel Jasper djasper at google.com
Mon Jul 9 06:24:08 PDT 2012


I have noticed that the CXXConstructExpr is not always created with the
correct parenthesis range. For example:

struct X { X(int x) {} };
void f() {
  X value(7);  // CXXConstructExpr has correct range (5, 12)
  X(7);  // CXXConstructExpr has incorrect range (3, 5), does not include
")"
}

Although I am not 100% sure this is a bug and not a feature, the attached
patch fixes it. Please take a look.

Cheers,
Daniel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120709/9b7065f8/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cxxconstructexpr.patch
Type: application/octet-stream
Size: 3134 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120709/9b7065f8/attachment.obj>


More information about the cfe-commits mailing list