https://bugzilla.wikimedia.org/show_bug.cgi?id=22018
Martin Keckeis <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|WONTFIX | --- Comment #5 from Martin Keckeis <[email protected]> 2010-01-05 13:38:20 UTC --- (In reply to comment #4) > print vs. echo and "constant" vs. 'constant' differ extremely little both in > performance and in readability. It's not the readability from the "function" itself. But the readability get slightly away when using 2 function for the same. About performance: So u mean it doesn't matter? It does MATTER! EVERY speed improvement helps wikipedia! They need not so much hardware, so they need less money! > I believe your autoloading suggestion is to > make it so that there's a 1:1 mapping from class name to file name so we can > get rid of the array in AutoLoader.php ; this could be done, but restructuring > our entire codebase is not worth it (I also don't like it, personally, but > that's not for here). Why it's not worth? At the moment u mean it's a big change? But in future it a really nice change! I've done that in our project and now you can code a lot faster, and don't have to think about classes which you have to include. > In reply to comment #2: reformatting a file *theoretically* doesn't matter, > but > reviewing the change is difficult and errors can creep in. I am using personally the new Zend Studio 7.1 which is based on Eclipse and to reformat a page you only have to press "Strg + Shift + F" -> and all is done. Reviewing is really hard right. But when it's done, it's done -> so in the future every PHP files looks the same which also brings a better readability. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug. You are on the CC list for the bug. _______________________________________________ Wikibugs-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
