gn desc: printing public_deps without --all and --tree

tools/gn/target.cc

 // Copyright (c) 2013 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/target.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 
@@ skipping 400 matching lines @@
          output_type_ == SHARED_LIBRARY ||
          output_type_ == LOADABLE_MODULE ||
          output_type_ == ACTION ||
          output_type_ == ACTION_FOREACH ||
          output_type_ == COPY_FILES ||
          output_type_ == CREATE_BUNDLE ||
          (output_type_ == STATIC_LIBRARY && complete_static_lib_);
 }
 
 DepsIteratorRange Target::GetDeps(DepsIterationType type) const {
   if (type == DEPS_LINKED) {

[brettw, 2017/05/23 17:23:05]

With the additional enum types, I think this function would be more readable as
a switch statement. Can you change it?

[mbonadei1, 2017/05/26 09:29:44]

Done.

     return DepsIteratorRange(DepsIterator(
         &public_deps_, &private_deps_, nullptr));
   }
+  if (type == DEPS_PRIVATE) {
+    return DepsIteratorRange(
+        DepsIterator(&private_deps_, &data_deps_, nullptr));
+  }
+  if (type == DEPS_PUBLIC) {
+    return DepsIteratorRange(DepsIterator(&public_deps_, nullptr, nullptr));
+  }
   // All deps.
   return DepsIteratorRange(DepsIterator(
       &public_deps_, &private_deps_, &data_deps_));
 }
 
 std::string Target::GetComputedOutputName() const {
   DCHECK(toolchain_)
       << "Toolchain must be specified before getting the computed output name.";
 
   const std::string& name = output_name_.empty() ? label().name()
@@ skipping 420 matching lines @@
     check_data_deps = g_scheduler->IsFileGeneratedByWriteRuntimeDeps(out_file);
     // Check object files (much slower and very rare) only if the "normal"
     // output check failed.
     consider_object_files = !check_data_deps;
     if (!EnsureFileIsGeneratedByDependency(this, out_file, true,
                                            consider_object_files,
                                            check_data_deps, &seen_targets))
       g_scheduler->AddUnknownGeneratedInput(this, source);
   }
 }