[flashrom-gerrit] [L] Change in flashrom[main]: tests: Unit tests for erase function selection algo

[Anastasia Klimchuk (Code Review)]

Attention is currently required from: Aarya, Edward O'Callaghan, Nikolai Artemiev, Peter Marheine, Simon Buhrow, Thomas Heijligen.

Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/67535?usp=email )

Change subject: tests: Unit tests for erase function selection algo
......................................................................

Patch Set 21:

(1 comment)

File tests/erase_func_algo.c:

https://review.coreboot.org/c/flashrom/+/67535/comment/a68e8635_832fd787 :
PS20, Line 62: 	struct test_case* current_test_case; /* Currently executed test case. */
> Can this be const, or is there mutability hidden inside it? (If there is, can it be moved into this  […]

There was a hidden mutability indeed, `eraseblocks_actual` and `eraseblocks_actual_ind` ! But I moved it into struct all_state, where it can live together with operational `buf`.
So now yes, the `current_test_case` can point to constant. Thank you for good idea.