<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Nov 23, 2013 at 1:15 PM, Jonathan Schleifer <span dir="ltr"><<a href="mailto:js@webkeks.org" target="_blank">js@webkeks.org</a>></span> wrote:<br>
<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">Am 23.11.2013 um 21:15 schrieb David Majnemer <<a href="mailto:david.majnemer@gmail.com">david.majnemer@gmail.com</a>>:<br>

<div class="im"><br>
> I see no tests in your patch, please add a test.<br>
<br>
</div>Ok, will add one.<br>
<div class="im"><br>
> Also, you seem to allow %k/%K in any context, not just ones where ObjCRuntime::getKind() == ObjFW.  Please fix this.<br>
<br>
</div>This is intentional! ObjFW is by no means limited to the ObjFW runtime, you can use it perfectly fine with the Apple runtime. This is also not limited to ObjC, as in .c files you can also use function like of_asprintf which accept these format specifiers. So while it might be debatable whether to limit it to ObjC files, limiting it to only the ObjFW runtime will result in code that compiles when using ObjFW with the ObjFW runtime, but is rejected by Clang with -Werror with the Apple runtime.<br>
</blockquote><div><br></div><div>We cannot enable this unconditionally. The vast majority of users will not be using ObjFW, allowing %k in all conditions may hide diagnostics that would be useful. It is also a potential problem if POSIX/IEEE Std 1003.1 chose to use 'k'/'K' for something.</div>
<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">
<br>
--<br>
Jonathan</blockquote></div><br></div></div>