Deprecation

- Removed deprecated 'report.X.limit' feature.

ChangeLog

@@ -140,6 +140,7 @@
 - Display debug/release build type in the diagnostics command.
 - Removed obsolete task-faq.5 and task-tutorial.5 man pages.
 - French localization (thanks to leowzukw).
+- Removed deprecated 'report.X.limit' feature.
 
 ------ current release ---------------------------
 

NEWS

@@ -46,6 +46,8 @@ Removed features in 2.4.0
   - Removed the 'tasksh' program from the Taskwarrior distribution. It is now
     a separate project.
   - Removed the 'complete.all.projects' setting that was not used.
+  - Removed support for 'report.X.limit', which can now be set in the report
+    filter 'limit:N'.
 
 Known Issues
 

doc/man/taskrc.5.in

@@ -1199,11 +1199,6 @@ report. See the
 .B annotations
 variable for details on the possible values.  Deprecated.
 
-.TP
-.B report.X.limit
-An optional value to a report limiting the number of displayed tasks in the
-generated report.  Deprecated.
-
 .TP
 Taskwarrior comes with a number of predefined reports, which are:
 

src/legacy.cpp

@@ -88,12 +88,6 @@ std::string legacyCheckForDeprecatedVariables ()
   std::map <std::string, std::string>::const_iterator it;
   for (it = context.config.begin (); it != context.config.end (); ++it)
   {
-    // report.*.limit
-    if (it->first.length () > 13 &&
-        it->first.substr (0, 7) == "report." &&
-        it->first.substr (it->first.length () - 6) == ".limit")
-      deprecated.push_back (it->first);
-
     // report.*.annotations
     if (it->first.length () > 19 &&
         it->first.substr (0, 7) == "report." &&