Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:39599
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.134.189 as permitted sender)
DomainKey-Status: bad
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=message-id:date:from:to:subject:cc:in-reply-to:mime-version
         :content-type:content-transfer-encoding:content-disposition
         :references;
        b=ahWq/FKdsykF5ULYFJI3WcXBsa/2RhNCblJ4SSxpEYKwg/uTDJ4SVVDboyNRFjty1P
         IV+aGC7GLOLqSD9OpZxgtZM8KoVFQAxb5jSCoxXogb4rK98T9H5M5+SCMYU32A84qE8k
         Zh8fGjh4Hi4X01+mkxfj2ukn8qlxneeEvBdsQ=
Message-ID: <7f3ed2c30808040049w7172205ey4ae460be4beabcee@mail.gmail.com>
Date: Mon, 4 Aug 2008 09:49:43 +0200
To: "Davey Shafik" <davey@php.net>
Cc: internals@lists.php.net
In-Reply-To: <220AF877-747F-4BDB-8631-235B54ED2503@php.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <220AF877-747F-4BDB-8631-235B54ED2503@php.net>
Subject: Re: [PHP-DEV] [PATCH] Alias stream_context_get_default() as stream_context_set_default()
From: hannes.magnusson@gmail.com ("Hannes Magnusson")

On Mon, Aug 4, 2008 at 01:29, Davey Shafik <davey@php.net> wrote:
> Hey,
>
> it has come to my attention that stream_context_get_default() is /grossly/
> misnamed, considering
> that it can be used (and is the only way) to SET default options. It works
> more like ini_set() in that
> you pass in the new defaults and it returns the old.
>
> With the current name, it is as bad as if we added an optional argument to
> ini_get() that would also
> set the value.
>
> I know we're already into PHP 5.3 alpha, but this is a simple alias, so I'm
> hoping you can squeeze it in.

That doesn't really solve anything as the argument is optional. In
fact, to me it reads like the default context would be reset to
nothing.

$ctx = stream_context_set_default();
Doesn't look any less weird than stream_context_get_default($ctx);

I do agree though. The naming convention of the stream_*() functions
are extremely bad.

-Hannes