Hello,<div><br></div><div>Attached is my new and improved patch to add a Data instance to Data.Text. The patch just adds:</div><div><br></div><div><div>+-- This instance preserves data abstraction at the cost of inefficiency.</div>
<div>+-- We omit reflection services for the sake of data abstraction.</div><div>+</div><div>+instance Data Text where</div><div>+  gfoldl f z txt = z pack `f` (unpack txt)</div><div>+  toConstr _     = error &quot;toConstr&quot;</div>
<div>+  gunfold _ _    = error &quot;gunfold&quot;</div><div>+  dataTypeOf _   = mkNoRepType &quot;Data.Text.Text&quot;</div><div><br></div><div><br></div><div>Which is based on what the Data instances for Set and Map do:</div>
<div><br></div><div><a href="http://www.haskell.org/ghc/docs/latest/html/libraries/containers-0.3.0.0/src/Data-Set.html">http://www.haskell.org/ghc/docs/latest/html/libraries/containers-0.3.0.0/src/Data-Set.html</a></div>
<div><a href="http://www.haskell.org/ghc/docs/latest/html/libraries/containers-0.3.0.0/src/Data-Map.html">http://www.haskell.org/ghc/docs/latest/html/libraries/containers-0.3.0.0/src/Data-Map.html</a></div><div><br></div>
<div>Yay for cargo culting!</div><div><br></div></div><div>It seems like this is better than nothing, possibly the correct answer, and if someone does decide to add better instances for toConstr and gunfold in the future, nothing should break? For happstack-data, I think we only need dataTypeOf. </div>
<div><br></div><div>The instance I posted before definitely did not have valid toConstr / gunfold instances, so I think we would have noticed if we were actually trying to use them..</div><div><br></div><div>- jeremy<br><br>
<div class="gmail_quote">On Fri, Jan 22, 2010 at 4:24 PM, Jeremy Shaw <span dir="ltr">&lt;<a href="mailto:jeremy@n-heptane.com">jeremy@n-heptane.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hello,<div><br></div><div>Would it be possible to get a Data instance for Data.Text.Text? This would allow us to create a Serialize instance of Text for use with happstack -- which would be extremely useful.</div><div><br>

</div><div>We (at seereason) are currently using this patch:</div><div><br></div><div><a href="http://src.seereason.com/haskell-text-debian/debian/patches/add_Data_instance.patch" target="_blank">http://src.seereason.com/haskell-text-debian/debian/patches/add_Data_instance.patch</a></div>

<div><br></div><div>which basically adds:</div><div><br></div><div><div>+textType = mkStringType &quot;Data.Text&quot;</div><div>+</div><div>+instance Data Text where</div><div>+   toConstr x = mkStringConstr textType (unpack x)</div>

<div>+   gunfold _k z c = case constrRep c of</div><div>+                     (CharConstr x) -&gt; z (pack [x])</div><div>+                     _ -&gt; error &quot;gunfold for Data.Text&quot;</div><div>+   dataTypeOf _ = textType</div>

<div>+</div><div><br></div><div>This particular implementation avoids exposing the internals of the Data.Text type by casting it to a String in toConstr and gunfold. That is similar to how Data is implemented for some numeric types. However, the space usage of casting in Float to a Double is far less than casting a Text to a String, so maybe that is not a good idea?</div>

<div><br></div><div>Alternatively, Data.ByteString just does &#39;deriving Data&#39;. However, bytestring also exports Data.ByteString.Internal, wheres Data.Text.Internal is not exported.</div><div><br></div><div>Any thoughts? I would like to get this handled upstream so that all happstack users can benefit from it.</div>

<div><br></div><font color="#888888"><div>- jeremy</div></font></div>
</blockquote></div><br></div>