Page MenuHomePhabricator

Skip anon functions in symbol generation script
ClosedPublic

Authored by Firehed on Jan 24 2014, 12:58 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 12, 12:44 AM
Unknown Object (File)
Tue, Mar 12, 12:44 AM
Unknown Object (File)
Tue, Mar 12, 12:44 AM
Unknown Object (File)
Feb 20 2024, 1:29 AM
Unknown Object (File)
Feb 16 2024, 11:46 AM
Unknown Object (File)
Feb 9 2024, 12:44 AM
Unknown Object (File)
Feb 8 2024, 4:55 PM
Unknown Object (File)
Feb 4 2024, 2:52 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).