<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 30, 2014 at 10:48 AM, Jose Fonseca <span dir="ltr"><<a href="mailto:jfonseca@vmware.com" target="_blank">jfonseca@vmware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Series looks good at a glance.  Just one minor remark inline.<br>
<br>
<br>
> + */<br>
> +<br>
> +<br>
> +#ifndef PIGLIT_MATRIX_H<br>
> +#define PIGLIT_MATRIX_H<br>
> +<br>
> +<br>
> +#ifdef __cplusplus<br>
> +extern "C" {<br>
> +#endif<br>
> +<br>
> +<br>
> +#include <stdbool.h><br>
<br>
It's not generally safe to add #includes inside `extern "C" { ... }`, because some system headers may use C++ internally (which would cause an error), and as matter of fact, some windows headers do precisely that.  (Though I'm sure stdbool.h is probably safe.) But as rule of thumb, all #includes should be before `extern "C" { ... }`.<br>
</blockquote><div><br></div><div>Yes, right. Thanks!<br><br></div><div>-Brian<br> <br></div></div><br></div></div>