Difference between revisions of "DIP25"

From D Wiki
Jump to: navigation, search
(Typechecking rules)
 
(56 intermediate revisions by 7 users not shown)
Line 1: Line 1:
 
== DIP25: Sealed references ==
 
== DIP25: Sealed references ==
 +
 
{| class="wikitable"
 
{| class="wikitable"
 
!Title:
 
!Title:
Line 11: Line 12:
 
|-
 
|-
 
|Status:
 
|Status:
|Draft
+
|Approved for 2.067
 
|-
 
|-
 
|Created:
 
|Created:
Line 17: Line 18:
 
|-
 
|-
 
|Last Modified:
 
|Last Modified:
|2013-02-05
+
|{{REVISIONYEAR}}-{{REVISIONMONTH}}-{{REVISIONDAY}}
 
|-
 
|-
 
|Author:
 
|Author:
|Andrei Alexandrescu and Walter Bright
+
|Walter Bright and Andrei Alexandrescu
 
|-
 
|-
 
|Links:
 
|Links:
|
+
|See also: [http://wiki.dlang.org/DIP71 DIP71: 'noscope' and 'out!param' attributes]
 
|}
 
|}
  
 
== Abstract ==
 
== Abstract ==
  
D offers a number of features aimed at systems-level coding, such as unrestricted pointers, casting between integers and pointers, and the [http://dlang.org/function.html#system-functions <code>@system</code>] attribute. These means, combined with the other features of D, make it a complete and expressive language for systems-level tasks. On the other hand, economy of means should be exercised in defining such powerful but dangerous features. Most other features should offer good safety guarantees with little or no loss in efficiency or expressiveness. This proposal makes <code>ref</code> provide such a guarantee: with the proposed rules, it is impossible in safe code to have <code>ref</code> refer to a destroyed object. The restrictions introduced are not backward compatible, but disallow code that is stylistically questionable and that can be easily replaced either with equivalent and clearer code.
+
D offers a number of features aimed at systems-level coding, such as unrestricted pointers, casting between integers and pointers, and the [http://dlang.org/function.html#system-functions <code>@system</code>] attribute. These means, combined with the other features of D, make it a complete and expressive language for systems-level tasks. On the other hand, economy of means should be exercised in defining such powerful but dangerous features. Most other features should offer good safety guarantees with little or no loss in efficiency or expressiveness. This proposal makes <code>ref</code> provide such a guarantee: with the proposed rules, it is impossible in safe code to have <code>ref</code> refer to a destroyed object. The restrictions introduced are not entirely backward compatible, but disallow code that is stylistically questionable and that can be easily replaced either with equivalent and clearer code.
  
 
== In a nutshell ==
 
== In a nutshell ==
 +
 +
This DIP proposes that any <code>ref</code> parameter that a function received and also wants to return must be also annotated with <code>return</code>. Annotation are deduced for templates and lambdas, but must be explicit for all other declarations. Example:
 +
 +
<syntaxhighlight lang=D>
 +
@safe:
 +
ref int fun(ref int a) { return a; } // ERROR
 +
ref int gun(return ref int a) { return a; } // FINE
 +
ref T hun(T)(ref T a) { return a; } // FINE, templates use deduction
 +
</syntaxhighlight>
  
 
== Description ==
 
== Description ==
Line 44: Line 54:
 
   int x;
 
   int x;
 
   return x; // Error: escaping reference to local variable x  
 
   return x; // Error: escaping reference to local variable x  
 +
}
 +
 +
struct S {
 +
    int x;
 +
}
 +
 +
ref int hun() {
 +
  S s;
 +
  return s.x; // see https://issues.dlang.org/show_bug.cgi?id=13902
 +
}
 +
 +
ref int iun() {
 +
  int a[42];
 +
  return a[5]; // see https://issues.dlang.org/show_bug.cgi?id=13902
 
}
 
}
 
</syntaxhighlight>
 
</syntaxhighlight>
  
However, this enforcement is shallow. The following code compiles and allows reads and writes through defunct stack locations, bypassing scoping and lifetime rules:
+
However, this enforcement is shallow (even after fixing [https://issues.dlang.org/show_bug.cgi?id=13902 issue 13902]). The following code compiles and allows reads and writes through defunct stack locations, bypassing scoping and lifetime rules:
  
 
<syntaxhighlight lang=D>
 
<syntaxhighlight lang=D>
ref int id(ref int x) {
+
ref int identity(ref int x) {
   return x;  
+
   return x; // pass-through function that does nothing
 
}
 
}
  
 
ref int fun(int x) {
 
ref int fun(int x) {
   return id(x);  
+
   return identity(x); // escape the address of a parameter
 
}
 
}
  
 
ref int gun() {
 
ref int gun() {
 
   int x;
 
   int x;
   return id(x);  
+
   return identity(x); // escape the address of a local
 +
}
 +
 
 +
struct S {
 +
    int x;
 +
    ref int get() { return x; }
 +
}
 +
 
 +
ref int hun(S x) {
 +
  return x.get; // escape the address of a part of a parameter
 +
}
 +
 
 +
ref int iun() {
 +
  S s;
 +
  return s.get; // escape the address of part of a local
 +
}
 +
 
 +
ref int jun() {
 +
  return S().get; // worst contender: escape the address of a part of an rvalue
 
}
 
}
 
</syntaxhighlight>
 
</syntaxhighlight>
  
The escape pattern is obvious in this simple example with all code in sight, and may be found automatically. The problem is that generally the compiler cannot see the body of <code>id</code>. We need to devise a method for compiling such functions separately.
+
The escape patterns are obvious in these simple examples that make all code available and use no recursion, and may be found automatically. The problem is that generally the compiler cannot see the body of <code>identity</code> or <code>S.get()</code>. We need to devise a method that derives enough information for safety analysis only given the function signatures, not their bodies.
  
We want to devise rules that allow us to pass objects by reference ''down'' into functions, and return references ''up'' from functions, while disallowing cases such as the above when a reference passed up ends up referring to a deallocated temporary.
+
This DIP devises rules that allow passing objects by reference ''down'' into functions, and return references ''up'' from functions, whilst disallowing cases such as the above when a reference passed up ends up referring to a deallocated temporary.
  
=== Typechecking rules ===
+
=== Adding <tt>return</tt> as a parameter attribute ===
  
The rules below discuss under what circumstances functions receiving and/or returning <code>ref T</code> may be called, where <code>T</code> is some arbitrary type. Let us also denote with <code>S<sub>T</sub></code> any <code>struct</code> that has a non-static member variable of type <code>T</code>.
+
The main issue is typechecking functions that return a <tt>ref T</tt> and accept some of their parameters by <tt>ref</tt>. Those that attempt to return locals or parts thereof are already addressed directly, contingent to [https://issues.dlang.org/show_bug.cgi?id=13902 Issue 13902]. The one case remaining is allowing a function returning <code>ref T</code> to return a (part of a) parameter passed by <code>ref</code>.
  
# An invocation of a function that takes a parameter of type <code>ref T</code> may pass one of the following:
+
The key is to distinguish legal from illegal cases. One simple but overly conservative option would be to simply disallow returning a <code>ref</code> parameter or part thereof. That makes <code>identity</code> impossible to implement, and as a consequence accessing elements of a container by reference becomes difficult or impossible to typecheck properly. Also, heap-allocated structures with deterministic destruction (e.g. reference counted) must insert member copies for all accesses.  
## An lvalue of type <code>T</code>, including function arguments, array and <code>struct</code> members;
 
## An incoming <code>ref T</code> parameter or a member of type <code>T</code> of <code>S<sub>T</sub></code> received as <code>ref S<sub>T</sub></code>
 
## The result of a function returning <code>ref T</code>, or a member of <code>S<sub>T</sub></code> returned as <code>ref S<sub>T</sub></code>.
 
# A function that returns a <code>ref T</code> may return one of the following:
 
##  A static lvalue of type <code>T</code>, including members of static <code>struct</code> values;
 
## A member variable of type <code>T</code> belonging to a <code>class</code> object;
 
## A <code>ref T</code> parameter;
 
## A member of type <code>T</code> of <code>S<sub>T</sub></code> that has been passed as <code>ref S<sub>T</sub></code> into the function;
 
## The invocation of a function <code>fun</code> returning <code>ref T</code> IF <code>fun</code> does NOT take any parameters of type <code>T</code> or <code>S<sub>T</sub></code>.
 
## The invocation of a function <code>fun</code> returning <code>ref T</code> IF none of <code>fun</code>'s parameters of type <code>ref T</code> and <code>ref S</code> are bound to local variables.
 
  
=== Discussion and Examples ===
+
This proposal promotes adding <code>return</code> as an attribute that propagates the lifetime of a parameter to the return value of a function. With the proposed semantics, a function is disallowed to return a <code>ref</code> parameter or a part thereof UNLESS the parameter is also annotated with <code>return</code>. Under the proposed semantics <code>identity</code> will be spelled as follows:
  
The rules allow unrestricted pass-down and conservatively restrict pass-up to avoid escaping values. Itemized discussion follows
+
<syntaxhighlight lang=D>
 +
@safe ref int wrongIdentity(ref int x) {
 +
    return x; // ERROR! Cannot return a ref, please use "return ref"
 +
}
 +
@safe ref int identity(return ref int x) {
 +
    return x; // fine
 +
}
 +
</syntaxhighlight>
  
1.1 Regular lvalues can be passed down:
+
Just by seeing the signature <code>ref int identity(return ref int x)</code> the compiler assumes that the result of identity must have a shorter or equal lifetime than <code>x</code> and typechecks callers accordingly. Example (given the previous definition of <code>identity</code>):
  
 
<syntaxhighlight lang=D>
 
<syntaxhighlight lang=D>
void fun(ref T);
+
@safe ref int fun(return ref int x) {
 +
    int a;
 +
    return a; // ERROR per current language rules
 +
    static int b;
 +
    return b; // fine per current language rules
 +
    return identity(a); // ERROR, this may escape the address of a local
 +
    return x; // fine, propagate x's lifetime to output
 +
    return identity(x); // fine, propagate x's lifetime through identity to the output
 +
    return identity(identity(x)); // fine, propagate x's lifetime twice through identity to the output
 +
}
 +
 
 +
@safe ref int gun(ref int input) {
 +
    static int[42] data;
 +
    return data[input]; // works, can always return static-lived data
 +
}
  
struct S { int a; T b; }
+
@safe struct S {
 +
    private int x;
 +
    ref int get() return { return x; } // should work, see next section
 +
}
 +
</syntaxhighlight>
  
void caller(T v1, S v2) {
+
===Interaction with <tt>auto ref</tt>===
    static T v3;
 
    T v4;
 
    static S v5;
 
    S v6;
 
  
    // Fine: pass argument
+
Syntactically it is illegal to use <tt>auto ref</tt> and <tt>return ref</tt> on the same parameter. Deduction of the <tt>return</tt> attribute still applies as discussed below.
    fun(v1);
+
 
    // Fine: pass member of argument
+
===Deduction===
    fun(v2.b);
+
 
    // Fine: pass static lvalue
+
Deduction of the <tt>return</tt> attribute will be effected under the same conditions as for <tt>pure</tt> (currently for generic and lambda functions). That means the generic <tt>identity</tt> function does not require the <tt>return</tt> attribute:
    fun(v3);
+
 
    // Fine: pass of stack variable
+
<syntaxhighlight lang=D>
    fun(v4);
+
auto ref T identity(auto ref T x) {
    // Fine: pass member of static struct
+
     return x; // correct, no need for return
    fun(v5.b);
 
     // Fine: pass member of local struct
 
    fun(v6.b);
 
 
}
 
}
 
</syntaxhighlight>
 
</syntaxhighlight>
  
1.2. This rule allows forwarding references transitively.
+
===Types of Result vs. Parameters===
 +
 
 +
Consider:
  
 
<syntaxhighlight lang=D>
 
<syntaxhighlight lang=D>
void fun(ref T);
+
@safe ref int fun(return ref float x);
 +
</syntaxhighlight>
 +
 
 +
This function arguably cannot return a value scoped within the lifetime of its argument for the simple reason it's impossible to find an <code>int</code> somewhere in a <code>float</code> (apart from unsafe address manipulation). However, this DIP ignores types; if a parameter is <code>return ref</code>, it is always considered potentially escaped as a result. It is in fact possible that the author of <code>fun</code> wants to constrain its output's lifetime for unrelated reasons.
 +
 
 +
Future versions of this DIP may relax this rule.
 +
 
 +
===Multiple Parameters===
  
struct S { int a; T b; }
+
If multiple <code>return ref</code> parameters are present, the result's lifetime is conservatively assumed to be enclosed in the lifetime of the shortest-lived of those arguments.
  
void caller(ref T v1, ref S v2) {
+
===Member Functions===
     // Fine: pass ref argument
+
Member functions of <code>struct</code>s must qualify <code>this</code> with <code>return</code> if they want to return a result by <code>ref</code> that won't outlive <code>this</code>. Example:
     fun(v1);
+
 
     // Fine: pass member of ref argument
+
<syntaxhighlight lang=D>
     fun(v2.b);
+
@safe struct S {
 +
     static int a;
 +
    int b;
 +
     ref int fun() { return a; } // fine, callers assume infinite lifetime
 +
     ref int gun() { return b; } // ERROR! Cannot return a direct member
 +
     ref int hun() return { return b; } // fine, result is scoped within this
 
}
 
}
 +
</syntaxhighlight>
 +
 +
===@safe===
 +
For the initial release, the requirement of returns for <code>ref</code> parameter data to be marked with <code>return</code> will only apply to <code>@safe</code> functions. The reasons for this are to avoid breaking existing code, and because it's not yet clear whether this feature will interfere with valid constructs in a system language.
 +
 +
<syntaxhighlight lang=D>
 +
@safe  ref int fun(ref int x)        { return x;} // Error
 +
@safe  ref int gun(return ref int x) { return x;} // OK
 +
@system ref int hun(ref int x)        { return x;} // OK for now, @system code.
 +
@system ref int jun(return ref int x) { return x;} // preferred, gives more hints to compiler for lifetime of return value
 
</syntaxhighlight>
 
</syntaxhighlight>
  
 
== Copyright ==
 
== Copyright ==
 
This document has been placed in the Public Domain.
 
This document has been placed in the Public Domain.
 +
 +
[[Category: DIP]]

Latest revision as of 19:51, 1 September 2015

DIP25: Sealed references

Title: Sealed references
DIP: 25
Version: 1
Status: Approved for 2.067
Created: 2013-02-05
Last Modified: 2015-09-1
Author: Walter Bright and Andrei Alexandrescu
Links: See also: DIP71: 'noscope' and 'out!param' attributes

Abstract

D offers a number of features aimed at systems-level coding, such as unrestricted pointers, casting between integers and pointers, and the @system attribute. These means, combined with the other features of D, make it a complete and expressive language for systems-level tasks. On the other hand, economy of means should be exercised in defining such powerful but dangerous features. Most other features should offer good safety guarantees with little or no loss in efficiency or expressiveness. This proposal makes ref provide such a guarantee: with the proposed rules, it is impossible in safe code to have ref refer to a destroyed object. The restrictions introduced are not entirely backward compatible, but disallow code that is stylistically questionable and that can be easily replaced either with equivalent and clearer code.

In a nutshell

This DIP proposes that any ref parameter that a function received and also wants to return must be also annotated with return. Annotation are deduced for templates and lambdas, but must be explicit for all other declarations. Example:

@safe:
ref int fun(ref int a) { return a; } // ERROR
ref int gun(return ref int a) { return a; } // FINE
ref T hun(T)(ref T a) { return a; } // FINE, templates use deduction

Description

Currently, D has some provisions for avoiding dangling references:

ref int fun(int x) {
  return x; // Error: escaping reference to local variable x 
}

ref int gun() {
  int x;
  return x; // Error: escaping reference to local variable x 
}

struct S {
    int x;
}

ref int hun() {
  S s;
  return s.x; // see https://issues.dlang.org/show_bug.cgi?id=13902
}

ref int iun() {
  int a[42];
  return a[5]; // see https://issues.dlang.org/show_bug.cgi?id=13902
}

However, this enforcement is shallow (even after fixing issue 13902). The following code compiles and allows reads and writes through defunct stack locations, bypassing scoping and lifetime rules:

ref int identity(ref int x) {
  return x; // pass-through function that does nothing 
}

ref int fun(int x) {
  return identity(x); // escape the address of a parameter 
}

ref int gun() {
  int x;
  return identity(x); // escape the address of a local
}

struct S {
    int x;
    ref int get() { return x; }
}

ref int hun(S x) {
  return x.get; // escape the address of a part of a parameter 
}

ref int iun() {
  S s;
  return s.get; // escape the address of part of a local
}

ref int jun() {
  return S().get; // worst contender: escape the address of a part of an rvalue
}

The escape patterns are obvious in these simple examples that make all code available and use no recursion, and may be found automatically. The problem is that generally the compiler cannot see the body of identity or S.get(). We need to devise a method that derives enough information for safety analysis only given the function signatures, not their bodies.

This DIP devises rules that allow passing objects by reference down into functions, and return references up from functions, whilst disallowing cases such as the above when a reference passed up ends up referring to a deallocated temporary.

Adding return as a parameter attribute

The main issue is typechecking functions that return a ref T and accept some of their parameters by ref. Those that attempt to return locals or parts thereof are already addressed directly, contingent to Issue 13902. The one case remaining is allowing a function returning ref T to return a (part of a) parameter passed by ref.

The key is to distinguish legal from illegal cases. One simple but overly conservative option would be to simply disallow returning a ref parameter or part thereof. That makes identity impossible to implement, and as a consequence accessing elements of a container by reference becomes difficult or impossible to typecheck properly. Also, heap-allocated structures with deterministic destruction (e.g. reference counted) must insert member copies for all accesses.

This proposal promotes adding return as an attribute that propagates the lifetime of a parameter to the return value of a function. With the proposed semantics, a function is disallowed to return a ref parameter or a part thereof UNLESS the parameter is also annotated with return. Under the proposed semantics identity will be spelled as follows:

@safe ref int wrongIdentity(ref int x) { 
    return x; // ERROR! Cannot return a ref, please use "return ref"
}
@safe ref int identity(return ref int x) { 
    return x; // fine
}

Just by seeing the signature ref int identity(return ref int x) the compiler assumes that the result of identity must have a shorter or equal lifetime than x and typechecks callers accordingly. Example (given the previous definition of identity):

@safe ref int fun(return ref int x) { 
    int a;
    return a; // ERROR per current language rules
    static int b;
    return b; // fine per current language rules
    return identity(a); // ERROR, this may escape the address of a local
    return x; // fine, propagate x's lifetime to output
    return identity(x); // fine, propagate x's lifetime through identity to the output
    return identity(identity(x)); // fine, propagate x's lifetime twice through identity to the output
}

@safe ref int gun(ref int input) {
    static int[42] data;
    return data[input]; // works, can always return static-lived data
}

@safe struct S {
    private int x;
    ref int get() return { return x; } // should work, see next section 
}

Interaction with auto ref

Syntactically it is illegal to use auto ref and return ref on the same parameter. Deduction of the return attribute still applies as discussed below.

Deduction

Deduction of the return attribute will be effected under the same conditions as for pure (currently for generic and lambda functions). That means the generic identity function does not require the return attribute:

auto ref T identity(auto ref T x) {
    return x; // correct, no need for return
}

Types of Result vs. Parameters

Consider:

@safe ref int fun(return ref float x);

This function arguably cannot return a value scoped within the lifetime of its argument for the simple reason it's impossible to find an int somewhere in a float (apart from unsafe address manipulation). However, this DIP ignores types; if a parameter is return ref, it is always considered potentially escaped as a result. It is in fact possible that the author of fun wants to constrain its output's lifetime for unrelated reasons.

Future versions of this DIP may relax this rule.

Multiple Parameters

If multiple return ref parameters are present, the result's lifetime is conservatively assumed to be enclosed in the lifetime of the shortest-lived of those arguments.

Member Functions

Member functions of structs must qualify this with return if they want to return a result by ref that won't outlive this. Example:

@safe struct S {
    static int a;
    int b;
    ref int fun() { return a; } // fine, callers assume infinite lifetime
    ref int gun() { return b; } // ERROR! Cannot return a direct member
    ref int hun() return { return b; } // fine, result is scoped within this
}

@safe

For the initial release, the requirement of returns for ref parameter data to be marked with return will only apply to @safe functions. The reasons for this are to avoid breaking existing code, and because it's not yet clear whether this feature will interfere with valid constructs in a system language.

@safe   ref int fun(ref int x)        { return x;} // Error
@safe   ref int gun(return ref int x) { return x;} // OK
@system ref int hun(ref int x)        { return x;} // OK for now, @system code.
@system ref int jun(return ref int x) { return x;} // preferred, gives more hints to compiler for lifetime of return value

Copyright

This document has been placed in the Public Domain.