[TYPO3-core] RFC #9285: Remove all @ in front of file_exists() calls

Dmitry Dulepov [typo3] dmitry at typo3.org
Thu Sep 4 07:53:44 CEST 2008


Hi!

Ingo Renner wrote:
> Problem:
> The use of the @ character in front of functions to surpress warnings 
> costs a lot of performance, and in case of file_exists() is not needed 
> anymore since a bug with this function throwing warnings has been fixed 
> in PHP in 2003 already.

Actually, it does not seem any performance problem with @ character. I attach here test file and the screenshot with profiling data. So the change does not make much sense.

As you can see from the graph, using @ actually is faster. I was able to reproduce this result 100%: a small 3% increase in speed when using @.

-- 
Dmitry Dulepov
TYPO3 Core team
My TYPO3 book: http://www.packtpub.com/typo3-extension-development/book
In the blog: http://typo3bloke.net/pages/book-reviews/presentation-zen-by-garr-reynolds/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: profiling.png
Type: image/png
Size: 54215 bytes
Desc: not available
Url : http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20080904/4cc31a21/attachment-0001.png 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20080904/4cc31a21/attachment-0001.htm 


More information about the TYPO3-team-core mailing list