<div dir="ltr"><div class="gmail_extra">Nits:</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 9, 2015 at 11:23 AM, Ben Langmuir <span dir="ltr"><<a href="mailto:blangmuir@apple.com" target="_blank">blangmuir@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":515" class="a3s" style="overflow:hidden">+  char hostname[256];<br>
+  hostname[255] = 0;<br>
+  hostname[0] = 0;<br>
+  gethostname(hostname, 255);<br></div></blockquote><div><br></div><div>It would be good to document that we want to actively ignore any error here.</div><div><br></div><div>Also, pedantically you should set [255] to 0 here rather than above. POSIX doesn't require null termination in the event it is truncated.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":515" class="a3s" style="overflow:hidden">
+  code = hash_combine(code, StringRef(hostname));<br>
+#endif</div></blockquote></div><br><br></div></div>