? Success

User tests: Successful: Unsuccessful:

avatar betweenbrain
betweenbrain
28 Mar 2014

The current code forJDocumentRendererHead/render uses an output buffer to return the result of a single echo statement. While very cool, I don't belive there is any advantage to this other than demonstrating excellent usage of output buffering.
With this change, my tests shows that we can expect the head portion of the page to render up to 0.000009 seconds faster while consuming up to 28,184 bytes less memory ;-) It also reduces the method to a single return statement.

Tracker item http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_id=8103&tracker_item_id=33534

avatar betweenbrain betweenbrain - open - 28 Mar 2014
avatar betweenbrain betweenbrain - change - 28 Mar 2014
Title
Micro-optimization of JDocumentRendererHead/render
[#33534] Micro-optimization of JDocumentRendererHead/render
avatar dongilbert
dongilbert - comment - 31 Mar 2014

:+1:

The fetchHead method that it's buffering doesn't have output anything, meaning there's nothing to buffer, so it's actually useless.

This PR is good to merge. Thanks Matt!

avatar betweenbrain
betweenbrain - comment - 31 Mar 2014

Thanks Don!

avatar betweenbrain betweenbrain - close - 6 Apr 2014
avatar betweenbrain betweenbrain - change - 6 Apr 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-04-06 13:34:35
Labels
avatar betweenbrain betweenbrain - close - 6 Apr 2014
avatar Bakual Bakual - reference | 6e54d05 - 12 May 14
avatar betweenbrain betweenbrain - head_ref_deleted - 16 Sep 2014

Add a Comment

Login with GitHub to post a comment