<div dir="ltr"><div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Mar 2, 2017 7:32 PM, "Emil Velikov" <<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail-m_8772315142652082888quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-m_8772315142652082888quoted-text">On 2 March 2017 at 18:58, Robert Bragg <<a href="mailto:robert@sixbynine.org" target="_blank">robert@sixbynine.org</a>> wrote:<br>
> Adds R/b from Dylan and Makefile fixups from Emil, including fixing race with<br>
> parallel make builds (thanks). Just holding fast on the use of #pragma once<br>
> though :-)<br>
><br>
</div>I think you want the "required=True" for all the fields, but feel free<br>
to do at a later stage.<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">The header and code args are checked like:<br><br>> if args.header:<br>>        header_file = open(args.header, 'w')<br>><br>> if args.code:<br>>        c_file = open(args.code, 'w')<br><br>and the c() and h() funcs are then be NOOPs when there's no open file to write to.<br><br></div><div>I just double checked that's the case and it works as expected to omit either arg.<br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail-m_8772315142652082888quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
As-is<br>
Reviewed-by: Emil Velikov <<a href="mailto:emil.velikov@collabora.com" target="_blank">emil.velikov@collabora.com</a>><br>
<font color="#888888"></font></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Thanks,</div><div dir="auto">- Robert</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail-m_8772315142652082888quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><font color="#888888"><br>
-Emil<br>
</font></blockquote></div><br></div></div></div>
</div>