diff options
author | Sergey Andreenko <seandree@microsoft.com> | 2018-11-01 17:43:17 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-11-01 17:43:17 -0700 |
commit | 61edf4d3200be7c6f59f2558ef5a5773170cfba9 (patch) | |
tree | 532816507017067fdbdbbd96508c64962e53d12e | |
parent | dc028d9893915c4c1bd62daf5e12325c15334154 (diff) | |
download | coreclr-61edf4d3200be7c6f59f2558ef5a5773170cfba9.tar.gz coreclr-61edf4d3200be7c6f59f2558ef5a5773170cfba9.tar.bz2 coreclr-61edf4d3200be7c6f59f2558ef5a5773170cfba9.zip |
Fix the test. (#20701)
As https://docs.microsoft.com/en-us/dotnet/api/system.runtime.interopservices.gchandle.free?view=netframework-4.7.2 says "The caller must ensure that for a given handle, Free is called only once.".
So delete the second call to `Free()`.
I believe it changes the desired behaviour for this test but it was added before 2010 (via source depo) and there are no comments about the regression that it was able to repro, so it is not worth to investigate that.
-rw-r--r-- | tests/src/JIT/Regression/CLR-x86-JIT/V1-M09/b13621/b13621.cs | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/tests/src/JIT/Regression/CLR-x86-JIT/V1-M09/b13621/b13621.cs b/tests/src/JIT/Regression/CLR-x86-JIT/V1-M09/b13621/b13621.cs index f7dd1fe097..ef5ab44ef7 100644 --- a/tests/src/JIT/Regression/CLR-x86-JIT/V1-M09/b13621/b13621.cs +++ b/tests/src/JIT/Regression/CLR-x86-JIT/V1-M09/b13621/b13621.cs @@ -45,7 +45,6 @@ namespace DefaultNamespace ~RootMem() { - root[n - 1].Free(); } } } |