Type casts should be used instead of calls to the *val functions as function calls impose additional overhead.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Commits
- rARC578f540a92b1: Add a linter rule for `*val` functions
Added unit tests.
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Branch
- master
- Lint
Lint Passed - Unit
Test Failures - Build Status
Buildable 9124 Build 10762: Run Core Tests Build 10761: arc lint + arc unit
Event Timeline
Comment Actions
I think "for consistency" is a stronger motivator here than overhead, but rule itself is fine.