From 12941488ce7fdbced9839116dc320bbcee6ad2e0 Mon Sep 17 00:00:00 2001 From: Paul Beckingham Date: Sat, 25 Oct 2014 13:44:56 -0400 Subject: [PATCH] CmdCustom - Removed redundant pair of parentheses from custom report filters. Only one pair is needed. --- src/commands/CmdCustom.cpp | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/src/commands/CmdCustom.cpp b/src/commands/CmdCustom.cpp index 1db9bf765..b6894de44 100644 --- a/src/commands/CmdCustom.cpp +++ b/src/commands/CmdCustom.cpp @@ -81,18 +81,6 @@ int CmdCustom::execute (std::string& output) split (sortOrder, reportSort, ','); validateSortColumns (sortOrder); -/* - TODO Wow, this addition causes memory errors. - - // Surround the command-line filter with parentheses, to protect it from - // the 'and' placed between the report filter and the command line filter. - context.parser.captureFirst ("("); - context.parser.captureLast (")"); -*/ - - if (reportFilter != "") - context.cli.add ("("); - // Prepend the argument list with those from the report filter. std::string lexeme; Lexer::Type type; @@ -105,9 +93,6 @@ int CmdCustom::execute (std::string& output) context.cli.add (lexeme); } - if (reportFilter != "") - context.cli.add (")"); - std::vector ::reverse_iterator arg; for (arg = filterArgs.rbegin (); arg != filterArgs.rend (); ++ arg) context.parser.captureFirst (*arg);