February 2003 - template-haskell

RE: [Template-haskell] Instance declarations
by Derek Elkins 19 Feb '03

19 Feb '03

Sorry about that. It looks like it was my fault. I made a list of functions rather than a list of clauses. >From: "Simon Peyton-Jones" <simonpj(a)microsoft.com> >To: "Derek Elkins" <ddarius86(a)hotmail.com>,<template-haskell(a)haskell.org> >Subject: RE: [Template-haskell] Instance declarations >Date: Wed, 19 Feb 2003 15:21:54 -0000 > >Can you give the actual source code for the offending program, please? > >Simon > >| -----Original Message----- >| From: Derek Elkins [mailto:ddarius86@hotmail.com] >| Sent: 13 February 2003 20:29 >| To: template-haskell(a)haskell.org >| Subject: [Template-haskell] Instance declarations >| >| My first email seems to have gotten lost, if it meanders it's way >here, then >| sorry for the repeat. >| >| Anyways, generating a set of pattern matching functions in an instance >| declaration (maybe in general, though I haven't tested it) causes a >| "conflicting definitions error". Rewriting as a case expression is >one >| work-around. >| >| E.g. >| >| instance (NFData a) => NFData (Tree a) where >| rnf (Leaf a) = rnf a >| rnf (Branch a b) = rnf a `seq` rnf b >| >| doesn't work, but rewriting as >| >| instance (NFData a) => NFData (Tree a) where >| rnf x = case x of >| (Leaf a) -> rnf a >| (Branch a b) -> rnf a `seq` rnf b >| >| does. >| >| I suspect that it may have to do with the generated code being >considered at >| a lower level (i.e. the transformation of the above into the below is >| already considered done). It may just be I'm creating the functions >| incorrectly, so if you can't get the error to happen I can provide the >code >| I'm using. >| >| Separately: I've written a variety of fold algebras over data >declarations. >| I'm not sure how useful they'll be, but if you're interested email me, >and >| I'll email the source. I haven't tested them, but it's pretty >mechanical >| code so I don't suspect any major problems and any should be >relatively >| straightforward to fix. See `Dealing with Large Bananas` for the >concept >| (and for documentation, as it has no documentation yet). >| >| _________________________________________________________________ >| STOP MORE SPAM with the new MSN 8 and get 2 months FREE* >| http://join.msn.com/?page=features/junkmail >| >| _______________________________________________ >| template-haskell mailing list >| template-haskell(a)haskell.org >| http://www.haskell.org/mailman/listinfo/template-haskell >_______________________________________________ >template-haskell mailing list >template-haskell(a)haskell.org >http://www.haskell.org/mailman/listinfo/template-haskell _________________________________________________________________ Help STOP SPAM with the new MSN 8 and get 2 months FREE* http://join.msn.com/?page=features/junkmail

1 0

RE: [Template-haskell] RE: Declaration reordering
by Simon Peyton-Jones 19 Feb '03

19 Feb '03

| I've had a quick look and I think something called by one or both of | "tcMonoExpr (HsBracket brack loc) aes_ty" and "tcBracket"? Oh, but the | former calls the latter, so probably somewhere in tcTopSrcDecls? I'll | try following this through if so. deSugar/DsMeta.lhs, line 156 or thereabouts, is where a declaration bracket gets turned into data structure. Instead of return (val_ds ++ catMaybes tycl_ds ++ inst_ds) you want return (sortByLoc (...the same thing...)) where sortByLoc does the sorting by source location. Have fun! Actually it should be really easy. Simon

1 0

RE: [Template-haskell] Instance declarations
by Simon Peyton-Jones 19 Feb '03

19 Feb '03

