<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Sep 11, 2013 at 8:14 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="im">On Wed, Sep 11, 2013 at 12:41 PM, Nicholas Gill <span dir="ltr"><<a href="mailto:mythagel@gmail.com" target="_blank">mythagel@gmail.com</a>></span> wrote:<br>
</div><div class="gmail_extra"><div class="gmail_quote"><div class="im">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello cfe-dev,<div><br></div><div>At present unknown keys in the compile_commands.json file will cause the JSONCompilationDatabase parser to reject the file.</div>

<div><br></div><div>1. Would a patch to relax this constraint be accepted?</div>
<div>2. Would other consumers of the compile_commands.json be negatively impacted by unknown keys?</div></div></blockquote><div><br></div></div><div>Yes. This can be rather big, and being able to quickly parse it is important for interactive use cases.</div>
<div class="im">
<div> </div></div></div></div></div></blockquote><div><br></div><div>I don't think that this is a very good reason. Parsing the compilation database in the JSON format is going to take O(project size) work, and anything O(project size) is not going to be adequate for interactive use cases anyway.</div>
<div><br></div><div>-- Sean Silva</div></div></div></div>