[cfe-dev] getLocStart versus getStartLoc
Stephen Kelly via cfe-dev
cfe-dev at lists.llvm.org
Mon Jul 9 14:05:45 PDT 2018
Stephen Kelly via cfe-dev wrote:
> 1) DeclarationNameInfo has both getLocEnd and getEndLoc. Both are used by
> calling code, and I can't tell if any use is incorrect. At any rate, if
> the two methods are to remain, the getLocEnd() one should be renamed to
> something more specific as it is the one doing something different to
> other getLocEnd/getEndLoc methods which generally don't 'try again' in the
> invalid case.
I created a patch to inline getLocEnd() into callers.
https://reviews.llvm.org/D49100
> <snip>
> Is that something we would really want to do? I value consistency, but
> breaking all Clang API users so dramatically is something to do only if it
> is seen as valuable to a consensus of the community.
This is still pending discussion of some sort.
Thanks,
Stephen.
More information about the cfe-dev
mailing list