On 07/29/2013 10:37 AM, Tim Shen wrote:
On Mon, Jul 29, 2013 at 4:26 PM, Paolo Carlini <paolo.carl...@oracle.com> wrote:
Minor nit: it's not clear to me why in the previous patch you added the
includes of <map> and <queue>.
I use them in regex_grep_matcher.h and regex_grep_matcher.tcc; Is
include/std/regex the right place where I include them?
I guess it's the right place yes. Sorry, I didn't notice the uses (when I think "refactoring", I don't think two more std:: containers are needed ;)

Paolo.

Reply via email to