<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Dec 10, 2015 at 2:21 PM Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>  * Including new things into the API: If an LLVM subcomponent has a C API already<br>
>  included, then expanding that C API is acceptable. Adding C API for subcomponents<br>
> -that don't currently have one is fine, and the details of how best to design that<br>
> -API should be discussed on the mailing list.<br>
> +that don't currently have one need to be discussed on the mailing list for design<br>
> +and maintainability feedback prior to implementation.<br>
><br>
<br>
s/need/needs/ no?<br>
<br>
<br></blockquote><div><br></div><div>Sure :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
>  * Documentation: Any changes to the C API are required to be documented in the<br>
>  release notes so that it’s clear to external users who do not follow the project<br>
<br>
More non-ascii :-)<br>
<br></blockquote><div><br></div><div>I think I've fixed it? I can't get a diff redoing it. If it's still there after r255306 feel free to get it :\</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Cheers,<br>
Rafael<br>
</blockquote></div></div>