Thanks for the review!
I have committed revision 214086.
I had thought about committing anyway and having a post-commit review, but
I still don't have a long track record so I thought it was safer to ask for
a pre-commit review.
Cheers,
Dario Domizioli
SN Systems - Sony Computer Entertainment Group
On 28 July 2014 15:22, Aaron Ballman <[email protected]> wrote:
> On Mon, Jul 28, 2014 at 10:16 AM, Dario Domizioli
> <[email protected]> wrote:
> >
> > Ping. :-)
> >
> > It's just a test improvement patch so it shouldn't be controversial.
>
> LGTM!
>
> Btw, improved test coverage is something that can usually be dealt
> with post-commit, especially when it's functionality you've been
> involved with before.
>
> ~Aaron
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
