<div dir="ltr">On Thu, Aug 1, 2013 at 4:30 PM, Konstantin Tokarev <span dir="ltr"><<a href="mailto:annulen@yandex.ru" target="_blank">annulen@yandex.ru</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
01.08.2013, 17:59, "Manuel Klimek" <<a href="mailto:klimek@google.com">klimek@google.com</a>>:<br>
<div class="im">> On Thu, Aug 1, 2013 at 3:06 PM, Vane, Edwin <<a href="mailto:edwin.vane@intel.com">edwin.vane@intel.com</a>> wrote:<br>
>>> -----Original Message-----<br>
>>> From: Alex Rosenberg [mailto:<a href="mailto:alexr@ohmantics.com">alexr@ohmantics.com</a>]<br>
>>> Sent: Thursday, August 01, 2013 1:32 AM<br>
>>> To: Vane, Edwin<br>
>>> Cc: Clang Dev List (<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a>)<br>
>>> Subject: Re: [cfe-dev] RFC: YAML as an intermediate format for<br>
>>> clang::tooling::Replacement data on disk<br>
>><br>
>> ...<br>
>><br>
>>> I have wanted for some time for rewriter tools to use diff output that can be<br>
>>> used with existing review tools. If a merge tool is created that generates diff, I'd<br>
>>> probably be less concerned, but I'd still want a way to handle this self-contained<br>
>>> in the LLVM/Clang frameworks without a separate merge process.<br>
>><br>
>> This can still happen. What you're describing is orthogonal to this proposal about using YAML as an intermediate representation of serialized data between the migrator and replacement coalescing tool. I don't think anything here will block your desire from coming true.<br>

>><br>
>> Speaking of JSON, this was suggested on IRC. It seems like it would be just as fine as YAML for this situation since there's no need for references or comments (yet, anyway). However, LLVM has no generic JSON parser, just a specific implementation for compilation databases. The YAML reader/writer that LLVM provides is completely generic and available right now even if it provides some features we don't need.<br>

><br>
> Note that the compilation databases use the YAML parser. JSON is a subset of YAML. I'd also (slightly) prefer to use JSON over YAML for the intermediate representation.<br>
<br>
</div>Why? YAML has less syntactic overhead.<br></blockquote><div><br></div><div>JSON is significantly simpler, and there's only "one way" to express something. That will be all I contribute to the bike-shedding - I also think it doesn't really matter much ...</div>
</div></div></div>