HomePhabricator

Additional cleanup following the previous patch

Authored by ssastry on Feb 20 2018, 12:18 AM.

Description

Additional cleanup following the previous patch

Since tokens enter the buildExpandedAttrs method only if
the token had expandable attributes, there is no need to track
modification state. For the small fraction of tokens that go
through it, treat them all as modified. This leads to simpler
code.

Change-Id: Icf0c53d13074ff740716dc2973d8462a287a19a6

Details

Committed
ssastryFeb 27 2018, 7:26 PM
Parents
rGPAR0c3abce08864: Don't process token attributes unnecessarily
Branches
Unknown
Tags
Unknown
References
refs/changes/34/412834/9
ChangeId
Icf0c53d13074ff740716dc2973d8462a287a19a6