Change option deprecated in 32.0 to be removed in 35.0.

This command options was only ever a warning message.  It was added in
32.0 in response to a change to xmltv.  It warns users that the option
is no longer valid in xmltv without rejecting their command, even
though the invocation of xmltv might fail.  Changing this option from
deprecated to removed will cause mythfilldatabase to reject the
argument immediately (although there shouldn't be anyone still using
this option).

mythtv/programs/mythfilldatabase/mythfilldatabase.cpp

@@ -79,10 +79,6 @@ int main(int argc, char *argv[])
     if (retval != GENERIC_EXIT_OK)
         return retval;
 
-    if (cmdline.toBool("ddgraball"))
-        LOG(VB_GENERAL, LOG_WARNING,
-            "Invalid option, see: mythfilldatabase --help dd-grab-all");
-
     if (cmdline.toBool("manual"))
     {
         std::cout << "###\n";

mythtv/programs/mythfilldatabase/mythfilldatabase_commandlineparser.cpp

@@ -139,9 +139,9 @@ void MythFillDatabaseCommandLineParser::LoadArguments(void)
     add("--no-mark-repeats", "markrepeats", true, "do not mark repeats", "");
 
     add("--dd-grab-all", "ddgraball", false, "", "")
-        ->SetDeprecated("It's no longer valid with Schedules Direct XMLTV.\n"
+        ->SetRemoved("It's no longer valid with Schedules Direct XMLTV.\n"
           "          Remove in mythtv-setup General -> Program Schedule\n"
-          "          -> Downloading Options -> Guide Data Arguements");
+                     "          -> Downloading Options -> Guide Data Arguements", "35.0");
     add("--no-resched", "noresched", false,
             "Do not invoke the rescheduler in the backend.",
 	    "This option prevents mythfilldatabase from asking the backend "