<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 19, 2016 at 10:25 AM, Eugene Zelenko via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Eugene.Zelenko added a comment.<br>
<br>
Also I agree that testing is good idea, it doesn't make sense in current incarnation which test only vector and set and only with containers' code snippets instead of real implementations. I wrote about last issue in cfe-dev, but idea was rejected.<br></blockquote><div><br></div><div>Not sure I quite follow why stub implementations wouldn't be a valid test strategy here?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Other issue is how to avoid code duplication for test conditions. Probably tests should be generated from templates for each container or should contain container definition and include shared test body.<br></blockquote><div><br></div><div>Are the tests really long enough to have a lot of duplication worth reducing/removing? Seems like the code should be pretty short...</div></div></div></div>