Merge pull request #728 from microsoft/fix717

Add BSTR.Length property

Author: AArnott

src/Microsoft.Windows.CsWin32/Generator.cs

@@ -4157,6 +4157,7 @@ private StructDeclarationSyntax DeclareTypeDefStruct(TypeDefinition typeDef, Typ
         {
             case "BSTR":
                 members = members.AddRange(this.CreateAdditionalTypeDefBSTRMembers());
+                members = members.AddRange(this.ExtractMembersFromTemplate(name.Identifier.ValueText));
                 break;
             case "PWSTR":
                 members = members.AddRange(this.CreateAdditionalTypeDefPWSTRMembers());

src/Microsoft.Windows.CsWin32/templates/BSTR.cs

@@ -0,0 +1,7 @@
+internal partial struct BSTR
+{
+	/// <summary>
+	/// Gets the length of the BSTR in characters.
+	/// </summary>
+	internal unsafe int Length => this.Value is null ? 0 : checked((int)(*(((uint*)this.Value) - 1) / sizeof(char)));
+}

test/GenerationSandbox.Tests/BasicTests.cs

@@ -80,6 +80,29 @@ public void BSTR_ToString_Null()
         Assert.Null(bstr.ToString());
     }
 
+    [Theory]
+    [InlineData("")]
+    [InlineData("h")]
+    [InlineData("hello")]
+    public void BSTR_Length(string value)
+    {
+        BSTR bstr = (BSTR)Marshal.StringToBSTR(value);
+        try
+        {
+            Assert.Equal(value.Length, bstr.Length);
+        }
+        finally
+        {
+            Marshal.FreeBSTR(bstr);
+        }
+    }
+
+    [Fact]
+    public void BSTR_Length_Null()
+    {
+        Assert.Equal(0, default(BSTR).Length);
+    }
+
     [Fact]
     public unsafe void BSTR_ImplicitConversionTo_ReadOnlySpan()
     {