Skip to content

Commit e059d8e

Browse files
committed
Simplify Constant methods in Absy.cs.
Addresses review comments in PR #767.
1 parent 89643d8 commit e059d8e

File tree

2 files changed

+4
-20
lines changed

2 files changed

+4
-20
lines changed

Source/Core/AST/Absy.cs

Lines changed: 2 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1429,23 +1429,7 @@ public class Constant : Variable
14291429
// from all other constants.
14301430
public readonly bool Unique;
14311431

1432-
public IList<Axiom> DefinitionAxioms => definitionAxioms;
1433-
1434-
private IList<Axiom> definitionAxioms = new List<Axiom>();
1435-
1436-
public bool RemoveDefinitionAxiom(Axiom axiom)
1437-
{
1438-
Contract.Requires(axiom != null);
1439-
1440-
return definitionAxioms.Remove(axiom);
1441-
}
1442-
1443-
public void AddDefinitionAxiom(Axiom axiom)
1444-
{
1445-
Contract.Requires(axiom != null);
1446-
1447-
DefinitionAxioms.Add(axiom);
1448-
}
1432+
public IList<Axiom> DefinitionAxioms { get; set; }
14491433

14501434
public Constant(IToken /*!*/ tok, TypedIdent /*!*/ typedIdent)
14511435
: this(tok, typedIdent, true)
@@ -1468,7 +1452,7 @@ public Constant(IToken /*!*/ tok, TypedIdent /*!*/ typedIdent,
14681452
Contract.Requires(typedIdent.Name != null && typedIdent.Name.Length > 0);
14691453
Contract.Requires(typedIdent.WhereExpr == null);
14701454
this.Unique = unique;
1471-
this.definitionAxioms = definitionAxioms ?? new List<Axiom>();
1455+
this.DefinitionAxioms = definitionAxioms ?? new List<Axiom>();
14721456
}
14731457

14741458
public override bool IsMutable => false;

Source/Core/Monomorphization.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1522,9 +1522,9 @@ void UpdateConstantDependencies(HashSet<Constant> constants)
15221522
{
15231523
foreach (var newAxiom in newAxioms.Where(ax => newAxiomConstants[ax].Contains(constant)))
15241524
{
1525-
constant.AddDefinitionAxiom(newAxiom);
1525+
constant.DefinitionAxioms.Add(newAxiom);
15261526
}
1527-
constant.RemoveDefinitionAxiom(oldAxiom);
1527+
constant.DefinitionAxioms.Remove(oldAxiom);
15281528
}
15291529
}
15301530
}

0 commit comments

Comments
 (0)