tidy: A couple of fixes.

1) Add braces around statements.

2) Initialize a variable.

3) Use the string equality operator instead of the compare function.

4) Quiet false positive warning about a potential nullptr passed to
function.  The static analyzers don't know that the argc argument to
main() will always be greater than zero.  Coping the first entry of
argv outside of the for loop eliminates the false positive.

mythtv/programs/mythbackend/mythbackend.cpp

@@ -185,10 +185,11 @@ int main(int argc, char **argv)
     {
         char ** newargv = new char * [argc + 2];
         std::string webonly = "--webonly";
-        int newargc = 0;
-        for (int ix = 0 ; ix < argc ; ++ix)
+        newargv[0] = argv[0];
+        int newargc = 1;
+        for (int ix = 1 ; ix < argc ; ++ix)
         {
-            if (webonly.compare(argv[ix]) != 0)
+            if (webonly != argv[ix])
                 newargv[newargc++] = argv[ix];
         }
         if (retval == 259)

mythtv/programs/mythbackend/servicesv2/v2dvr.cpp

@@ -713,15 +713,17 @@ bool V2Dvr::ReactivateRecording(int RecordedId,
             MythDB::DBError("ReactivateRecording", query);
             return false;
         }
-        int recId;
+        int recId {0};
         if (query.next())
             recId = query.value(0).toInt();
         else
             return false;
         ri = RecordingInfo(recId);
     }
     else
+    {
         throw QString("Recorded ID or Channel ID and StartTime or RecordId are invalid.");
+    }
     if (ri.GetChanID() && ri.HasPathname())
     {
         ri.ReactivateRecording();