diff options
| author | Andrew Reynolds <andrew.j.reynolds@gmail.com> | 2020-02-24 17:28:03 -0600 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-02-24 17:28:03 -0600 |
| commit | a3b9a99404ee00bde5db42aab63ab08df3712ba3 (patch) | |
| tree | 0602557ad2c7edd0f6282a4257a2df3e5668b4d5 | |
| parent | b0fa6b29a1e15b231547eab147b49f2883a139de (diff) | |
Fixes for quantifiers documentation (#3811)
Minor fixes discovered during development of sygus-inst.
| -rw-r--r-- | src/theory/quantifiers/quant_util.h | 3 | ||||
| -rw-r--r-- | src/theory/quantifiers/sygus/synth_conjecture.h | 7 |
2 files changed, 9 insertions, 1 deletions
diff --git a/src/theory/quantifiers/quant_util.h b/src/theory/quantifiers/quant_util.h index 640a62780..723a56415 100644 --- a/src/theory/quantifiers/quant_util.h +++ b/src/theory/quantifiers/quant_util.h @@ -119,7 +119,8 @@ class QuantifiersModule { * * Called once for new quantified formulas q that are pre-registered by the * quantifiers theory, after internal ownership of quantified formulas is - * finalized. This does context-dependent initialization of this module. + * finalized. This does context-independent initialization of this module + * for quantified formula q. */ virtual void registerQuantifier(Node q) {} /** Pre-register quantifier diff --git a/src/theory/quantifiers/sygus/synth_conjecture.h b/src/theory/quantifiers/sygus/synth_conjecture.h index a4f4af2e8..126facfa8 100644 --- a/src/theory/quantifiers/sygus/synth_conjecture.h +++ b/src/theory/quantifiers/sygus/synth_conjecture.h @@ -59,6 +59,13 @@ class EnumValGenerator * Increment this value generator. If this returns false, then we are out of * values. If this returns true, getCurrent(), if non-null, returns the * current term. + * + * Notice that increment() may return true and afterwards it may be the case + * getCurrent() is null. We do this so that increment() does not take too + * much time per call, which can be the case for grammars where it is + * difficult to find the next (non-redundant) term. Returning true with + * a null current term gives the caller the chance to interleave other + * reasoning. */ virtual bool increment() = 0; /** Get the current concrete value generated by this class. */ |