Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:106344 Return-Path: Delivered-To: mailing list internals@lists.php.net Received: (qmail 64327 invoked from network); 30 Jul 2019 15:10:06 -0000 Received: from unknown (HELO mail-io1-f47.google.com) (209.85.166.47) by pb1.pair.com with SMTP; 30 Jul 2019 15:10:06 -0000 Received: by mail-io1-f47.google.com with SMTP id s7so127472843iob.11 for ; Tue, 30 Jul 2019 05:35:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to; bh=Q9Hw8iTAEtGY4x+x+vhUsKwVWnMPXd5F2z8n3/Zj/aI=; b=Ew4zWtgK+UVBurMaOxEn7ifpPdq60Y37KOpanvNELRwxuhs4FwFrmWUV1UqGlIQfpT eJfBdA8EwH55YBeRv9A/xVsg7pmiqbyiCQ+p7zVnHc7CZ7EGbjLUnvDpAHvNCKx2uUcm YQ6MXntWYBUC9cSi+SzOBkk9L1U16rBqTl8V7KsAs8sx4oGdEwuB1e1TrDbdSF5nPyhH G99p/Sufw/jLE3d1CP+VF+xPu4l6f7/uxZpzs7ssVj763GueMiPueR/c2RP4ImlFDf/8 SypPeSYPmqS0v9rgSOFP7staVwRgo5IBiRP9psTvDnhHr6RszyN+o6f9AN8/QAUemyVk //Cw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to; bh=Q9Hw8iTAEtGY4x+x+vhUsKwVWnMPXd5F2z8n3/Zj/aI=; b=VzDgyLDPMpJEy6D/AXsUpCpmuvDj4TvQDLHmRdc0r+fvDIHDEMmzmkgiAITRPsRn6I kqa3KbzjcLy1SMAkEVUISJBoF7O8YRHzmLbD6IDtj9phO3FX4Isnm5yFy8Vgk6B8/sPB whTwDMSLY1BLzdazhBq8s2zVblx0lCyG9mjJILVjKcQrM5wB50Ej24QWcJ9uO3zTdfXm KPAZu8ymrDwCtNLpbQ5WJI7xELBByM609uqSg3uwvj0TE4CSLEZ7lhAoDuobzXBjfE3h v4rWMRjqkPOkrALqzQ3I65IJFK20+dbEB+kxx+ecEMRyslOEXBfI11SBrcNuln9Bdh6+ lyFQ== X-Gm-Message-State: APjAAAVS9TQt/Y1ZrJxH0e1RoAQ7wTkobz2e+6f6U3YOd6hNdOtI0VT/ OPT7wJkK40K0YRHvSnGSVFv/w0O1Y5Ak5+3LxCJ0G172 X-Google-Smtp-Source: APXvYqy7+VkIMF0cXtTJLAjkXvRYR0blQYSEiCWlhcqeGYWMwCE0MIcUam21OEILLvu3uqc9eyIwfuajJjQFMOshMIY= X-Received: by 2002:a6b:b214:: with SMTP id b20mr51258714iof.149.1564490099295; Tue, 30 Jul 2019 05:34:59 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: Date: Tue, 30 Jul 2019 13:34:48 +0100 Message-ID: To: PHP Internals List Content-Type: multipart/alternative; boundary="0000000000009f8b73058ee53c84" Subject: Re: [PHP-DEV] [RFC] Explicit call-site send-by-ref syntax From: rowan.collins@gmail.com (Rowan Collins) --0000000000009f8b73058ee53c84 Content-Type: text/plain; charset="UTF-8" On Tue, 30 Jul 2019 at 11:28, Nikita Popov wrote: > With that in mind, I don't see an issue with reusing the previous > call-time pass-by-ref syntax here. The & at the call-site still means that > the value is going to be passed by refrence (or error), so it's not like > someone who was around during the call-time pass-by-ref times would > misunderstand what the code does based on their prior knowledge. > > The only concern I see here is the emotional issue: Bringing & back as a > call-site annotation is an admission that the original migration of by-ref > passing has been badly botched: Yes, you removed those & at the call-site > for nothing. Yes, you shouldn't have been forced to do that! > > Mistakes happen and hindsight is 20/20. But we should own up to those > mistakes. > I think that's a reasonable summary. Could you add a short section to the RFC just acknowledging that history, and clarifying that this functionality is effectively a subset of the previously removed feature, but without its problems? You're right that it's more of an emotional reaction than a rational one, and shouldn't necessarily be a show-stopper IF we agree the proposed behaviour is a big enough gain. Regards, -- Rowan Collins [IMSoP] --0000000000009f8b73058ee53c84--