Hello, Andrey!

Thanks for the review!

> I think usually write only commit year. Something tells me you can safely
write 2025 there.
Done.

> Can't wrap my head why do you need this?
Oops, copy-paste, fixed.

> I think this comment describes behavior before the fix in present tense.
Fixed.

> Snapshot business seems incorrect to me here...
Hm, it seems like it is correct. `snapshot` variable is deleted, we only
use `state->snapshot` now (if it is required at all).

Best regards,
Mikhail.

Attachment: v2-0001-amcheck-Fix-bt_index_parent_check-behavior-with-C.patch
Description: Binary data

Reply via email to