Ilia Alshanetsky wrote: > Jay, > > I've noticed that you're still using the ini parser in this new code, > would it not be easier and allow for greater code re-use to use the CSV > database and the underlying fgetcsv code? > > Ilia >
Hi Ilia, long time no irc... It probably would be a lot easier to go with the CSV parser as far as code maintenance is concerned, but I think the wtf factor may make it more annoying than it's worth. I can already picture bug reports saying "my browscap.ini file isn't being read any more, what gives". I'm sure it would be no where close to as bad as register_globals, but still, it's some unnecessary BC breakage that will probably be more trouble than it's worth. This is a x.1 release, so I guess this would be the time to do a semi-major change like this, but I personally don't think it's warranted from the users perspective. Just an unnecessary (albeit quite minor) hurdle to upgrading to 5.1, imho. J -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
