Page MenuHomePhabricator

Improve the robustness of the "alias function" linter rule
ClosedPublic

Authored by joshuaspence on May 19 2015, 8:12 AM.
Tags
None
Referenced Files
F13216219: D12921.diff
Fri, May 17, 11:11 PM
F13210052: D12921.id31084.diff
Fri, May 17, 3:37 AM
F13195160: D12921.diff
Sun, May 12, 10:11 PM
F13177955: D12921.diff
Wed, May 8, 8:04 PM
Unknown Object (File)
Thu, Apr 25, 1:04 AM
Unknown Object (File)
Apr 5 2024, 3:41 PM
Unknown Object (File)
Apr 1 2024, 9:03 AM
Unknown Object (File)
Mar 5 2024, 2:21 AM
Subscribers

Details

Summary

Improve the ArcanistXHPASTLinter::LINT_ALIAS_FUNCTION linter rule. Currently this rule does not correctly handle alias functions which are not strictly lowercase.

Test Plan

Added a test case.

Diff Detail

Repository
rARC Arcanist
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6101
Build 6121: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Improve the robustness of the "alias function" linter rule.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/lint/linter/ArcanistXHPASTLinter.php
4120

Sort of irrelevant since these should never be utf8, but prefer phutil_utf8_strtolower() for consistency?

This revision is now accepted and ready to land.May 19 2015, 1:34 PM
joshuaspence edited edge metadata.

Use phutil_utf8_strtolower

This revision was automatically updated to reflect the committed changes.