From 6f01d91975fda19667b189edb056db2b60e16a82 Mon Sep 17 00:00:00 2001
From: firewave <firewave@users.noreply.github.com>
Date: Mon, 14 Aug 2023 23:54:29 +0200
Subject: [PATCH] CmdLineParser: deprecated `--template <template>` and
 `--template-location <template>`

---
 cli/cmdlineparser.cpp      |  4 ++--
 releasenotes.txt           |  4 +++-
 test/testcmdlineparser.cpp | 26 +++++++++++++++-----------
 3 files changed, 20 insertions(+), 14 deletions(-)

diff --git a/cli/cmdlineparser.cpp b/cli/cmdlineparser.cpp
index f7826fef540..e2a575533cc 100644
--- a/cli/cmdlineparser.cpp
+++ b/cli/cmdlineparser.cpp
@@ -899,7 +899,6 @@ bool CmdLineParser::parseFromArgs(int argc, const char* const argv[])
                 }
             }
 
-            // TODO: deprecate "--template <template>"
             // Output formatter
             else if (std::strcmp(argv[i], "--template") == 0 ||
                      std::strncmp(argv[i], "--template=", 11) == 0) {
@@ -907,6 +906,7 @@ bool CmdLineParser::parseFromArgs(int argc, const char* const argv[])
                 if (argv[i][10] == '=')
                     mSettings.templateFormat = argv[i] + 11;
                 else if ((i+1) < argc && argv[i+1][0] != '-') {
+                    printMessage("'--template <template>' is deprecated and will be removed in 2.13 - please use '--template=<template>' instead");
                     ++i;
                     mSettings.templateFormat = argv[i];
                 } else {
@@ -935,13 +935,13 @@ bool CmdLineParser::parseFromArgs(int argc, const char* const argv[])
                 }
             }
 
