Ticket #145 (new enhancement)

Opened 7 years ago

Last modified 5 years ago

Type class instances: link to source

Reported by: MartijnVanSteenbergen Owned by:
Priority: major Milestone:
Version: 2.7.2 Keywords:
Cc: mmitar@…, pho@…, byorgey@…


Currently the detailed documentation of a type class lists known instances of the type class. It'd be really helpful if the instances linked to their source wherever possible.

Change History

Changed 7 years ago by mitar

  • cc mmitar@… added

Changed 6 years ago by PHO

  • cc pho@… added

I really want this feature too, as instance declarations are rarely documented while their behaviours aren't always obvious.

Changed 5 years ago by anonymous

  • milestone 2.7.0 deleted

Milestone 2.7.0 deleted

Changed 5 years ago by byorgey

  • cc byorgey@… added

This came up again today on the libraries@ mailing list (and I would also love to see this feature), so I spent some time trying to figure out what would be necessary to implement this. Here are my (quite sketchy) notes:


       + ppDecl calls
         + ppDataDecl which calls
           + ppInstances which calls
             + Xhtml/Layout.hs:subInstances -- does actual generation
               of html for instances list.

       ppDataDecl gets a [DocInstance DocName];

       type DocInstance name = (InstHead name, Maybe (Doc name)) --
       from src/Haddock/Types.hs

       type InstHead name = ([HsType name], name, [HsType name])

       So already at this point we've forgotten source location
       information.  But if we had the information in the DocInstance
       type we could output it here.

       Tracing instances info backwards from ppDecl:

       + ppDecl is called from...
       + Haddock.Backends.Xhtml.processExport -- gets instance list from
	 pattern-matching on (ExportDecl ...) :: ExportItem DocName.
       + called from ifaceToHtml, mapped over "exports", defined as 
	 exports = numberSectionHeadings (ifaceRnExportItems iface)

	 ifaceRnExportItems is a record selector.

       + called from ppHtmlModule (iface passed along)
       + called from ppHtml, mapped over visible_ifaces, which is
         filtered from argument ifaces.

       + ppHtml called from Main.hs:render.  render gets ifaces as

       + render called from renderStep, which passes on argument
       + render called from main.  interfaces argument comes from call
         to readPackagesAndProcessModules.
       + readPackagesAndProcessModules gets ifaces from call to
       + processModules is in Haddock.Interface.

	 This is the point at which I ran out of steam. But the hope
         is that somewhere in processModules or functions called by
         it, the actual source location information of instances is

	 The DocInstance type would have to be modified to store
	 source location info.
Note: See TracTickets for help on using tickets.