In order to increase consistency and learnability of AST APIs, I have committed the following transformations on AST Node APIs:
getLocStart -> getBeginLoc
getStartLoc -> getBeginLoc
getLocEnd -> getEndLoc
This is consistent with other SourceLocation accessors, which follow a pattern of get.*Loc.
The rationale for this change can be reviewed here:
I have already ported the clang and clang-tools-extras code to use the new method names.
All third-party code must be similarly updated.
Currently, the old names remain in the code, but annotated as deprecated. I will remove the old names by committing
on or after 6th September 2018. Please subscribe to that change for notifications. I will also ping this mailing list again 1 week prior to committing the removal.
Please let me know if that timeframe does not work for you or causes problems so that I can postpone.
On systems with git and sed, porting may be as simple as
git grep -l getLocEnd | xargs sed -i 's/getLocEnd/getEndLoc/g'
git grep -l getLocStart | xargs sed -i 's/getLocStart/getBeginLoc/g'
git grep -l getStartLoc | xargs sed -i 's/getStartLoc\b/getBeginLoc/g'
The getStartLocation API is distinct and should be excluded from porting as above.
Because all before and after spellings are the same length, this has a neutral impact on the style of your code.