Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:98818 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 68347 invoked from network); 18 Apr 2017 16:58:11 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 18 Apr 2017 16:58:11 -0000 Authentication-Results: pb1.pair.com smtp.mail=adam.baratz@gmail.com; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=adambaratz@php.net; sender-id=unknown Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.220.175 as permitted sender) X-PHP-List-Original-Sender: adam.baratz@gmail.com X-Host-Fingerprint: 209.85.220.175 mail-qk0-f175.google.com Received: from [209.85.220.175] ([209.85.220.175:36029] helo=mail-qk0-f175.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 22/9D-61625-D9546F85 for ; Tue, 18 Apr 2017 12:58:07 -0400 Received: by mail-qk0-f175.google.com with SMTP id d131so135679287qkc.3 for ; Tue, 18 Apr 2017 09:58:05 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=NYGL+jnEcpsmtua/lpAQThx2I+mEOf1/slq9jnv4ze8=; b=STDLnmKJ3hvLAd8Eaz6nP8aMT7Oj+VDc/jcI5EdNJo0UCc9VfUKt7bibbpA/7KlBJk 1ylhFnn0Y/eoYkc287nGyCch5LGHdBSDhqjCIBGSwK/b+rBRnllJgFyTJY1+ItPoHS7i k9yuu8VlUjsNWvgi+xAyRDHPBCrKXIdhJ0DbpqYvqlYDC1/AyLOi1Adtsd1IkUy5grkq OclApJLqBvS41Ah56pgchP644C93ehg0vyOkVDu0AqsRgefA7qBIygR8BnkoWLrSflJM 2GKMAG2CgcNHSZGWZuLcyUUMnpay3m8Ez86nJgq6E0IMvseEncyr5iv8dSVQzX12Us84 U42A== X-Gm-Message-State: AN3rC/7XwP0yHsPAjk9FBPkQ0twSYXpAmO7tY77OOXYK61R6gYcwjBd7 i8CHjqozSM43nLxguDg= X-Received: by 10.55.158.215 with SMTP id h206mr13269938qke.154.1492534683023; Tue, 18 Apr 2017 09:58:03 -0700 (PDT) Received: from mail-qt0-f174.google.com (mail-qt0-f174.google.com. [209.85.216.174]) by smtp.gmail.com with ESMTPSA id n5sm6998067qtd.7.2017.04.18.09.58.02 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 18 Apr 2017 09:58:02 -0700 (PDT) Received: by mail-qt0-f174.google.com with SMTP id c45so130396012qtb.1 for ; Tue, 18 Apr 2017 09:58:02 -0700 (PDT) X-Received: by 10.200.37.208 with SMTP id f16mr14095915qtf.118.1492534682207; Tue, 18 Apr 2017 09:58:02 -0700 (PDT) MIME-Version: 1.0 Received: by 10.12.135.212 with HTTP; Tue, 18 Apr 2017 09:58:01 -0700 (PDT) In-Reply-To: <86e4681f-9030-80fc-896b-56255c4b84c7@beccati.com> References: <5e3e3e9b-f5b9-a19e-c4cb-6af0f5985d94@beccati.com> <8923084a-a357-91e9-44b5-52a4af0a5df1@beccati.com> <86e4681f-9030-80fc-896b-56255c4b84c7@beccati.com> Date: Tue, 18 Apr 2017 12:58:01 -0400 X-Gmail-Original-Message-ID: Message-ID: To: Matteo Beccati Cc: "internals@lists.php.net" , Andrea Faulds Content-Type: multipart/alternative; boundary=001a113e64a68c57ee054d73cfc5 Subject: Re: [PHP-DEV] [RFC] PDO Float Type From: adambaratz@php.net (Adam Baratz) --001a113e64a68c57ee054d73cfc5 Content-Type: text/plain; charset=UTF-8 > > > I'd be most comfortable with this approach. I'd worry about adding a > > PDO::PARAM_NUMERIC type without investigating how it needs to function > for > > each DB, which feels like scope creep. If it starts off as an alias for > > PDO::PARAM_STR, there could be issues updating it to work correctly, > > especially if the right design involves modeling the precision > somewhere. I > > added a "Future Scope" section covering this. > > > > Let me know if there are major problems with this or other points to > cover. > > Otherwise, I'll aim to open voting on Monday. > > Let's just agree to disagree. I believe they should be investigated and > proposed in a single RFC. Having just one and relying on an obscure > documentation warning is not enough IMHO. I looked more closely at each of the APIs. My conclusion was that a single type will be appropriate for floats, doubles, and fixed-precision. I updated the RFC with details. If it's accepted, it could be worth including some of this content in the documentation so people better understand the impact of each PDO param type. I'll create a separate thread to announce the vote. Thanks, Adam --001a113e64a68c57ee054d73cfc5--