Skip to content

Commit

Permalink
Merge pull request #2174 from Reapism/patch-1
Browse files Browse the repository at this point in the history
Fix some formatting and comments.
  • Loading branch information
dansiegel committed Aug 22, 2020
2 parents 7b614ec + 6f3f297 commit c200a23
Showing 1 changed file with 9 additions and 12 deletions.
21 changes: 9 additions & 12 deletions src/Prism.Core/Events/SubscriptionToken.cs
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@



using System;

namespace Prism.Events
Expand Down Expand Up @@ -50,13 +47,13 @@ public override bool Equals(object obj)
return Equals(obj as SubscriptionToken);
}

///<summary>
///Serves as a hash function for a particular type.
///</summary>
///<returns>
///A hash code for the current <see cref="T:System.Object" />.
///</returns>
///<filterpriority>2</filterpriority>
/// <summary>
/// Serves as a hash function for a particular type.
/// </summary>
/// <returns>
/// A hash code for the current <see cref="T:System.Object" />.
/// </returns>
/// <filterpriority>2</filterpriority>
public override int GetHashCode()
{
return _token.GetHashCode();
Expand All @@ -68,7 +65,7 @@ public override int GetHashCode()
[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Design", "CA1063:ImplementIDisposableCorrectly", Justification = "Should never have need for a finalizer, hence no need for Dispose(bool).")]
public virtual void Dispose()
{
// While the SubsctiptionToken class implements IDisposable, in the case of weak subscriptions
// While the SubscriptionToken class implements IDisposable, in the case of weak subscriptions
// (i.e. keepSubscriberReferenceAlive set to false in the Subscribe method) it's not necessary to unsubscribe,
// as no resources should be kept alive by the event subscription.
// In such cases, if a warning is issued, it could be suppressed.
Expand All @@ -82,4 +79,4 @@ public virtual void Dispose()
GC.SuppressFinalize(this);
}
}
}
}

0 comments on commit c200a23

Please sign in to comment.