Question

Here’s a little something I discovered in vb.net which I cannot figure out, I’ve just got a form with a treeview on it and then the following:

Private Sub Form1_Load(sender As System.Object, e As System.EventArgs) Handles MyBase.Load
    treeTest.Nodes.Add("a")
    treeTest.Nodes(0).Test()
End Sub

Test is an extension method:

Imports System.Runtime.CompilerServices
Public Module ExtModule
    <Extension()>
    Public Sub Test(ByRef node As TreeNode)
    End Sub
End Module

If I use ByRef then my treeview looks like:

And with ByVal I get:

This seems totally backwards, if I’m simply sending a reference why is the node appearing twice, while if I make a copy it only appears once?

Answer 1

Okay, I’ve worked out some of what’s going on.

It’s got relatively little to do with extension methods per se. It’s more about how VB handles ByRef in general, and some odd behaviour of TreeView.Nodes by the looks of it.

In particular, you’ll get the exact same behaviour if you change this:

treeTest.Nodes(0).Test()

to:

ExtModule.Test(treeTest.Nodes(0))

… even if you remove the ExtensionAttribute.

Here’s some C# code which demonstrates the same effect, without using ref parameters or extension methods at all:

using System.Drawing;
using System.Windows.Forms;

class Test
{
    static void Main()
    {
        TreeView tree = new TreeView { Nodes = { "a" } };
        Form form = new Form { Controls = { tree } };
        form.Load += delegate {
            TreeNode node = tree.Nodes[0];
            tree.Nodes[0] = node;
        };
        Application.Run(form);
    }
}

The important lines are these ones:

TreeNode node = tree.Nodes[0];
tree.Nodes[0] = node;

When your empty extension method has a ByRef parameter, your code is equivalent to the above C# code – because VB fakes “real” ByRef behaviour by using a temporary variable and then assigning back to the original property.

When your empty extension method has a ByVal parameter, your code is just equivalent to:

TreeNode node = tree.Nodes[0];
// Do nothing

… and that doesn’t create a second node.