sqlite3 single thread #110
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Remove unused defined CPP macros in fpm SAPI - PHP_FPM_SYSTEMD - PHP_FPM_USER - PHP_FPM_GROUP * [skip ci] Update UPGRADING.INTERNALS
This reverts commit 1b47fd0. I (@youkidearitai) join to CODEOWNERS for mbstring.
- All arguments quoted - PHP_VERSION, PHP_VERSION_ID, PHP_LDFLAGS are used only in templates with @...@ placeholders - These are not used in generated Makefile neither in templates: abs_builddir, abs_srcdir, DEBUG_CFLAGS - These are used only in generated Makefile: EXTRA_LDFLAGS, EXTRA_LDFLAGS_PROGRAM, ZEND_EXTRA_LIBS, INCLUDES, EXTRA_INCLUDES, INSTALL_IT, NATIVE_RPATHS
mainly for scenarios when pcntl_fork/pcntl_exec are involved so when the latter is executed, we avoid unwarranted effects with the file descriptors, instead the socket will be closed on success. close phpGH-14606
* Update include declaration in Dom css selectors May fix php@88da914#r143708340 * Remove pointless guards * Fix Windows build * Use relative path
- Argument quoted - CFLAGS variable changed into normal shell variable - GDLIB_CFLAGS and GDLIB_LIBS are not used in the generated Makefile
- Arguments quoted - Redundant comments removed (some basic help info is in the build/php.m4) - APXS variable as such isn't used in the generated Makefile, the path to the apxs tool is inserted during the configure step directly
- Argument quoted - ODBC_INCDIR and ODBC_LIBDIR are not used in the generated Makefile
* PHP-8.3: Use pattern tags for NDBM ext/dba/tests (php#14755)
- Arguments quoted - Duplicate redundant prefix variable substitution removed
This is just a sync and alignment with Autoconf documentation style
order of checks in configure.ac ("Standard configure.ac Layout"):
https://www.gnu.org/software/autoconf/manual/autoconf-2.72/autoconf.html#Autoconf-Input-Layout
There was also redundant backslash escaping.
The PHP_ARG_ENABLE and PHP_ARG_WITH 2nd argument is the check message in the configure log output.
Symbol was added when PHP-8.3 JIT engine was used with the IR JIT and is now unused and redundant.
see https://wiki.php.net/rfc/deprecated_attribute Co-authored-by: Tim Düsterhus <[email protected]> Co-authored-by: Ilija Tovilo <[email protected]>
The exact same code already exists above this.
opcode cannot be ZEND_POST_INC because of the if check above.
prop_info is NULL in this branch.
- over-quoted arguments reduced - AS_VAR_IF used - php_cv_var_PS_STRINGS cache variable name used instead of cli_cv_* - Macro help text synced according to empty definition
- Synced CS: AS_VAR_IF style checks and M4 macro arguments quoted - Error message normalized without checking other errors appended (once more dependencies were required and additional error messages were appended)
not been updated since php 4 whereas 2.2 is available even in Win2000.
M4 interprets the dnl in this combination of m4_ifnblank as part of the preceding text so the [] can be used to avoid this issue.
* PHP-8.3: Fix bug #55639: Digest autentication dont work
* zend build making sigjmp_buf and api check as mandatory. all unixes support it since long time, the few which don't do not meet the requirements to build php anyway (minix, dietlibc, ...).
It's indeed possible this is NULL. When you create a new text-like node in libxml and pass NULL as content, you do get NULL in the content field instead of the empty string. You can hit this by creating DOMText or DOMComment directly and not passing any argument. This could also be created internally. We refactor the code such that this detail is hidden and we add a test to check that it correctly throws an exception.
This is mostly about mentioning that VLAs which are a required part of C99 conforming implementations (although made optional in the C11 standard) *must* *not* be used in php-src.
* PHP-8.3: Revert "Skip bug45161.phpt on Windows"
Use our own string builder instead of using libxml's and then having to
copy over.
For the following test:
```
$dom = Dom\HTMLDocument::createEmpty();
$root = $dom->appendChild($dom->createElement('root'));
$root->append('abc', 'def', 'ghi');
$f = $root->firstChild;
for ($i = 0; $i < 1000000; $i++)
$f->wholeText;
```
The following results were obtained on an i7-4790:
```
Benchmark 1: ./sapi/cli/php x.php
Time (mean ± σ): 57.2 ms ± 2.3 ms [User: 53.2 ms, System: 3.4 ms]
Range (min … max): 54.7 ms … 69.3 ms 52 runs
Benchmark 2: ./sapi/cli/php_old x.php
Time (mean ± σ): 89.4 ms ± 3.4 ms [User: 85.6 ms, System: 3.0 ms]
Range (min … max): 86.1 ms … 105.8 ms 32 runs
Summary
./sapi/cli/php x.php ran
1.56 ± 0.09 times faster than ./sapi/cli/php_old x.php
```
As expected, ArrayObject is cursed
Follow-up for 2d029ef
see https://wiki.php.net/rfc/correctly_name_the_rounding_mode_and_make_it_an_enum Co-authored-by: Saki Takamachi <[email protected]> Co-authored-by: Niels Dossche <[email protected]>
This introduces a new helper php_dom_create_nullable_object() that does the NULL check and puts NULL in return_value. Otherwise it runs php_dom_create_object(). This deduplicates a bit of code.
This adds notice in the UPGRADING.INTERNALS file and removes redundant undefinition from Windows config header. Follow-up of phpGH-14942.
The zend_object.properties HashTable needs to be built just in time by calling rebuild_object_properties() on the object before accessing it. Normally this is done automatically in zend_std_get_properties(), but we do it manually in a few places. In this change I introduce an inline variant of zend_std_build_properties(), and refactor these places to use it instead of calling rebuild_object_properties() manually. rebuild_object_properties() renamed as rebuild_object_properties_internal(), to enforce usage of zend_std_get_properties() or zend_std_build_properties_ex(). Closes phpGH-14996
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.