compiler warnings generated in catch2 test coverage which are basically unused variables that were left out among some other warnings.
few examples:
/home/runner/work/pufferfish-software/pufferfish-software/firmware/ventilator-controller-stm32/Core/Test/Pufferfish/Protocols/CRCElements.cpp:30:9: warning: typedef ‘using TestParsedCRCElement = Pufferfish::Protocols::ParsedCRCElement<254>’ locally defined but not used [-Wunused-local-typedefs] 30 | using TestParsedCRCElement = PF::Protocols::ParsedCRCElement<buffer_size>; | ^~~~~~~~~~~~~~~~~~~~
/home/runner/work/pufferfish-software/pufferfish-software/firmware/ventilator-controller-stm32/Core/Test/Pufferfish/Protocols/Chunks.cpp: In function ‘void ____C_A_T_C_H____T_E_S_T____0()’: /home/runner/work/pufferfish-software/pufferfish-software/firmware/ventilator-controller-stm32/Core/Test/Pufferfish/Protocols/Chunks.cpp:32:35: warning: unused variable ‘status’ [-Wunused-variable] 32 | PF::Protocols::ChunkInputStatus status; | ^~~~~~
This PR fixes:
few examples:
/home/runner/work/pufferfish-software/pufferfish-software/firmware/ventilator-controller-stm32/Core/Test/Pufferfish/Protocols/CRCElements.cpp:30:9: warning: typedef ‘using TestParsedCRCElement = Pufferfish::Protocols::ParsedCRCElement<254>’ locally defined but not used [-Wunused-local-typedefs] 30 | using TestParsedCRCElement = PF::Protocols::ParsedCRCElement<buffer_size>; | ^~~~~~~~~~~~~~~~~~~~/home/runner/work/pufferfish-software/pufferfish-software/firmware/ventilator-controller-stm32/Core/Test/Pufferfish/Protocols/Chunks.cpp: In function ‘void ____C_A_T_C_H____T_E_S_T____0()’: /home/runner/work/pufferfish-software/pufferfish-software/firmware/ventilator-controller-stm32/Core/Test/Pufferfish/Protocols/Chunks.cpp:32:35: warning: unused variable ‘status’ [-Wunused-variable] 32 | PF::Protocols::ChunkInputStatus status; | ^~~~~~The full log can be viewed here - under 'Run make'