[wp-hackers] Adding wp-includes/class folder

Peter Westwood peter.westwood at ftwr.co.uk
Fri Aug 31 13:13:41 GMT 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Jacob Santos wrote:
> I have patches, or a set patches for XMLRPC split. I'm planning on
> making a enhancement bug report, if most people are on board.
> 
> I took a look at the Tree implementation and thought that it was very
> interesting. The push for PHP 5 advantage is a longer goal. Lazy? Yes,
> but there is a reason it exists, and I forgot what the reason is, but
> okay. I don't use it, but I would like too.
> 
> I suggestion a convention that either we have all classes split up or we
> don't. I think dependency classes are great candidates for splits,
> because it allows me to focus on including just the file that I need. If
> included at the top, people can know which classes you are using below
> (mostly, my patches still use bootstrap that include everything, in
> order of the dependency. wp-includes/class/IXR.php includes everything
> in the IXR class namespace.
> 

For things like IXR / POP3 support / PHPMailer - splitting the classes
out is a really bad plan IMHO - they are external libraries that we
include and we need an easy way of diffing between our version (with
some patches) and the upstream version so as to make it easy to pull in
updates.

cheers

westi
- --
Peter Westwood
http://blog.ftwr.co.uk
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFG2BQFVPRdzag0AcURAiGNAKCQP7Jf5CnHGL8qysv1NjiHxmAOqgCgsBfm
rk06bSrF2f6hRlGl5YHGBjo=
=yP8m
-----END PGP SIGNATURE-----


More information about the wp-hackers mailing list