Trying to reduce file_exists() calls at bootstrap, testing issue [#1978522]

Removes the drupal_load() static cache: it is useless because it superseeds both APC and PHP core, which already provide this. It also free some memory (a huge amount of memory when you reach 200+ modules).
Removes some useless file_exists() calls in some edge cases, those can reache the include_once statement safely anyway: better to have a warning that tells a file does not exist than skipping it silentely.

Comment	File	Size	Author
#1	1978522-1-test.patch	1.4 KB	pounard
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

pounard

French

CreditAttribution: pounard commented 24 April 2013 at 10:15

Status:

Active

» Needs review

File	Size
1978522-1-test.patch	1.4 KB

Comment #2

24 April 2013 at 11:10

Status:

Needs review

» Needs work

The last submitted patch, 1978522-1-test.patch, failed testing.

Comment #3

pounard

French

CreditAttribution: pounard commented 24 April 2013 at 11:32

Very interesting testbot log, since all there is are warnings and no fatal errors. All happen only during the upgrade tests. This mean we can at least get rid of these checks and this infamous static cache (which happens to be useless because PHP will do it for us).