From accde4343cf7b85620c0efbdedf5ed5b8e711fba Mon Sep 17 00:00:00 2001 From: Douglas Gregor Date: Mon, 17 Nov 2008 17:14:10 +0000 Subject: [PATCH] Add TODO item for adding more source-range information to declarators llvm-svn: 59446 --- clang/TODO.txt | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/clang/TODO.txt b/clang/TODO.txt index 111f747275d9..78f965b426aa 100644 --- a/clang/TODO.txt +++ b/clang/TODO.txt @@ -32,3 +32,24 @@ This specific diagnostic is implemented, but others should be updated. C++ (checker): For iterators, warn of the use of "iterator++" instead of "++iterator" when when the value returned by operator++(int) is ignored. + +//===---------------------------------------------------------------------===// + +We want to keep more source range information in Declarator to help +produce better diagnostics. Declarator::getSourceRange() should be +implemented to give a range for the whole declarator with all of its +specifiers, and DeclaratorChunk::ParamInfo should also have a source +range covering the whole parameter, so that an error message like this: + +overloaded-operator-decl.cpp:37:23: error: parameter of overloaded post-increment operator must have type 'int' (not 'float') +X operator++(X&, const float& f); + ^ +can be turned into something like this: + +overloaded-operator-decl.cpp:37:23: error: parameter of overloaded post-increment operator must have type 'int' (not 'float') +X operator++(X&, const float& f); + ^ ~~~~~~~~~~~~~~ + + + +