ref parameters: there is no escape
Jacob Carlborg
doob at me.com
Sun Aug 14 08:37:25 PDT 2011
On 2011-08-14 16:20, Andrei Alexandrescu wrote:
> Walter and I have had a long discussion and we thought we'd bring an
> idea for community review.
>
> We believe it would be useful for safety purposes to disallow escaping
> addresses of ref parameters. Consider:
>
> class C {
> int * p;
> this(ref int x) {
> p = &x; // escapes the address of a ref parameter
> }
> }
>
> Such code is accepted today. We believe it is error-prone and dangerous,
> particularly because the caller has no syntactic cue that the address of
> the parameter is passed into the function (in this case constructor).
> Worse, such a function cannot be characterized as @safe.
>
> So we want to make the above an error. The workaround is obvious - just
> take int* as a parameter instead of ref int. What a function can do with
> a ref parameter in general is:
>
> * use it directly just like a local;
>
> * pass it down to other functions (which may take it by value or
> reference);
>
> * pass its address down to pure functions because a pure function cannot
> escape the address anyway (cool insight by Walter);
>
> * take its address as long as the address doesn't outlive the frame of
> the function.
>
> The third bullet is not easy to implement as it requires flow analysis,
> but we may start with a conservative version first. Probably there won't
> be a lot of broken code anyway.
>
> Please chime in with any comments you might have!
>
>
> Thanks,
>
> Andrei
I have code relying on this, probably not could practice but it works.
This is a usage example:
void main ()
{
int i = 3;
restore(i) in {
i = 4;
};
assert(i == 3);
}
Restore returns a struct which overloads the "in" operator and stores a
pointer to the value pass to "restore". I'm overloading the "in"
operator have a nicer looking delegate syntax. But I guess this could be
seen as operator overload abuse. If D just could have a good looking
syntax for delegate literals, like this:
restore(i) {
i = 4;
}
Then this wouldn't be needed.
--
/Jacob Carlborg
More information about the Digitalmars-d
mailing list