Breaking APIs for no good reason is a bad thing. (tm)

Now, I'll take some blame here for not being involved over the past couple years and having not noticed this inanity sooner, but could someone explain this cluster-f? http://svn.php.net/viewvc/php/php-src/branches/PHP_5_3/Zend/zend_API.h?r1=268281&r2=269153 This breaks the existing module API because any rational extension developer has been putting 'static' in front of ZEND_BEGIN_ARG_INFO_EX() _where appropriate_ for the better part of a decade and it does ABSOLUTELY NOTHING more than the very /minor/ optimization of cleaning up the symbol space of poorly written modules. Worse, it breaks the ability of extension developers to keep arg_info structures next to the functions they belong to while still keeping those functions in separate files. Someone give me a good reason not to revert this. -Sara