Can you give the actual source code for the offending program, please? Simon | -----Original Message----- | From: Derek Elkins [mailto:ddarius86@hotmail.com] | Sent: 13 February 2003 20:29 | To: template-haskell(a)haskell.org | Subject: [Template-haskell] Instance declarations | | My first email seems to have gotten lost, if it meanders it's way here, then | sorry for the repeat. | | Anyways, generating a set of pattern matching functions in an instance | declaration (maybe in general, though I haven't tested it) causes a | "conflicting definitions error". Rewriting as a case expression is one | work-around. | | E.g. | | instance (NFData a) => NFData (Tree a) where | rnf (Leaf a) = rnf a | rnf (Branch a b) = rnf a `seq` rnf b | | doesn't work, but rewriting as | | instance (NFData a) => NFData (Tree a) where | rnf x = case x of | (Leaf a) -> rnf a | (Branch a b) -> rnf a `seq` rnf b | | does. | | I suspect that it may have to do with the generated code being considered at | a lower level (i.e. the transformation of the above into the below is | already considered done). It may just be I'm creating the functions | incorrectly, so if you can't get the error to happen I can provide the code | I'm using. | | Separately: I've written a variety of fold algebras over data declarations. | I'm not sure how useful they'll be, but if you're interested email me, and | I'll email the source. I haven't tested them, but it's pretty mechanical | code so I don't suspect any major problems and any should be relatively | straightforward to fix. See `Dealing with Large Bananas` for the concept | (and for documentation, as it has no documentation yet). | | _________________________________________________________________ | STOP MORE SPAM with the new MSN 8 and get 2 months FREE* | http://join.msn.com/?page=features/junkmail | | _______________________________________________ | template-haskell mailing list | template-haskell(a)haskell.org | http://www.haskell.org/mailman/listinfo/template-haskell

1 0

Re: [Template-haskell] Record constructors
by Derek Elkins 18 Feb '03

18 Feb '03

>From: Ian Lynagh <igloo(a)earth.li> >To: Simon Peyton-Jones <simonpj(a)microsoft.com> >CC: template-haskell(a)haskell.org >Subject: Re: [Template-haskell] Record constructors >Date: Mon, 17 Feb 2003 15:32:46 +0000 > >On Mon, Feb 17, 2003 at 03:15:27PM -0000, Simon Peyton-Jones wrote: > > > > Now I'm confused. > > > > Ian: you are the one who's going to have a go at this, correct? > >Sorry, misread Derek's reply. Will do. > > >Thanks >Ian I've made the changes for the contexts on datatypes and records constructors. They seem to work all right. I just need to make a patch and send it somewhere. I could add the strictness stuff if you want. I know pretty much what would need to change (or at least part of it). Another bug I might go after is deriving seems to generate multiple instances. E.g. $([d| data X = X String deriving Show |]) causes a duplicate instance declaration error and panics when compiling. _________________________________________________________________ The new MSN 8: smart spam protection and 2 months FREE* http://join.msn.com/?page=features/junkmail

1 0

RE: [Template-haskell] Record constructors
by Simon Peyton-Jones 17 Feb '03

17 Feb '03

Now I'm confused. Ian: you are the one who's going to have a go at this, correct? Re you qn about strictness annotations, what you suggest looks fine. Simon | -----Original Message----- | From: Ian Lynagh [mailto:igloo@earth.li] | Sent: 17 February 2003 14:38 | To: template-haskell(a)haskell.org | Subject: Re: [Template-haskell] Record constructors | | On Fri, Feb 14, 2003 at 02:35:28PM -0500, Derek Elkins wrote: | > >From: "Simon Peyton-Jones" <simonpj(a)microsoft.com> | > >To: "Derek Elkins" <ddarius86(a)hotmail.com>,<template-haskell(a)haskell.org> | > >Subject: RE: [Template-haskell] Record constructors | > >Date: Fri, 14 Feb 2003 10:18:31 -0000 | > > | > >Good point. Indeed, THSyntax has no provision for record types and | > >record expressions. | > >Haskell is a pretty large language and I didn't cover the whole of it in | > >the first pass, I'm afraid. | > > | > > | > > IAN L, would you care to put in the THSyntax for | > > records | > > contexts in data type declarations | > > and add suitable stuff to DsMeta and Convert to get them to and | > >fro? | > > | > >(Or someone else!) I'm too much of a bottleneck for stuff like this. | > >I'm inclined to focus my time on things where it really has to be me. | > > | > >Simon | > | > Okay. I'll take a swing at it. | | OK - if you have problems I will try to help. | | Is there a reason why strictness annotations are ignored, or just the | first pass issue again? | Presumably any StrictnessMark except NotMarkedStrict should be reported | as strict? I think | data Con = Constr String [(Strictness, Typ)] | data Strictness = Strict | NonStrict | is probably as good a representation as any other. | | Any objections? | | | Thanks | Ian | | _______________________________________________ | template-haskell mailing list | template-haskell(a)haskell.org | http://www.haskell.org/mailman/listinfo/template-haskell

