Opened 15 years ago
Closed 15 years ago
#315 closed enhancement (done)
Hide the traits class parameters from the doc
Reported by: | Peter Kovacs | Owned by: | Peter Kovacs |
---|---|---|---|
Priority: | major | Milestone: | LEMON 1.2 release |
Component: | documentation | Version: | hg main |
Keywords: | Cc: | ||
Revision id: |
Description
The last template parameter of many algorithm classes (Bfs
, BfsVisit
, Dfs
, DfsVisit
, Dijkstra
, BellmanFord
, Preflow
, Circulation
, MinCostArborescence
) is a traits class. It is always shown in the doc, but it is typically not documented. This could be misleading, since the paramater seems to be mandatory.
A good solution would be to hide all these template parameters entirely.
This ticket is a follow-up of #179.
Attachments (2)
Change History (9)
comment:1 follow-up: 2 Changed 15 years ago by
comment:2 Changed 15 years ago by
Replying to alpar:
To be precise, this is only a satisfactory solution. The good one would be to fully document these traits class parameters.
Recall that they were fully documented in the 0.x releases, and we decided to hide these parameters, see the changeset in #185. Just like the basic iterator interfaces of the graph structures: they are needed to implement e.g. adaptors, but we do not document them.
I don't think we should change our mind and revert these parts of the doc.
comment:3 Changed 15 years ago by
Status: | new → assigned |
---|
Another solution is to list these template parameters in the doc for all classes, saying something like that: "The traits class that defines various types for the algorithm. This parameter is typically not set directly, consider to use the named template parameters instead."
comment:4 Changed 15 years ago by
comment:5 Changed 15 years ago by
The attached patch [70e6378230c3] adds a proper documentation to all these traits class parameters.
Changed 15 years ago by
Attachment: | 315-doc-traits-class-70e6378230c3.patch added |
---|
Changed 15 years ago by
Attachment: | 315-doc-traits-class-75e6020b19b1.patch added |
---|
comment:6 follow-up: 7 Changed 15 years ago by
[75e6020b19b1] is a slightly better version, consider to use this one.
comment:7 Changed 15 years ago by
Resolution: | → done |
---|---|
Status: | assigned → closed |
Replying to kpeter:
[75e6020b19b1] is a slightly better version, consider to use this one.
It is in the main branch.
Replying to kpeter:
To be precise, this is only a satisfactory solution. The good one would be to fully document these traits class parameters.