IGNITE-3711 .NET: Fix PlatformRawMemory coverage issue
Project: http://git-wip-us.apache.org/repos/asf/ignite/repo Commit: http://git-wip-us.apache.org/repos/asf/ignite/commit/4b87830d Tree: http://git-wip-us.apache.org/repos/asf/ignite/tree/4b87830d Diff: http://git-wip-us.apache.org/repos/asf/ignite/diff/4b87830d Branch: refs/heads/ignite-3220-1 Commit: 4b87830da594558d68d19d648bfe9fbf0b3af64f Parents: 3404ed5 Author: Pavel Tupitsyn <[email protected]> Authored: Fri Aug 19 17:34:00 2016 +0300 Committer: Pavel Tupitsyn <[email protected]> Committed: Fri Aug 19 17:34:00 2016 +0300 ---------------------------------------------------------------------- .../Impl/Memory/PlatformRawMemory.cs | 1 + .../Impl/Unmanaged/UnmanagedCallbacks.cs | 14 +++++--------- 2 files changed, 6 insertions(+), 9 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ignite/blob/4b87830d/modules/platforms/dotnet/Apache.Ignite.Core/Impl/Memory/PlatformRawMemory.cs ---------------------------------------------------------------------- diff --git a/modules/platforms/dotnet/Apache.Ignite.Core/Impl/Memory/PlatformRawMemory.cs b/modules/platforms/dotnet/Apache.Ignite.Core/Impl/Memory/PlatformRawMemory.cs index f252ef3..c3cbee2 100644 --- a/modules/platforms/dotnet/Apache.Ignite.Core/Impl/Memory/PlatformRawMemory.cs +++ b/modules/platforms/dotnet/Apache.Ignite.Core/Impl/Memory/PlatformRawMemory.cs @@ -24,6 +24,7 @@ namespace Apache.Ignite.Core.Impl.Memory /// Non-resizeable raw memory chunk without metadata header. /// </summary> [CLSCompliant(false)] + [ExcludeFromCodeCoverage] public class PlatformRawMemory : IPlatformMemory { /** */ http://git-wip-us.apache.org/repos/asf/ignite/blob/4b87830d/modules/platforms/dotnet/Apache.Ignite.Core/Impl/Unmanaged/UnmanagedCallbacks.cs ---------------------------------------------------------------------- diff --git a/modules/platforms/dotnet/Apache.Ignite.Core/Impl/Unmanaged/UnmanagedCallbacks.cs b/modules/platforms/dotnet/Apache.Ignite.Core/Impl/Unmanaged/UnmanagedCallbacks.cs index c9284d5..68dbe4e 100644 --- a/modules/platforms/dotnet/Apache.Ignite.Core/Impl/Unmanaged/UnmanagedCallbacks.cs +++ b/modules/platforms/dotnet/Apache.Ignite.Core/Impl/Unmanaged/UnmanagedCallbacks.cs @@ -1102,6 +1102,11 @@ namespace Apache.Ignite.Core.Impl.Unmanaged private void Error(void* target, int errType, sbyte* errClsChars, int errClsCharsLen, sbyte* errMsgChars, int errMsgCharsLen, sbyte* stackTraceChars, int stackTraceCharsLen, void* errData, int errDataLen) { + // errData mechanism is only needed for CachePartialUpdateException and is no longer used, + // since CacheImpl handles all errors itself. + Debug.Assert(errDataLen == 0); + Debug.Assert(errData == null); + string errCls = IgniteUtils.Utf8UnmanagedToString(errClsChars, errClsCharsLen); string errMsg = IgniteUtils.Utf8UnmanagedToString(errMsgChars, errMsgCharsLen); string stackTrace = IgniteUtils.Utf8UnmanagedToString(stackTraceChars, stackTraceCharsLen); @@ -1109,15 +1114,6 @@ namespace Apache.Ignite.Core.Impl.Unmanaged switch (errType) { case ErrGeneric: - if (_ignite != null && errDataLen > 0) - { - // Stream disposal intentionally omitted: IGNITE-1598 - var stream = new PlatformRawMemory(errData, errDataLen).GetStream(); - - throw ExceptionUtils.GetException(_ignite, errCls, errMsg, stackTrace, - _ignite.Marshaller.StartUnmarshal(stream)); - } - throw ExceptionUtils.GetException(_ignite, errCls, errMsg, stackTrace); case ErrJvmInit:
