symfony/symfony — comment on commit 282d3ae1d8a7fb73e65553b0c016f4c1369e34bf
File: src/Symfony/Component/HttpFoundation/Session/Session.php line 307
By drak:

I am not entirely convinced about this because close() is only used in unit tests. Given this method dosn't do anything now I think it should be removed. I understand about keeping BC, but things **should** break on an occasion like this to allow refactoring. That is far more reliable than code continuing to work but in a now unexpected way. Overall, since it only would break tests, I think it should be removed.