Fix the FP introduced in this PR

Author: pfultz2

lib/checkother.cpp

@@ -3351,6 +3351,10 @@ bool CheckOtherImpl::testIfNonZeroExpressionIsPositive(const Token *tok, const V
         return false;
     }
 
+    if (const Variable* var = nonZeroExpr->variable())
+        if (var->typeStartToken()->isTemplateArg())
+            return false;
+
     const ValueType* vt = nonZeroExpr->valueType();
     return vt && (vt->pointer || vt->sign == ValueType::UNSIGNED);
 }

test/testother.cpp

@@ -9641,6 +9641,15 @@ class TestOther : public TestFixture {
               "}\n");
         ASSERT_EQUALS("", errout_str());
 
+        // template parameter type is deduced as unsigned from the call
+        check("template<class T> bool pos(T x) { return x >= 0; }\n"
+              "void use() { unsigned u = 3; pos(u); }\n");
+        ASSERT_EQUALS("", errout_str());
+
+        check("template<class T> bool neg(T x) { return 0 <= x; }\n"
+              "void use() { unsigned u = 3; neg(u); }\n");
+        ASSERT_EQUALS("", errout_str());
+
         // #13734
         check("void f() {\n"
               "    uint8_t a[N + 1];\n"