CStorageSpec: Show instance to Pretty instance

Aaron Tomb aarontomb at gmail.com
Sun Aug 16 19:11:18 EDT 2009


Hi all,

I just noticed that L.C.Syntax.AST defines a Show instance for  
CStorageSpec, but not for any of its other types. That Show instance  
is only used in two places, one of which is to define a Pretty instance.

Can anyone think of any reason why this should remain? I have a patch  
to remove the Show instance from L.C.Syntax.AST and create an  
equivalent Pretty instance in L.C.Pretty. Any objections to pushing  
this patch?

It could presumably break client code, but it seems unlikely that  
there's code out there that needs to pretty-print storage  
specifications but nothing else.

Thanks,
Aaro



More information about the Language-c mailing list