From 2b8cb3e2515def97edfe555e09e9cd173edd79cd Mon Sep 17 00:00:00 2001
From: Paul Beckingham <paul@beckingham.net>
Date: Tue, 2 Apr 2013 17:41:13 -0400
Subject: [PATCH] Bug

- Fixed a bug so that the report filter order is preserved, to enable an
  optimization that was being skipped.
---
 ChangeLog                  | 2 ++
 src/commands/CmdCustom.cpp | 5 +++--
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index c3077e140..149dce9ce 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -136,6 +136,8 @@ Bugs
    (thanks to Rainer Müller).
  + Fixed hook tests to work on released code (thanks to Jakub Wilk).
  + Fixed bug where arguments passed to 'task execute' needed double escaping.
+ + Fixed a bug so that the report filter order is preserved, to enable an
+   optimization that was being skipped.
 
 ------ old releases ------------------------------
 
diff --git a/src/commands/CmdCustom.cpp b/src/commands/CmdCustom.cpp
index a683c3b03..6e0e5fdd3 100644
--- a/src/commands/CmdCustom.cpp
+++ b/src/commands/CmdCustom.cpp
@@ -86,11 +86,12 @@ int CmdCustom::execute (std::string& output)
   // Prepend the argument list with those from the report filter.
   std::vector <std::string> filterArgs;
   splitq (filterArgs, reportFilter, ' ');
-  std::vector <std::string>::iterator arg;
-  for (arg = filterArgs.begin (); arg != filterArgs.end (); ++ arg)
+  std::vector <std::string>::reverse_iterator arg;
+  for (arg = filterArgs.rbegin (); arg != filterArgs.rend (); ++ arg)
     context.a3.capture_first (*arg);
 
   context.a3.categorize ();
+  context.a3.dump ("A3::categorize");
 
   // Load the data.
   handleRecurrence ();