Note that the "for testing only" comment goes only in the .cpp file, not the .t.cpp, right? 



---
Sent From Bloomberg Mobile MSG

---- Original Message ----
From: Weiyeh <reply@reply.github.com>
At: 2/14/2013 23:35

In groups/bsl/bslalg/bslalg_arraydestructionprimitives.t.cpp:
>  #include <bslalg_scalarprimitives.h>             // for testing only
>  
> -#include <bslmf_istriviallycopyable.h>           // for testing only

I did a little scan through the 161 Test Drivers in scope and there are 38 Test Drivers ( '.t.cpp' files )  that have the '// for testing only' annotation.  We will update our 'TestDriver Analysis Requirements' and our remediation procedure documentation to ensure this deficiency is remediated along with the 'alphabetize_includes' branch.

Also separately, outside of the 'alphabetize_includes' branch, we will add a deficiency detection line item in 'TestDriver Analysis Requirements' where if  '// for testing only' annotation appears, then the included package must be in the same package as the current component's package and the include must appear in the Group 1 includes of the component '.cpp' file according to BDE C++ Coding Standards Section 4.3.5.

            
              &mdash;

              Reply to this email directly or view it on GitHub.
            
            {https://github.com/notifications/beacon/8p9wzwlzvt2eivdu4muotule-bb55io9wh8ysbo5_mbk7mtroilugoo905lfk8px.gif}