<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 21, 2015 at 6:18 PM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@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 class="HOEnZb adM"><div class="">> On 2015-Jan-21, at 16:39, Chandler Carruth <<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>> wrote:<br>
><br>
><br>
> On Wed, Jan 21, 2015 at 3:45 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>> wrote:<br>
> [canonicalization] Refactor how we create new stores into a helper<br>
> function. This is a bit tidier anyways and will make a subsquent patch<br>
> simpler as I want to add another case to this combine.<br>
><br>
> I didn't end up needing this, but it seems cleaner to leave it in. Let me know if anyone prefers the old code.<br>
<br>
</div></div>New code looks better to me.  Might be cleaner without the return value<br>
if you're not using it, though.</blockquote></div><br>I lied, and will end up using it after all. Just need to move it in my next commit.</div><div class="gmail_extra"><br></div><div class="gmail_extra">I'm still not using the return value, but it felt weird to not return it. I can definitely imagine a caller in the future wanting it, so just wanted to make the utility function useful to them if/when they arrive as it adds essentially no complexity.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Still, I'm happy to remove it if that's your preference. Just let me know.</div></div>