Project

General

Profile

Actions
Copy link

Task #55372

closed

Remove useless count function calls in stdWrap

Added by Tim Lochmüller about 10 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Could have
Assignee:
-
Category:
Performance
Target version:
-
Start date:
2014-01-27
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Remove thousands uselesse count function calls in the stdWrap. stdWrap is often used and the count calls are not needed at all.

The first is always a array so we don't check it in front of the foreach.
And the second one just check if it empty, but we don't have to count the whole array.

Regards,
Tim

Files

Download all files
Bildschirmfoto_2014-01-25_um_17.30.55.png (64.4 KB) Bildschirmfoto_2014-01-25_um_17.30.55.png Tim Lochmüller, 2014-01-27 20:17
Screen_Shot_2014-01-27_at_23.07.28.png (32.7 KB) Screen_Shot_2014-01-27_at_23.07.28.png Nice! Michiel Roos, 2014-01-29 01:13
Screen_Shot_2014-01-29_at_01.12.36.png (31.7 KB) Screen_Shot_2014-01-29_at_01.12.36.png Measurements Michiel Roos, 2014-01-29 01:14
Actions
Copy link
 #1

Updated by Gerrit Code Review about 10 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27078

Actions
Copy link
 #2

Updated by Gerrit Code Review about 10 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27078

Actions
Copy link
 #3

Updated by Gerrit Code Review about 10 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27078

Actions
Copy link
 #4

Updated by Gerrit Code Review about 10 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27078

Actions
Copy link
 #5

Updated by Gerrit Code Review about 10 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27078

Actions
Copy link
 #6

Updated by Gerrit Code Review about 10 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27078

Actions
Copy link
 #8

Updated by Michiel Roos about 10 years ago

Please ignore Screen_Shot_2014-01-27_at_23.07.28.png . . .

Actions
Copy link
 #9

Updated by Tim Lochmüller about 10 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #10

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF