Skip to content

Commit 5009c5e

Browse files
committed
explain what not complete (but sound) means
1 parent 0480a9e commit 5009c5e

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

src/lib/provable/dynamic-array.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,9 @@ class DynamicArrayBase<ProvableValue = any, Value = any> {
189189
* the array.
190190
*
191191
* Note: The correct type for `i` is actually UInt16 which doesn't exist. The
192-
* method is not complete (but sound) for i >= 2^16.
192+
* method is not complete (but sound) for i >= 2^16. This means that if the
193+
* index is larger than 2^16, the constraints could be satisfiable but the
194+
* result is not correct (because the capacity can at most be 2^16).
193195
*/
194196
getOption(i: Field): Option<ProvableValue> {
195197
let type = this.innerType;

0 commit comments

Comments
 (0)