T0b1/wasmtime
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Loop the array of strings only once.

Browse Source 
Remove unnecessary loop when marshaling lists of strings as pointers to UTF-8
bytes.
This commit is contained in:
Peter Huene
2020-02-24 18:26:26 -08:00
parent 78ee043524
commit a6ec8f85a6
1 changed files with 1 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
crates/misc/dotnet/src/Interop.cs
View File

@@ -496,17 +496,13 @@ namespace Wasmtime
// See: https://github.com/dotnet/runtime/issues/7315 // See: https://github.com/dotnet/runtime/issues/7315
// Therefore, we need to marshal the strings manually // Therefore, we need to marshal the strings manually
var handles = new GCHandle[strings.Count]; var handles = new GCHandle[strings.Count];
var ptrs = new byte*[strings.Count];
for (int i = 0; i < strings.Count; ++i) for (int i = 0; i < strings.Count; ++i)
{ {
handles[i] = GCHandle.Alloc( handles[i] = GCHandle.Alloc(
Encoding.UTF8.GetBytes(strings[i] + '\0'), Encoding.UTF8.GetBytes(strings[i] + '\0'),
GCHandleType.Pinned GCHandleType.Pinned
); );
}
var ptrs = new byte*[strings.Count];
for (int i = 0; i < strings.Count; ++i)
{
ptrs[i] = (byte*)handles[i].AddrOfPinnedObject(); ptrs[i] = (byte*)handles[i].AddrOfPinnedObject();
} }