[wp-meta] [Making WordPress.org] #3135: Browse Happy API: Replace require() with require_once()
Making WordPress.org
noreply at wordpress.org
Tue Sep 19 08:04:54 UTC 2017
#3135: Browse Happy API: Replace require() with require_once()
--------------------------+----------------------
Reporter: ronakganatra | Owner:
Type: enhancement | Status: closed
Priority: normal | Milestone:
Component: API | Resolution: wontfix
Keywords: has-patch |
--------------------------+----------------------
Comment (by SergeyBiryukov):
Replying to [comment:3 ronakganatra]:
> I have added this as an improvement. because it adds file every time
instead if we use require_once it only include that file once so,
efficiency will be improved.
`require_once()` is actually [http://gazehawk.com/blog/php-require-
performance/ slower than require()], because it has to check if the file
has already been included. The performance difference might be negligible,
but the function still should not be changed without a good reason.
--
Ticket URL: <https://meta.trac.wordpress.org/ticket/3135#comment:5>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org
More information about the wp-meta
mailing list