2 1

RE: [Template-haskell] Record constructors
by Derek Elkins 17 Feb '03

17 Feb '03

>From: "Simon Peyton-Jones" <simonpj(a)microsoft.com> >To: "Derek Elkins" <ddarius86(a)hotmail.com>,<template-haskell(a)haskell.org> >Subject: RE: [Template-haskell] Record constructors >Date: Fri, 14 Feb 2003 10:18:31 -0000 > >Good point. Indeed, THSyntax has no provision for record types and >record expressions. >Haskell is a pretty large language and I didn't cover the whole of it in >the first pass, I'm afraid. > > > IAN L, would you care to put in the THSyntax for > records > contexts in data type declarations > and add suitable stuff to DsMeta and Convert to get them to and >fro? > >(Or someone else!) I'm too much of a bottleneck for stuff like this. >I'm inclined to focus my time on things where it really has to be me. > >Simon Okay. I'll take a swing at it. I've made all the changes I think are necessary for contexts on datatypes, but I won't know until I recompile. For record constructors, I'm thinking of adding `RecConstr String [(String, Typ)]` to Con in THSyntax. This is slightly modified from the HsSyn version `HsRecDecl SrcLoc HsName [([HsName],HsBangType)]` which I suppose is setup to handle `foo,bar :: a -> b` I don't see this as very convenient for generation or reification. So unless I'm missing something or another layout is preferable, that's the setup I'm going with. _________________________________________________________________ Tired of spam? Get advanced junk mail protection with MSN 8. http://join.msn.com/?page=features/junkmail

2 1

RE: Declaration reordering
by Simon Peyton-Jones 17 Feb '03

17 Feb '03

I'm not sure I do agree that it's worthwhile. At least, of course it would be nice but the question is whether it's worth the bother. Why are you keen on it? You sent your original message to ghc-users; perhaps you'd like to summarise for the TH list. Your first option looks best. Indeed, each decl has its SrcLoc which includes a line number, so perhaps you could unscramble them that way. You are welcome to have a go; if you need help figuring out where, I'll help. Simon | -----Original Message----- | From: Ian Lynagh [mailto:igloo@earth.li] | Sent: 14 February 2003 03:11 | To: Simon Peyton-Jones | Subject: Re: Declaration reordering | | On Thu, Feb 13, 2003 at 09:16:46AM -0000, Simon Peyton-Jones wrote: | > | > The renamer performs a strongly-connected component analysis of | > declarations. Reason: it's the standard way to get maximal | > polymorphism. It's even part of the language specification. | > | > It'd be quite tricky to un-do that stuff before doing the quotation | > part. So I'm really not sure how to tackle your problem. | | Do you agree that it is worthwhile? | | The possibilities that spring to mind are: | | * tagging each declaration with its position in the list and sorting | them before returning them. The only arguments against this are it | adds a bit of code and will have a time/space impact (but I'd have | thought negligible, especially if the tagging was done strictly). | | * Keeping a copy of the original input around to pass back. Doubles | space used by declaration lists and I suspect will have problems with | right infix operators. | | * Doing the SCC analysis and type checking on a copy and throwing it | away afterwards. Would also have the problems of the above and is | probably more complicated to implement. | | | Thanks | Ian

2 1

RE: [Template-haskell] Record constructors
by Simon Peyton-Jones 14 Feb '03

14 Feb '03

