Skip to content

Instantly share code, notes, and snippets.

@salathe
Created July 7, 2011 15:03
Show Gist options
  • Save salathe/1069702 to your computer and use it in GitHub Desktop.
Save salathe/1069702 to your computer and use it in GitHub Desktop.
Deprecate __autoload() -- mixed responses.
[15:56] <@salathe> do you think anyone would mind if we deprecated __autoload ?
[15:56] <@philip__> why?
[15:57] <@Derick> ?!
[15:57] <cyth> because __autoload sucks compared to spl_register
[15:57] <@Derick> it works
[15:57] <@Derick> print also sucks over echo
[15:58] <cyth> yeah but it's kind of bad practice
[15:58] <cyth> you could be clobering third party autoloaders
[15:58] <@Derick> not every project cares
[15:58] <@bjori> not every project uses third party autoloaders
[15:58] <cyth> I wouldn't really call print bad practice in that regards though.
[15:58] <cyth> bjori, until they do
[15:58] <cyth> then they are wondering wtf code won't work.
[15:58] <@ekneuss> I wouldn't mind deprecating __autoload, softly, i.e. in the docs
[15:59] <@bjori> my `shell scripts` in php use __autoload
[15:59] <@Derick> this definitely goes into the category of "why the fuck are you messing with this?"
[15:59] <cyth> using spl_register is just as easy
[15:59] <cyth> and you don't trap yourself later.
[15:59] <@ekneuss> well, that's usually what deprecation is for, replacing old stuff with new and better
[15:59] <@bjori> using singletons are also bad practice
[16:00] <@Derick> ugh
[16:00] <@philip__> feel free to update the autoload docs to refer to spl_register as a preferred alternative
[16:00] <@Derick> don't be stupid
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment