Attention is currently required from: Anastasia Klimchuk, Nikolai Artemiev, Thomas Heijligen.
4 comments:
File flashrom.c:
Can this be a macro in flash. […]
Done. I also moved the declaration of this into flash.h, alongside the other `g_test_*_injector`s so it should trigger a compile-time error if somebody changes the definition without also changing the declaration.
File include/flash.h:
Patch Set #5, Line 325: TEST_ERASE_INJECTOR
Maybe remove this value? I know some existing tests are using it, but they can be adjusted. […]
Done
File tests/erase_func_algo.c:
Patch Set #5, Line 198: block_erase_chip, block_erase_chip, block_erase_chip, block_erase_chip, block_erase_chip
What if […]
I had avoided doing that just because it requires updating all of the existing test cases, but done now. It does seem worth verifying that, now that we have the capability.
File tests/meson.build:
Patch Set #5, Line 127: '-DFLASHROM_TEST',
Just to test , have you built with this option undefined? […]
Yes, building the regular flashrom binary still works.
I added this because it may not be clear that the features it currently guards are intended only to enable unit testing. Guarding them with a symbol that we only define when building tests makes that clearer, and doesn't add much complexity.
To view, visit change 82251. To unsubscribe, or for help writing mail filters, visit settings.