Create your own GitHub profile
Sign up for your own profile on GitHub, the best place to host code, manage projects, and build software alongside 50 million developers.
Sign up
Pinned
3,091 contributions in the last year
Contribution activity
August 2020
Created a pull request in php/php-src that received 3 comments
Check for duplicate names in gen_stub.php
With named arguments in php 8.0, it's important that php's modules or PECL extensions using gen_stub.php don't generate functions with duplicate na…
+6
−0
•
3
comments
- Update PHP-Parser from 4.3.0 to 4.9.0
- Fix more basic function stubs
- Colorize --show-diff, only on stdout
- Rename some standard array function parameters from $arr/$arg to $array
- Add `run-tests.php --context [n]` option.
- Use short array values in run-tests.php
- Make diff section contents red(-)/green(+) in run-tests.php
- Support NO_COLOR environment variable in run-tests.php
- Proposal: Custom callback to dump results in `php -a`
- Proposal: Make match expression a constant expression (when parts are constant)
- [skip ci] Document Phar metadata unserialization change
- [Proposal] Bigint shorthand (123n) for GMP objects
- [Proposal] Warn about the loss of precision in binary/octal/hexadecimal literals
- Fix grammar/typo nits in docs
Created an issue in microsoft/tolerant-php-parser that received 4 comments
Publish a 0.0.21 release with support for lexing changes in php 8.0 (T_NAME_*)?
See #331 - People using php 8.0 dev won't be able to use this library until https://wiki.php.net/rfc/namespaced_names_as_token is supported. The fe…
4
comments
- Dead code detection should treat preg_replace[_callback] as a definition of $count
- Unused variable detection should treat array_splice($x['offset'], ...) as making $x a reference
- Plugin idea: Warn about `$string >= int|float` or `not-quite-numeric-string cmp any`
- Warn about unused variables due to switch fallthroughs?
- Support multiple declare() directives in the tolerant-php-parser polyfill
- Suggest named parameters with similar names or similar types to incorrect named argument
- Mention included_extension_subset if configured for "Did you mean to configure a stub with"
- Validate entries when reading from .phan/config.php as well
- Add non-empty-array type for array functions such as array_rand()
- Raise severity of PhanTypeArraySuspicious for non-ArrayAccess objects, scalars other than false/null
- Support `@no-named-arguments` on classlikes and functionlikes.
- Exclude known version control directories (.git, .svn, etc) by default?
- Warn about $x being an undefined variable/offset in $x?->val, $y['offset']->method(), etc.
- seal-properties/phan-forbid-undeclared-magic-properties should apply to dynamic properties
- Warn about forward compatibility of elements named fn, match, etc.
- Extract parameter defaults from php 8.0 stub files and bundle with phan for php7 error messages
- Update tooling to prefer extracting real types from php-src '*.stub.php' files instead
- Fix incorrect php 8.0 types for curl_init (etc) in FunctionSignatureMapReal.php and FunctionSignatureMap_php80_delta.php
- Work around realpath platform issues on Windows
- Warn about $this?->propOrMethod, $this ?? default, etc
- Warn about $errcontext in custom error handlers
- Warn about incompatible magic method return types if real type is declared
- Warn about define() with case insensitive flag, it is no longer supported in php8
- Idea: Warn about parameter name choices in internal functions, as well as in php code
- Suggest directory_list entries for undeclared element warnings based on vendor/composer/installed.json
- Some issues not shown.
- Look into phpunit test failures on https://github.com/runkit7/SimpleStaticMock with opcache
- Document that opcache optimizations should be disabled with runkit7
- Sync php.net documentation with 4.0.0dev parameter names
- Support PHP 8.0 named parameters
- Copy attribute groups when copying php 8.0 functionlikes
- Make Reflection arginfo consistent for php 8
- Cannot suppress ImpureByReferenceAssignment
- Type of $this can be affected by call to include()
- Psalter adds "@return array-key[]" instead of "@return int[]|string[]" for MissingReturnType
- Should ParamNameMismatch be invoked for magic methods (__unserialize, __call, __callStatic)?
- TypeComparisonResult of UnionTypeComparator.php may depend on order of union type list?
- Support additional ways to mark parameters as unused that other tools can use?