Your solution is flawed, it would not allow autoloading a function that has the
same
(namespaced) name as a builtin.
Example:
// autoloadable function
namespace foo {
// make arg order consistent
function array_filter($callback, $input) {
return \array_filter($input, $callback);
}
function is_positive($x) {
return $x > 0;
}
}
namespace foo {
var_dump(array_filter('foo\is_positive', [-2, -1, 0, 1, 2]));
}
Always triggering the autoloader is not feasible either, as it will lead to
tons of stat calls,
one per namespace that any global function is called in.
Potential solutions: Only autoload fully qualified or imported functions.
But there is lots of potential for bikeshedding there, plus it requires more
implementation
effort, and quite possibly more performance and cache related considerations
that I am
not aware of.
As much as I would like to see function autoloading, I don't think bundling the
proposals
is going to be beneficial. Which is why I'd rather just focus on one issue at a
time.
Regards,
Igor
On May 3, 2013, at 12:18 AM, Sebastian Krebs <krebs....@gmail.com> wrote:
> Well, it is not as complex as it sounds
>
> This is the behaviour right now
>
> - Does X exists as namespaced function (current namespace or imported)?
> - Does X exists as global (builtin) function?
> - Trigger error
>
> The behaviour including autoloading would be more like
>
> - Does X exists as namespaced function (current namespace or imported)?
> - Does X exists as global (builtin) function?
> - Trigger autoload
> - Does X exists as namespaced function (current namespace or imported) now?
> - Maybe: Does X exists as global function now?
> - Trigger error
>
> The autoloading itself could use parts already used for class-autoloading.
>
> // Signature
> // spl_autoload_register($callback[, $type = AUTOLOAD_CLASS]);
>
> $autoload = function ($name, $type = AUTOLOAD_CLASS) {
> // Do something
> };
> spl_autoload_register($autoload, AUTOLOAD_CLASS | AUTOLOAD_FUNCTION |
> AUTOLOAD_CONSTANT); // "constant" here just to make it complete
>
> Namespaced functions without autoloading seems kindof incomplete, especially
> because it isn't such a big deal to simply use the prefixed one, so the
> benefit seems quite small to me :X
>
> use MyFoo\Bar\MathFunctions as math;
> $x = math\sin($y);
>
>
> Just saying :)
>
>
> Regards,
> Sebastian
>
>
>
>
>
> On May 2, 2013, at 11:40 PM, Sebastian Krebs <krebs....@gmail.com> wrote:
>
> > Hi,
> >
> > Are you going to cover autoloading of functions too?
> >
> > Regards,
> > Sebastian
>
>
>
> --
> github.com/KingCrunch
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
