Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:100297
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain bohwaz.net designates 5.135.163.151 as permitted sender)
Date: Thu, 24 Aug 2017 22:25:30 +1200
To: "internals@lists.php.net" <internals@lists.php.net>
Message-ID: <20170824222530.402f231a@platypus>
In-Reply-To: <def04a63-5ced-88bb-ac75-135009228d4b@gmx.de>
References: <1e398f12adbdbed6b7caf3b0af07f284@bohwaz.net>
 <CADxrgCOVu8+91THShrpGLvzqRKBr7URm69s792K2Ufi=GxoTSw@mail.gmail.com>
 <069c08bfe15b856007debab5d0d961be@bohwaz.net>
 <CADyq6sKDTp02dk816_2pqaFFRa5kEwXa=8AcjsD=UhZ7dgSmHw@mail.gmail.com>
 <e52aa112ca7aa2fa041a40b5eb2dc729@bohwaz.net>
 <CADxrgCPFdyCHch2B=Y9=GGuRq5jB2i84RdPLO5xtccsnUQddAQ@mail.gmail.com>
 <def04a63-5ced-88bb-ac75-135009228d4b@gmx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Matching PDO_SQLite features with SQLite3 extension
From: php@bohwaz.net (BohwaZ)

On Thu, 24 Aug 2017 11:55:10 +0200 / "Christoph M. Becker"
<cmbecker69@gmx.de> said :

> It seems to me this is rather simple: if a feature is uncontroversial,
> it doesn't need an RFC.  Since nobody objected against PR #2528, it
> was merged (after some weeks).  However, there have been objections
> against PR #2698, so an RFC appears to be appropriate.

Cool, that's a rule that makes sense :) maybe it should be in the
contributing guide?

Can someone give me RFC power on the wiki (my account is "bohwaz")?
Thanks.

Should I include the proposal for adding the PDOStatement::readOnly()
method?

Should it be named PDOStatement::sqliteReadOnly() or readOnly()? I have
no idea. Or should it even be PDO::sqliteIsReadOnly($statement)?

I think that adding read-only and read-write opening options for SQLite
PDO shouldn't raise any eyebrows here as it doesn't involve any new
method, so I'll do a separate patch for that. Anyone objecting?

Cheers.