-            // TODO: deprecate "--template-location <template>"
             else if (std::strcmp(argv[i], "--template-location") == 0 ||
                      std::strncmp(argv[i], "--template-location=", 20) == 0) {
                 // "--template-location format"
                 if (argv[i][19] == '=')
                     mSettings.templateLocation = argv[i] + 20;
                 else if ((i+1) < argc && argv[i+1][0] != '-') {
+                    printMessage("'--template-location <template>' is deprecated and will be removed in 2.13 - please use '--template-location=<template>' instead");
                     ++i;
                     mSettings.templateLocation = argv[i];
                 } else {
diff --git a/releasenotes.txt b/releasenotes.txt
index b535a56c4b0..57e6d0210fa 100644
--- a/releasenotes.txt
+++ b/releasenotes.txt
@@ -1,4 +1,4 @@
-release notes for cppcheck-2.12
+Release Notes for Cppcheck 2.12
 
 New checks:
 - uselessOverride finds overriding functions that either duplicate code from or delegate back to the base class implementation
@@ -14,6 +14,8 @@ Changed interface:
 
 Deprecations:
 - The qmake build system has been deprecated and will be removed in a future version.
+- Command-line option '--template <template>' is deprecated and will be removed in 2.13 - please use '--template=<template>' instead.
+- Command-line option '--template-location <template>' is deprecated and will be removed in 2.13 - please use '--template-location=<template>' instead.
 
 Other:
 - "USE_QT6=On" will no longer fallback to Qt5 when Qt6 is not found.
diff --git a/test/testcmdlineparser.cpp b/test/testcmdlineparser.cpp
index d60006ccd5f..54e844f899d 100644
--- a/test/testcmdlineparser.cpp
+++ b/test/testcmdlineparser.cpp
@@ -1335,10 +1335,10 @@ class TestCmdlineParser : public TestFixture {
 
     void templates() {
         REDIRECT;
-        const char * const argv[] = {"cppcheck", "--template", "{file}:{line},{severity},{id},{message}", "--template-location={file}:{line}:{column} {info}", "file.cpp"};
+        const char * const argv[] = {"cppcheck", "--template={file}:{line},{severity},{id},{message}", "--template-location={file}:{line}:{column} {info}", "file.cpp"};
         settings->templateFormat.clear();
         settings->templateLocation.clear();
-        ASSERT(parser->parseFromArgs(5, argv));
+        ASSERT(parser->parseFromArgs(4, argv));
         ASSERT_EQUALS("{file}:{line},{severity},{id},{message}", settings->templateFormat);
         ASSERT_EQUALS("{file}:{line}:{column} {info}", settings->templateLocation);
         ASSERT_EQUALS("", GET_REDIRECT_OUTPUT);
@@ -1346,10 +1346,10 @@ class TestCmdlineParser : public TestFixture {
 
     void templatesGcc() {
         REDIRECT;
-        const char * const argv[] = {"cppcheck", "--template", "gcc", "file.cpp"};
+        const char * const argv[] = {"cppcheck", "--template=gcc", "file.cpp"};
         settings->templateFormat.clear();
         settings->templateLocation.clear();
-        ASSERT(parser->parseFromArgs(4, argv));
+        ASSERT(parser->parseFromArgs(3, argv));
         ASSERT_EQUALS("{file}:{line}:{column}: warning: {message} [{id}]\n{code}", settings->templateFormat);
         ASSERT_EQUALS("{file}:{line}:{column}: note: {info}\n{code}", settings->templateLocation);
         ASSERT_EQUALS("", GET_REDIRECT_OUTPUT);
@@ -1357,10 +1357,10 @@ class TestCmdlineParser : public TestFixture {
 
     void templatesVs() {
         REDIRECT;
-        const char * const argv[] = {"cppcheck", "--template", "vs", "file.cpp"};
+        const char * const argv[] = {"cppcheck", "--template=vs", "file.cpp"};
         settings->templateFormat.clear();
         settings->templateLocation.clear();
-        ASSERT(parser->parseFromArgs(4, argv));
+        ASSERT(parser->parseFromArgs(3, argv));
         ASSERT_EQUALS("{file}({line}): {severity}: {message}", settings->templateFormat);
         ASSERT_EQUALS("", settings->templateLocation);
         ASSERT_EQUALS("", GET_REDIRECT_OUTPUT);
@@ -1368,10 +1368,10 @@ class TestCmdlineParser : public TestFixture {
 
     void templatesEdit() {
         REDIRECT;
-        const char * const argv[] = {"cppcheck", "--template", "edit", "file.cpp"};
+        const char * const argv[] = {"cppcheck", "--template=edit", "file.cpp"};
         settings->templateFormat.clear();
         settings->templateLocation.clear();
-        ASSERT(parser->parseFromArgs(4, argv));
+        ASSERT(parser->parseFromArgs(3, argv));
         ASSERT_EQUALS("{file} +{line}: {severity}: {message}", settings->templateFormat);
         ASSERT_EQUALS("", settings->templateLocation);
         ASSERT_EQUALS("", GET_REDIRECT_OUTPUT);
@@ -1439,7 +1439,9 @@ class TestCmdlineParser : public TestFixture {
         ASSERT(!parser->parseFromArgs(3, argv));
         ASSERT_EQUALS("file.cpp", settings->templateFormat);
         ASSERT_EQUALS("", settings->templateLocation);
-        TODO_ASSERT_EQUALS("cppcheck: error: argument to '--template' is missing.\n", "cppcheck: error: no C or C++ source files found.\n", GET_REDIRECT_OUTPUT);
+        TODO_ASSERT_EQUALS("cppcheck: error: argument to '--template' is missing.\n",
+                           "cppcheck: '--template <template>' is deprecated and will be removed in 2.13 - please use '--template=<template>' instead\n"
+                           "cppcheck: error: no C or C++ source files found.\n", GET_REDIRECT_OUTPUT);
     }
 
     // will use the default
@@ -1462,7 +1464,7 @@ class TestCmdlineParser : public TestFixture {
         ASSERT_EQUALS("cppcheck: error: argument to '--template-location' is missing.\n", GET_REDIRECT_OUTPUT);
     }
 
-    // TODO: will not error out as he next option does not start with a "-"
+    // TODO: will not error out as the next option does not start with a "-"
     void templateLocationInvalid2() {
         REDIRECT;
         settings->templateFormat.clear();
@@ -1471,7 +1473,9 @@ class TestCmdlineParser : public TestFixture {
         ASSERT(!parser->parseFromArgs(3, argv));
         ASSERT_EQUALS("{file}:{line}:{column}: {inconclusive:}{severity}:{inconclusive: inconclusive:} {message} [{id}]\n{code}", settings->templateFormat);
         ASSERT_EQUALS("file.cpp", settings->templateLocation);
-        TODO_ASSERT_EQUALS("", "cppcheck: error: no C or C++ source files found.\n", GET_REDIRECT_OUTPUT);
+        TODO_ASSERT_EQUALS("",
+                           "cppcheck: '--template-location <template>' is deprecated and will be removed in 2.13 - please use '--template-location=<template>' instead\n"
+                           "cppcheck: error: no C or C++ source files found.\n", GET_REDIRECT_OUTPUT);
     }
 
     // will use the default