Fix GN issues identified by MSVC static analysis.

tools/gn/command_format.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "tools/gn/command_format.h"
 
 #include <stddef.h>
 
 #include <sstream>
 
@@ skipping 305 matching lines @@
     const base::StringPiece lhs = ident->value().value();
     if (lhs == "data" || lhs == "datadeps" || lhs == "data_deps" ||
         lhs == "deps" || lhs == "inputs" || lhs == "outputs" ||
         lhs == "public" || lhs == "public_deps" || lhs == "sources") {
       const_cast<ListNode*>(list)->set_prefer_multiline(true);
     }
   }
 }
 
 void Printer::SortIfSourcesOrDeps(const BinaryOpNode* binop) {
-  if (binop && binop->comments() && !binop->comments()->before().empty() &&
+  if (binop->comments() && !binop->comments()->before().empty() &&
       binop->comments()->before()[0].value().as_string() == "# NOSORT") {
     // Allow disabling of sort for specific actions that might be
     // order-sensitive.
     return;
   }
   const IdentifierNode* ident = binop->left()->AsIdentifier();
   const ListNode* list = binop->right()->AsList();
   if ((binop->op().value() == "=" || binop->op().value() == "+=" ||
        binop->op().value() == "-=") &&
       ident && list) {
@@ skipping 402 matching lines @@
   int start_column = CurrentColumn();
   Print(func_call->function().value());
   Print("(");
 
   bool have_block = func_call->block() != nullptr;
   bool force_multiline = false;
 
   const auto& list = func_call->args()->contents();
   const ParseNode* end = func_call->args()->End();
 
-  if (end && end->comments() && !end->comments()->before().empty())
+  if (end->comments() && !end->comments()->before().empty())
     force_multiline = true;
 
   // If there's before line comments, make sure we have a place to put them.
   for (const auto& i : list) {
     if (i->comments() && !i->comments()->before().empty())
       force_multiline = true;
   }
 
   // Calculate the penalties for 3 possible layouts:
   // 1. all on same line;
@@ skipping 313 matching lines @@
         }
         printf("Wrote formatted to '%s'.\n", to_write.AsUTF8Unsafe().c_str());
       }
     }
   }
 
   return 0;
 }
 
 }  // namespace commands