2011/5/25 Hannes Magnusson <hannes.magnus...@gmail.com> > Hi all > > I'm still looking at ways to optimize PhD.. > By remove all the class constant lookups we can shave off 9-10%.. > Thats 8-9 seconds on my laptop (for a full build). > > It does however make the code slightly weirder to read, so I'm > wondering if you guys think its worth it? > I did add comments to Reader.php explaining what those seemingly > random numbers mean.. and the other constants we use are for the > namespace declarations... > > See the attached patch. > > -Hannes >
hi. how much would we gain with "real" constants (eg. define)? my biggest concern would be the code duplication, and that it would be harder to keep everything in sync. maybe we could also do some kind of preprocessing, eg replace the constants with scalar values before running the script. Tyrael
