If both of those aren't valid strings in the domain you're using this map for, sure. (pedantry: StringRef("", 1) would suffice, since StringRef's don't need to be null terminated :))<br><br><div>On Fri Dec 20 2013 at 12:10:12 PM, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>> wrote:</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Or can I use StringRef() as an empty key and StringRef("\0", 1) as a tombstone?<br>
<br>
2013/12/21 Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>>:<br>
> 2013/12/20 <a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a> <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>>:<br>
>> If you don't need null values, you could have <null, ""> be the empty key<br>
>> and <"", null> be the tombstone, or something like that.<br>
><br>
> Or I can use an explicit second DenseMap template parameter... will<br>
> give it a try.<br>
><br>
> Would appreciate if you continue to review the rest of the patch<br>
> assuming I'll fix this header soon :)<br>
</blockquote>