@Dan Ackroyd <dan...@basereality.com> > That's an aesthetic choice, which not everyone will agree with. >
Yes, I freely state that in the RFC. Everyone is free to vote how they like. > > "There shouldn't be any backwards incompatible changes" > > It apparently breaks 25 tests that compare the output of var_export > with an expected version. Userland users who have similar tests will > experience similar breaks. > > A test that compares var_export output is a pretty broken test, in my opinion. This is similar to a test that would use the value of a constant to test that the constant works. You're testing PHP code you shouldn't care if it uses the long form array construct or short form. Just test that it gives you an array. With that said, I will update the RFC to make note of these tests and I will fix them should the RFC be voted in. > Is there any reason to do this change other than aesthetic choice? > > Just because it is aesthetic is not actually an argument against. Why wouldn't you want to improve the aesthetics of PHP? Is it only an aesthetic choice, however? Not entirely. I also state that it is easier to read and type (saves space) and is more inline with JSON notation which is a lot more ubiquitous these days. These are all pros. I see no cons in that change.
