<div dir="ltr">I like your idea of using tyConHash for the Int version and tyConFingerprint to refer to the Fingerprint. <div><br></div><div>The former fits more closely with the usage elsewhere e.g. hashUnique, hashStableName.<div><div><br></div><div>-Edward</div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 11, 2015 at 5:39 AM, Simon Peyton Jones <span dir="ltr"><<a href="mailto:simonpj@microsoft.com" target="_blank">simonpj@microsoft.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-GB" link="#0563C1" vlink="#954F72">
<div>
<p>data TyCon = TyCon {<u></u><u></u></p>
<p>   tyConHash    :: {-# UNPACK #-} !Fingerprint, -- ^ @since 4.8.0.0<u></u><u></u></p>
<p>   tyConPackage :: String, -- ^ @since 4.5.0.0<u></u><u></u></p>
<p>   tyConModule  :: String, -- ^ @since 4.5.0.0<u></u><u></u></p>
<p>   tyConName    :: String  -- ^ @since 4.5.0.0<u></u><u></u></p>
<p>}<u></u><u></u></p>
<p class="MsoNormal"><span style="font-size:12.0pt"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:12.0pt">Friends,<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:12.0pt">Is tyConHash a good name here?  Wouldn’t tyConFingerprint be better? 
<u></u><u></u></span></p>
<p><u></u><span style="font-size:12.0pt;font-family:Symbol"><span>·<span style="font:7.0pt "Times New Roman"">        
</span></span></span><u></u><span style="font-size:12.0pt">Hash functions usually yield a Int.<u></u><u></u></span></p>
<p><u></u><span style="font-size:12.0pt;font-family:Symbol"><span>·<span style="font:7.0pt "Times New Roman"">        
</span></span></span><u></u><span style="font-size:12.0pt">tyConFingerprint :: TyCon -> Fingerprint makes the name match the type.<u></u><u></u></span></p>
<p><u></u><span style="font-size:12.0pt;font-family:Symbol"><span>·<span style="font:7.0pt "Times New Roman"">        
</span></span></span><u></u><span style="font-size:12.0pt">If we had fingerprintHash:: Fingerprint -> Int, then we might want<br>
tyConHash :: TyCon -> Int<br>
tyConHash = fingerprintHash . tyConFingerpring<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:12.0pt"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:12.0pt">This is new in 7.10, so we could fix it now with no trouble.<span class="HOEnZb"><font color="#888888"><u></u><u></u></font></span></span></p><span class="HOEnZb"><font color="#888888">
<p class="MsoNormal"><span style="font-size:12.0pt">Simon<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:12.0pt"><u></u> <u></u></span></p>
</font></span></div><span class="HOEnZb"><font color="#888888">
</font></span></div><span class="HOEnZb"><font color="#888888">


<p></p>

-- <br>
You received this message because you are subscribed to the Google Groups "haskell-core-libraries" group.<br>
To unsubscribe from this group and stop receiving emails from it, send an email to <a href="mailto:haskell-core-libraries+unsubscribe@googlegroups.com" target="_blank">haskell-core-libraries+unsubscribe@googlegroups.com</a>.<br>
For more options, visit <a href="https://groups.google.com/d/optout" target="_blank">https://groups.google.com/d/optout</a>.<br>
</font></span></blockquote></div><br></div>