<div dir="ltr"><div dir="ltr"><div dir="ltr">Hi<div><br></div><div>There has been a lot of talk about making covoar use more C++ </div><div>features. It seems to be an issue on every patch. I almost </div><div>replied to Gedare's comment at the bottom of a patch </div><div>but decided it needed another thread:<br><br>"I still struggle reviewing this codebase, in part because it is C+C++</div>(TM) and in part because I'm not so proficient in C++ to make concrete<br>recommendations how to write this better. I think, if the goal is<br>eventually to make this more C++ like code, then new code coming in<br>should aim to move the needle in that direction rather than continue<br>to propagate the old ways of doing."</div><div dir="ltr"><br></div><div dir="ltr">I personally do NOT want to see changes to C++ in one leaf class and</div><div dir="ltr">the other architectures not get the same changes. I would prefer to see </div><div dir="ltr">all these corrections and base changes in the same style with limited</div><div>changes to C-isms. I'm not opposed to the changes but let's take the</div><div>Target class one. There are multiple target classes. Changing one </div><div>independent of the others isn't a good idea.</div><div><br></div><div>I'd like to see us get a working baseline in and then do something like</div><div>sweep std::string through Target* as a single patch. This is easier to</div><div>test and review since it would only be C string to std::string. Perhaps </div><div>switch to C++ output a file at a time. Redo the report output. Etc. </div><div>Discrete chunks instead of piecemeal.</div><div><br></div><div>Covoar has spent years broken and some is from changing working </div><div>things to do things "a better way" with no baseline to check against.</div><div>We need to get a baseline.</div><div><br></div><div>Please. Let's get a working baseline and then approach this more </div><div>methodically. No one is going to suffer from seeing a C string a little</div><div>while longer. :)</div><div><br></div><div>Thanks.</div><div><br></div><div>--joel</div><div dir="ltr"><br><div class="gmail-yj6qo gmail-ajU" style="outline:none;padding:10px 0px;width:22px;margin:2px 0px 0px"><br class="gmail-Apple-interchange-newline"></div></div></div></div>