[PATCH] D43667: [clang-doc] Implement a YAML generator

[Athos via Phabricator via cfe-commits]

Athosvk added a comment.

I'm a bit late on this, but I'd say that YAML is usually not a 'final' format. What would be the use-cases for this? And if is meant as an alternative intermediate format, why not instead of having one big file, have one file per input file? Just wondering what the particular motivation for that could be



================
Comment at: clang-doc/generators/YAMLGenerator.cpp:159
+    if (!C->Position.empty()) IO.mapRequired("Position", C->Position);
+    if (!C->Children.empty()) IO.mapRequired("Children", C->Children);
+  }
----------------
You should easily be able to unify these 'empty' checks


https://reviews.llvm.org/D43667