Page MenuHomePhabricator

Add "random_int" and "random_bytes" as builtins so the linter stops complaining about them
ClosedPublic

Authored by epriestley on Feb 20 2019, 10:59 PM.
Tags
None
Referenced Files
F15476168: D20202.id48237.diff
Mon, Apr 7, 4:34 AM
F15474906: D20202.diff
Sun, Apr 6, 1:29 PM
F15472876: D20202.diff
Sat, Apr 5, 5:07 PM
F15472575: D20202.id48237.diff
Sat, Apr 5, 2:49 PM
F15429227: D20202.id48237.diff
Mon, Mar 24, 1:13 AM
F15409503: D20202.id48237.diff
Wed, Mar 19, 4:37 AM
F15377260: D20202.id48237.diff
Thu, Mar 13, 8:29 AM
F15331504: D20202.id48237.diff
Mar 7 2025, 10:43 AM
Subscribers
None

Details

Summary

There's a "right" way to do this by regenerating this file from PHPCompatInfo, but that requires installing composer and a whole lot of legwork. For now, I think we only actually care about these two functions, so I just added them manually.

Test Plan

On PHP7, ran arc lint on Filesystem.php and got no warnings. This change should also stop us from getting warnings on earlier versions of PHP.

Diff Detail

Repository
rPHU libphutil
Branch
builtins1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22093
Build 30180: Run Core Tests
Build 30179: arc lint + arc unit