I don't think so - I only made a PR for the header there is a JADiag.cpp. I mainly wanted to discuss the interface but in hind sight its hard to fully see with the cpp file as well. I'll include the cpp when I make these other changes