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

2021-07-08 Thread Chris Johns
On 9/7/21 3:06 am, Alex White wrote: > On Wed, Jul 7, 2021 at 6:55 PM Chris Johns wrote: >> >> On 7/7/21 11:37 pm, Ryan Long wrote: >> > I'll get those pointers changed to references, and remove the whitespace >> > changes. Is there a particular reason to not use '\n' instead of std::endl? >> >>

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

2021-07-08 Thread Alex White
On Wed, Jul 7, 2021 at 6:55 PM Chris Johns wrote: > > On 7/7/21 11:37 pm, Ryan Long wrote: > > I'll get those pointers changed to references, and remove the whitespace > > changes. Is there a particular reason to not use '\n' instead of std::endl? > > Awesome and thanks. > > > I read that std::e

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

2021-07-07 Thread Chris Johns
On 7/7/21 11:37 pm, Ryan Long wrote: > I'll get those pointers changed to references, and remove the whitespace > changes. Is there a particular reason to not use '\n' instead of std::endl? Awesome and thanks. > I read that std::endl is slower since it's flushing the buffer each time that > it

RE: [PATCH] Reports: Convert to C++

2021-07-07 Thread Ryan Long
: Chris Johns Sent: Tuesday, July 6, 2021 7:13 PM To: Ryan Long ; devel@rtems.org Subject: Re: [PATCH] Reports: Convert to C++ On 7/7/21 1:53 am, Ryan Long wrote: > Made formatting consistent and converted C code to C++. > --- > tester/covoar/ReportsBase.cc | 525 +-- &

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

2021-07-06 Thread Chris Johns
On 7/7/21 1:53 am, Ryan Long wrote: > Made formatting consistent and converted C code to C++. > --- > tester/covoar/ReportsBase.cc | 525 +-- > tester/covoar/ReportsBase.h | 156 +++--- > tester/covoar/ReportsHtml.cc | 1183 > +- > tester