Make "callable" typehint optional

[squizlabs]

Make "callable" typehint optional

[oliverklee]

PHP 5.3 has reached its end of life on 2013-06-11 anyway: http://php.net/archive/2013.php#id2013-07-11-1

[aik099]

Some projects are so happy about, that they have set their minimum PHP version requirement to PHP 5.4 already. However fact, that PHP is quick about releasing new versions doesn't mean that amount of projects (and legacy code) still using it suddenly becomes smaller.

I think PHP_CodeSniffer should tolerate PHP version-specific code and don't enforce upgrade to next PHP version.

[gsherwood]

PHP_CodeSniffer doesn't enforce anything; it is the standard that enforces things. If you don't like the standard, change it, customise it, or make your own.

[aik099]

Code is located in Squiz.Commenting.FunctionComment sniff's processParams method, which is 306 lines long with no way for easy overriding.

Overriding that long method will prevent me from getting any improvements to that sniff in future. That's what I'm worried about.

That's actually well known problem in sniffs: too long process method with no easy way to change small bits of sniff logic without overriding long methods.