Skip to content

Commit

Permalink
Removed a couple of comments from HashSetState that aren't important …
Browse files Browse the repository at this point in the history
…now that this is only one possible implementation, intended for relatively small states.
  • Loading branch information
sdcondon committed Jun 22, 2024
1 parent 6b36d84 commit 7bbe185
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions src/SCClassicalPlanning/HashSetState.cs
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,6 @@ public HashSetState(Sentence sentence) : this(ConstructionVisitor.Visit(sentence
/// </summary>
public ImmutableHashSet<Predicate> Elements { get; }

// TODO-PERFORMANCE: at some point look at (test me!) efficiency here. ImmutableHashSet builder stuff might be of use?
/// <inheritdoc />
IState IState.Apply(Effect effect) => new HashSetState(Elements.Except(effect.DeleteList).Union(effect.AddList));

Expand Down Expand Up @@ -150,7 +149,6 @@ public override int GetHashCode()
{
var hashCode = new HashCode();

// Again, terrible for large states.
foreach (var element in Elements.OrderBy(e => e.GetHashCode()))
{
hashCode.Add(element);
Expand Down

0 comments on commit 7bbe185

Please sign in to comment.