HomePhabricator

Strip everything before the first pf token

Authored by Arlolra on Jan 24 2018, 6:44 PM.
This commit no longer exists in the repository. It may have been part of a branch which was deleted.This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.

Description

Strip everything before the first pf token

In this isolate case,
{{safe<includeonly />subst:#if:{{{1}}}}}
only "subst:" is part of the same token as the prefix so
normalizeTarget doesn't take care of remove the "safesubst:"

The fix here is from noting that we wouldn't be at this place in the
code if the target didn't normalize a canonical function name, so the
contents before the prefix can be safely discarded.

This is another patch along the line of 5608e83 and bcd5572

Bug: T185643
Change-Id: I7002804fcf5305ffb6c88d93c201d9e972f9037f

Details

Committed
ArlolraJan 29 2018, 7:53 PM
ChangeId
I7002804fcf5305ffb6c88d93c201d9e972f9037f

Commit No Longer Exists

This commit no longer exists in the repository.