[TYPO3-core] RFC: #9283: declare marker handling functions of t3lib_parsehtml static

Ingo Renner ingo at typo3.org
Wed Sep 3 21:02:40 CEST 2008


Hi

This is an SVN patch request.

Type: Cleanup

Bugtracker references:
http://bugs.typo3.org/view.php?id=9283

Branches: trunk

Problem:
1) The marker handling functions in t3lib_parsehtml are often called 
staticaly although there're not declared static (even is the case within 
t3lib_parsehtml itself).
According to http://ilia.ws/files/ez_performance.pdf [^] this results in 
a drop of performance, other than that it produces notices or warnings 
and it is not good in general of course.

2) There's an inconsistency: although a method substituteMarkerArray() 
exists, there's no substituteSubpartArray()


Solution:
1) Declare those methods static
2) introduce the missing method substituteSubpartArray()


Notice:
There's also some cleanup in t3lib_parsehtml and tslib_content. No 
functionality has been changed except for the addition of the new method 
in t3lib_parsehtml


Ingo

-- 
Ingo Renner
TYPO3 Core Developer, Release Manager TYPO3 4.2

-------------- next part --------------
A non-text attachment was scrubbed...
Name: t3lib_parsehtml-cleanup.diff
Type: text/x-diff
Size: 23366 bytes
Desc: not available
Url : http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20080903/723bba50/attachment-0001.diff 


More information about the TYPO3-team-core mailing list