Page MenuHomePhabricator

[libTooling] Change `addInclude` to use expansion locs.

Authored by ymandel on Dec 22 2020, 7:06 AM.



This patch changes the default range used to anchor the include insertion to use
an expansion loc. This ensures that the location is valid, when the user relies
on the default range.

Driveby: extend a FIXME for a problem that was emphasized by this change; fix some spellings.

Diff Detail

Event Timeline

ymandel requested review of this revision.Dec 22 2020, 7:06 AM
ymandel created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptDec 22 2020, 7:06 AM

Gentle ping...

tdl-g accepted this revision.Jan 14 2021, 6:42 AM
This revision is now accepted and ready to land.Jan 14 2021, 6:42 AM
This revision was landed with ongoing or failed builds.Jan 14 2021, 7:13 PM
This revision was automatically updated to reflect the committed changes.