Page MenuHomePhabricator

Skip anon functions in symbol generation script
ClosedPublic

Authored by Firehed on Jan 24 2014, 12:58 AM.
Tags
None
Referenced Files
F18078764: D8054.diff
Mon, Aug 4, 8:23 PM
Unknown Object (File)
Jun 9 2025, 1:53 AM
Unknown Object (File)
Jun 6 2025, 11:30 PM
Unknown Object (File)
Jun 4 2025, 6:12 AM
Unknown Object (File)
Jun 3 2025, 7:52 PM
Unknown Object (File)
May 7 2025, 2:39 AM
Unknown Object (File)
Apr 27 2025, 7:51 AM
Unknown Object (File)
Apr 26 2025, 9:37 AM
Tokens
"Mountain of Wealth" token, awarded by raguiar.

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP14f070a0af79: Skip anon functions in symbol generation script
Summary

Filters closures out of symbol generator script, per @epriestley's
comment in T4334

Test Plan

Before:

eric@eric-dev ~/phabricator/scripts/symbols: echo 'closure.php' | ./generate_php_symbols.php 
  function php  /closure.php
 d function php 10 /closure.php
  function php  /closure.php
 a class php 3 /closure.php
a b method php 4 /closure.php

After:

eric@eric-dev ~/phabricator/scripts/symbols: echo 'closure.php' | ./generate_php_symbols.php 
 d function php 10 /closure.php
 a class php 3 /closure.php
a b method php 4 /closure.php

eric@eric-dev ~/phabricator/scripts/symbols: cat closure.php

<?php

class a {
  function b() {
    $c = function() { return 1; };
    $c();
  }
}

function d() {
  return 2;
}
$e = function() {
  return 3;
};

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Thanks!

StdClass

I think if this is spelled stdClass we recognize it (maybe).

scripts/symbols/generate_php_symbols.php
38

For completeness this could have a strlen() check, but you can't name a function "0" anyway I suppose.

Closed by commit rP14f070a0af79 (authored by @Firehed, committed by @epriestley).