Fix proof of non-inclusion in SMT Proof Verification algorithm - #320
Conversation
This was discussed in #319 (comment) Adds support for non-inclusion proofs to the algorithm description and pseudocode. Fixes some minor syntactic errors (missing semicolons) and slightly improves readability with whitespace. The `cachedZero` pseudocode was also updated to use `n` as the index variable name to better match the longer pseudocode. And today I learned that mdBook's `smart-punctuation` option allows rendering em dashes with triple-hyphens. https://rust-lang.github.io/mdBook/format/markdown.html#smart-punctuation Em dashes looked better than parentheses and commas.
|
Another change I made that I forgot to note: The bit-indexing was replaced with a |
|
on a separate and unrelated note. I think |
Hmm, yeah, we did write it like nonce: OPTIONAL 256-bit nonce generated for each update. MUST be encoded as a string ... I sure don't remember whatever discussion we had last year about why this might be optional or not, but it does seem like we kind of require it now. |
|
@rxgrant had some thoughts on this, specifically. I don't have a problem with it either way. The |
This was discussed in #319 (comment)
Adds support for non-inclusion proofs to the algorithm description and pseudocode. Fixes some minor syntactic errors (missing semicolons) and slightly improves readability with whitespace.
The
cachedZeropseudocode was also updated to usenas the index variable name to better match the longer pseudocode.And today I learned that mdBook's
smart-punctuationoption allows rendering em dashes with triple-hyphens. https://rust-lang.github.io/mdBook/format/markdown.html#smart-punctuation Em dashes looked better than parentheses and commas.