Commit ff133a2

committed

Generated arginfo headers: combine preprocessor conditional blocks (2)

When global constants' or class constants' availability is based on some preprocessor condition, the generated arginfo header files wrap the declarations in the preprocessor `#if` conditional blocks, one per declaration, even if they are in the same conditional block based on comments in the stub file. Instead of having multiple conditional blocks one after the other with the same condition, combine them into a single conditional block.

1 parent 6b95875 commit ff133a2Copy full SHA for ff133a2

File tree

24 files changed

+13

-811

lines changed

build
- gen_stub.php
ext
- com_dotnet
  - com_extension_arginfo.h
- curl
  - curl_arginfo.h
- dba
  - dba_arginfo.h
- gd
  - gd_arginfo.h
- intl
  - listformatter
    - listformatter_arginfo.h
  - spoofchecker
    - spoofchecker_arginfo.h
  - uchar
    - uchar_arginfo.h
- ldap
  - ldap_arginfo.h
- odbc
  - odbc_arginfo.h
- openssl
  - openssl_arginfo.h
  - openssl_pwhash_arginfo.h
- pcntl
  - pcntl_arginfo.h
- pdo_mysql
  - pdo_mysql_arginfo.h
- sockets
  - sockets_arginfo.h
- sodium
  - libsodium_arginfo.h
  - sodium_pwhash_arginfo.h
- standard
- tidy
  - tidy_arginfo.h
- xsl
  - php_xsl_arginfo.h
- zip
  - php_zip_arginfo.h
main
- main_arginfo.h

24 files changed

+13

-811

lines changed

`‎build/gen_stub.php‎`

Lines changed: 13 additions & 17 deletions

Original file line number	Diff line number	Diff line change
`@@ -2742,23 +2742,15 @@ public function getDeclaration(array $allConstInfos): string`
`2742`	`2742`	`throw new Exception("Constant " . $this->name->__toString() . " must have a @cvalue annotation");`
`2743`	`2743`	`}`
`2744`	`2744`
`2745`		`- $code = "";`
`2746`		`-`
`2747`		`- if ($this->cond) {`
`2748`		`- $code .= "#if {$this->cond}\n";`
`2749`		`- }`
	`2745`	`+ // Condition will be added by generateCodeWithConditions()`
`2750`	`2746`
`2751`	`2747`	`if ($this->name->isClassConst()) {`
`2752`		`- $code .= $this->getClassConstDeclaration($value, $allConstInfos);`
	`2748`	`+ $code = $this->getClassConstDeclaration($value, $allConstInfos);`
`2753`	`2749`	`} else {`
`2754`		`- $code .= $this->getGlobalConstDeclaration($value);`
	`2750`	`+ $code = $this->getGlobalConstDeclaration($value);`
`2755`	`2751`	`}`
`2756`	`2752`	`$code .= $this->getValueAssertion($value);`
`2757`	`2753`
`2758`		`- if ($this->cond) {`
`2759`		`- $code .= "#endif\n";`
`2760`		`- }`
`2761`		`-`
`2762`	`2754`	`return $code;`
`2763`	`2755`	`}`
`2764`	`2756`
`@@ -3553,9 +3545,11 @@ function (Name $item) {`
`3553`	`3545`	`$code .= "\tzend_register_class_alias(\"" . str_replace("\\", "\\\\", $this->alias) . "\", class_entry);\n";`
`3554`	`3546`	`}`
`3555`	`3547`
`3556`		`- foreach ($this->constInfos as $const) {`
`3557`		`- $code .= $const->getDeclaration($allConstInfos);`
`3558`		`- }`
	`3548`	`+ $code .= generateCodeWithConditions(`
	`3549`	`+ $this->constInfos,`
	`3550`	`+ '',`
	`3551`	`+ static fn ( $const ) => $const->getDeclaration($allConstInfos)`
	`3552`	`+ );`
`3559`	`3553`
`3560`	`3554`	`foreach ($this->enumCaseInfos as $enumCase) {`
`3561`	`3555`	`$code .= $enumCase->getDeclaration($allConstInfos);`
`@@ -5187,9 +5181,11 @@ static function (FuncInfo $funcInfo) use ($fileInfo, &$generatedFunctionDeclarat`
`5187`	`5181`	`$code .= "\nstatic void register_{$stubFilenameWithoutExtension}_symbols(int module_number)\n";`
`5188`	`5182`	`$code .= "{\n";`
`5189`	`5183`
`5190`		`- foreach ($fileInfo->constInfos as $constInfo) {`
`5191`		`- $code .= $constInfo->getDeclaration($allConstInfos);`
`5192`		`- }`
	`5184`	`+ $code .= generateCodeWithConditions(`
	`5185`	`+ $fileInfo->constInfos,`
	`5186`	`+ '',`
	`5187`	`+ static fn ( $constInfo ) => $constInfo->getDeclaration($allConstInfos)`
	`5188`	`+ );`
`5193`	`5189`
`5194`	`5190`	`if ($attributeInitializationCode !== "" && $fileInfo->constInfos) {`
`5195`	`5191`	`$code .= "\n";`

`‎ext/com_dotnet/com_extension_arginfo.h‎`

Lines changed: 0 additions & 2 deletions

Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

`‎ext/curl/curl_arginfo.h‎`

Lines changed: 0 additions & 158 deletions

Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

`‎ext/dba/dba_arginfo.h‎`

Lines changed: 0 additions & 2 deletions

Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

`‎ext/gd/gd_arginfo.h‎`

Lines changed: 0 additions & 18 deletions

Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

`‎ext/intl/listformatter/listformatter_arginfo.h‎`

Lines changed: 0 additions & 4 deletions

Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

`‎ext/intl/spoofchecker/spoofchecker_arginfo.h‎`

Lines changed: 0 additions & 12 deletions

Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

`‎ext/intl/uchar/uchar_arginfo.h‎`

Lines changed: 0 additions & 4 deletions

Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit ff133a2

File tree

24 files changed

24 files changed

`‎build/gen_stub.php‎`

`‎ext/com_dotnet/com_extension_arginfo.h‎`

`‎ext/curl/curl_arginfo.h‎`

`‎ext/dba/dba_arginfo.h‎`

`‎ext/gd/gd_arginfo.h‎`

`‎ext/intl/listformatter/listformatter_arginfo.h‎`

`‎ext/intl/spoofchecker/spoofchecker_arginfo.h‎`

`‎ext/intl/uchar/uchar_arginfo.h‎`

0 commit comments