Re: [PATCH v1] Reports: Convert to C++

2021-07-13 Thread Chris Johns
Hi Ryan, Looks great and thank you for making the changes. I have a minor nit below and with that change the patch to OK to push. There is no need for a fruther review. Thanks Chris On 12/7/21 11:42 pm, Ryan Long wrote: > --- > tester/covoar/ReportsBase.cc | 296 ++-- > tester/covoar/R

[PATCH v1] Reports: Convert to C++

2021-07-12 Thread Ryan Long
--- tester/covoar/ReportsBase.cc | 296 ++-- tester/covoar/ReportsBase.h | 118 +++-- tester/covoar/ReportsHtml.cc | 1074 +- tester/covoar/ReportsHtml.h | 94 ++-- tester/covoar/ReportsText.cc | 261 +- tester/covoar/ReportsText.h |