Good point. Indeed, THSyntax has no provision for record types and record expressions. Haskell is a pretty large language and I didn't cover the whole of it in the first pass, I'm afraid. IAN L, would you care to put in the THSyntax for records contexts in data type declarations and add suitable stuff to DsMeta and Convert to get them to and fro? (Or someone else!) I'm too much of a bottleneck for stuff like this. I'm inclined to focus my time on things where it really has to be me. Simon | -----Original Message----- | From: Derek Elkins [mailto:ddarius86@hotmail.com] | Sent: 14 February 2003 03:09 | To: template-haskell(a)haskell.org | Subject: [Template-haskell] Record constructors | | Quasiquoted expressions involving record construction syntax dies. | Furthermore, how do you even generate a record constructor declaration? | There's no place to put the record name. | | module Main where | import Language.Haskell.THSyntax | import Text.PrettyPrint.HughesPJ (render) | | data Lang a b = Lang { | tvarAlg :: String -> a, | arrowAlg :: a -> a -> a, | varAlg :: String -> b, | applyAlg :: b -> b -> b, | lambdaAlg :: (String, a) -> b -> b | } | | e = [| let t = Lang { tvarAlg = \s -> (), | arrowAlg = \x y -> (), | varAlg = \s -> (), | applyAlg = \x y -> (), | lambdaAlg = \x y -> () | } | in tvarAlg t |] | | main = do output <- runQ e | putStrLn $ render $ pprExp output | putStrLn "\n" | putStrLn $ show output | | _________________________________________________________________ | Add photos to your messages with MSN 8. Get 2 months FREE*. | http://join.msn.com/?page=features/featuredemail | | _______________________________________________ | template-haskell mailing list | template-haskell(a)haskell.org | http://www.haskell.org/mailman/listinfo/template-haskell

1 0

Record constructors
by Derek Elkins 14 Feb '03

14 Feb '03

Quasiquoted expressions involving record construction syntax dies. Furthermore, how do you even generate a record constructor declaration? There's no place to put the record name. module Main where import Language.Haskell.THSyntax import Text.PrettyPrint.HughesPJ (render) data Lang a b = Lang { tvarAlg :: String -> a, arrowAlg :: a -> a -> a, varAlg :: String -> b, applyAlg :: b -> b -> b, lambdaAlg :: (String, a) -> b -> b } e = [| let t = Lang { tvarAlg = \s -> (), arrowAlg = \x y -> (), varAlg = \s -> (), applyAlg = \x y -> (), lambdaAlg = \x y -> () } in tvarAlg t |] main = do output <- runQ e putStrLn $ render $ pprExp output putStrLn "\n" putStrLn $ show output _________________________________________________________________ Add photos to your messages with MSN 8. Get 2 months FREE*. http://join.msn.com/?page=features/featuredemail

1 0

Instance declarations
by Derek Elkins 13 Feb '03

13 Feb '03

My first email seems to have gotten lost, if it meanders it's way here, then sorry for the repeat. Anyways, generating a set of pattern matching functions in an instance declaration (maybe in general, though I haven't tested it) causes a "conflicting definitions error". Rewriting as a case expression is one work-around. E.g. instance (NFData a) => NFData (Tree a) where rnf (Leaf a) = rnf a rnf (Branch a b) = rnf a `seq` rnf b doesn't work, but rewriting as instance (NFData a) => NFData (Tree a) where rnf x = case x of (Leaf a) -> rnf a (Branch a b) -> rnf a `seq` rnf b does. I suspect that it may have to do with the generated code being considered at a lower level (i.e. the transformation of the above into the below is already considered done). It may just be I'm creating the functions incorrectly, so if you can't get the error to happen I can provide the code I'm using. Separately: I've written a variety of fold algebras over data declarations. I'm not sure how useful they'll be, but if you're interested email me, and I'll email the source. I haven't tested them, but it's pretty mechanical code so I don't suspect any major problems and any should be relatively straightforward to fix. See `Dealing with Large Bananas` for the concept (and for documentation, as it has no documentation yet). _________________________________________________________________ STOP MORE SPAM with the new MSN 8 and get 2 months FREE* http://join.msn.com/?page=features/junkmail

1 0