From 2aa6204f440b2cf0c773ea2b9207ba51d18a4b89 Mon Sep 17 00:00:00 2001 From: Eddy Luo Date: Wed, 12 Jun 2019 10:55:24 -0700 Subject: [PATCH] Updated comments, removed unused method --- src/Microsoft.CorrelationVector/CorrelationVector.cs | 8 -------- src/Microsoft.CorrelationVector/CorrelationVectorV1.cs | 1 + src/Microsoft.CorrelationVector/CorrelationVectorV2.cs | 1 + 3 files changed, 2 insertions(+), 8 deletions(-) diff --git a/src/Microsoft.CorrelationVector/CorrelationVector.cs b/src/Microsoft.CorrelationVector/CorrelationVector.cs index 0a5e3db..377bea6 100644 --- a/src/Microsoft.CorrelationVector/CorrelationVector.cs +++ b/src/Microsoft.CorrelationVector/CorrelationVector.cs @@ -143,14 +143,6 @@ namespace Microsoft.CorrelationVector } } - /// - /// Run this to throw an exception for non-supported cV methods in RunStaticMethod. - /// - private static CorrelationVector NotHandledMethod(string correlationVector) - { - throw new InvalidOperationException("Method not supported in this version"); - } - public abstract Tuple Reset(); public abstract string Increment(); diff --git a/src/Microsoft.CorrelationVector/CorrelationVectorV1.cs b/src/Microsoft.CorrelationVector/CorrelationVectorV1.cs index 9837d32..a3ca40f 100644 --- a/src/Microsoft.CorrelationVector/CorrelationVectorV1.cs +++ b/src/Microsoft.CorrelationVector/CorrelationVectorV1.cs @@ -10,6 +10,7 @@ namespace Microsoft.CorrelationVector /// /// This class represents a lightweight vector for identifying and measuring /// causality. + /// This is version 1 of the cV, which uses a shorter base length and can only be incremented. /// public sealed class CorrelationVectorV1 : CorrelationVector { diff --git a/src/Microsoft.CorrelationVector/CorrelationVectorV2.cs b/src/Microsoft.CorrelationVector/CorrelationVectorV2.cs index 38f6582..3035404 100644 --- a/src/Microsoft.CorrelationVector/CorrelationVectorV2.cs +++ b/src/Microsoft.CorrelationVector/CorrelationVectorV2.cs @@ -10,6 +10,7 @@ namespace Microsoft.CorrelationVector /// /// This class represents a lightweight vector for identifying and measuring /// causality. + /// This is version 2 of the cV, which is longer and can utilize the Spin operations. /// public sealed class CorrelationVectorV2 : CorrelationVector {