<div dir="ltr"><div>On Tue, Oct 6, 2015 at 3:36 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Split <math.h> out of <cmath>. This is a big change, but the same pattern as the prior ones.</div><div dir="ltr"><br></div></blockquote>In this patch, you replicate the #ifdef XXX, __libcpp_XXX, #undef XXX dance for all the isXXX functions. Is that because they're not required to be actual functions in a C library?<div><br></div><div>Other than that Q, LGTM. Like the extended tests.</div><div><br></div></div>-- Marshall</div></div>