On 2023/01/16 13:38, Kamil Tekiela <tekiela...@gmail.com> wrote: > Did you create an RFC already? Or is this RFC-like discussion?
No. I followed https://wiki.php.net/rfc/howto and this is step 1. Creating the actual RFC is step 3. (I'm new to PHP and this process - so far, I've only contributed PHP code through GitHub, and I was surprised by today's sudden resistance against my code cleanup by two PHP developers, after the others welcomed by changes.) > In either case, if you are asking community to come to a decision, we need > to see some background. > Why do you want to change this? What's the benefit? For cleaner code, more readable code (less obscurity in huge amounts of undocumented #includes), more correct code, less fragile code, reduced compile times. > What's the impact on other maintainers, especially extension > maintainers? Other maintainers may need to determine which includes they really need, instead of relying on everything always already there by including one random zend_* header. Extension maintainers may see build failures because, for example, they forgot to include errno.h but used "errno". This previously worked because all PHP headers included errno.h even though there was no reason to. These build failures are bugs in those extensions, and revealing them is a good thing, even though it seems tedious at first. > Do you see any downsides to your new approach? Like any code cleanup, this can result in regressions for parts of PHP that are not covered by a test and not built with the CI, like the DTrace regression yesterday. And code cleanups can easily reveal existing bugs, which is a good thing, but those bugs can be hidden at first - e.g. failure to explicitly include "php_config.h" will (silently) disable features and break things. That may cause some temporary pain, but in the end will result in better code, though some will believe that it isn't a worthy goal or not worth the temporary pain. Derick Rethans wrote my idea "adds nothing but clutter", but I guess he should explain what bothers him; I don't comprehend this, because from my perspective, I intend to remove clutter. Dmitry Stogov said it's "just a useless permutation", but I can't understand this argument either. Max -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: https://www.php.net/unsub.php
