Skip to content

RFC: Deprecate remains of string evaluated code assertions #11671

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jul 13, 2023

Conversation

Girgias
Copy link
Member

@Girgias Girgias commented Jul 11, 2023

Copy link
Member

@nielsdos nielsdos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some cases where indentation of the assert call was forgotten in the tests.
For some, like the syntax serializations, I guess that's fine to prevent lots of extra changes due to indentation.

@Girgias
Copy link
Member Author

Girgias commented Jul 11, 2023

There are some cases where indentation of the assert call was forgotten in the tests. For some, like the syntax serializations, I guess that's fine to prevent lots of extra changes due to indentation.

Basically yes, I can fix the indentation of some of the simple ones but I didn't want the diff to be huge!

Copy link
Member

@bukka bukka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good in general to me. Just some minor things to address.

@@ -124,6 +124,12 @@ PHP 8.3 UPGRADE NOTES
. The MT_RAND_PHP Mt19937 variant is deprecated.
RFC: https://wiki.php.net/rfc/deprecations_php_8_3#mt_rand_php

- Standard:
. The assert_option() function is now deprecated.
Copy link
Contributor

@andypost andypost Jul 14, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it should be assert_options()

arnaud-lb added a commit to arnaud-lb/php-src that referenced this pull request Jul 16, 2023
* up/master: (571 commits)
  Expose time spent collecting cycles in gc_status() (php#11523)
  Warn when fpm socket was not registered on the expected path
  Implement DOMElement::id
  Fix ?
  Implement DOMParentNode::replaceChildren()
  Implement DOMElement::className
  RFC: Deprecate remains of string evaluated code assertions (php#11671)
  Prevent decimal int precision loss in number_format()
  Implement DOMNode::getRootNode()
  Implement DOMElement::getAttributeNames()
  Refactor dom_node_node_name_read() to avoid double allocation
  Handle fragments consisting out of multiple children without a single root correctly
  Avoid allocations in DOMElement::getAttribute()
  Avoid string allocation in dom_get_dom1_attribute() for as long as possible
  Fix use-of-uninitialized-value when calling php_posix_stream_get_fd (php#11694)
  Reorder list construction in the function php_intpow10 (php#11683)
  proc_open: Use posix_spawn(3) interface on systems where it is profitable
  zend_gdb disable gdb detection for FreeBSD < 11.
  Fix iface const visibility variance check
  Fix missing iface class const inheritance type check
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants