Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:103487
MIME-Version: 1.0
References: <BY1PR02MB12606B0B2C6878B6F73D151BBFD40@BY1PR02MB1260.namprd02.prod.outlook.com>
 <CAF+90c_6Vwz+GguSfCGQw5ncmcSBGXnXP4yo1KymLDMEaL19Hw@mail.gmail.com>
 <BY1PR02MB1260D6A74881B45987CC462EBFD40@BY1PR02MB1260.namprd02.prod.outlook.com>
 <CAEZPtU6YK0O4Q5Pud3OeuGnv6LuP_tKAL1=3W-4pu+S48sd82w@mail.gmail.com>
In-Reply-To: <CAEZPtU6YK0O4Q5Pud3OeuGnv6LuP_tKAL1=3W-4pu+S48sd82w@mail.gmail.com>
Date: Fri, 23 Nov 2018 10:23:01 -0600
Message-ID: <CAESVnVoD+6J3RuSZpwO+vK19821thw4kQgZDwLRFXhM775gr6Q@mail.gmail.com>
To: Pierre Joye <pierre.php@gmail.com>
Cc: Dmitry Stogov <dmitry@zend.com>, Nikita Popov <nikita.ppv@gmail.com>, 
	PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000004c9a79057b57665c"
Subject: Re: [PHP-DEV] [mini-RFC] Disable opcache per script using "declare(cache=0)"
From: pollita@php.net (Sara Golemon)

--0000000000004c9a79057b57665c
Content-Type: text/plain; charset="UTF-8"

On Fri, Nov 23, 2018 at 8:36 AM Pierre Joye <pierre.php@gmail.com> wrote:

> > Especially, the main preload.php, usually should be marked, to disable
> its
> > caching.
> >
>
> should this specific case not be done automatically then?
>
> I'm with Pierre on this.  During the preload phase, we should probably
have implicit caching disabled and only cache things brought in via
opcache_compile_file().
Outside of the preload pragma, I'd worry that this declare would be
mistakenly overused in most cases.  Question though... who wins if a file
is explicitly loaded with opcache_compile_file *and* it has the "don't
cache me, bro" pragma?  I'd expect the file wouldn't be cached in that case.

That said, I'm of the "here's your gun, try not to shoot yourself" school
on most things, and I see no harm in this when used judiciously, so no
objections from me.

-Sara

--0000000000004c9a79057b57665c--