Opened 11 years ago
Last modified 11 years ago
#6391 new Bugs
New reverse_graph edge descriptor type breaks shared property maps
| Reported by: | Owned by: | Jeremiah Willcock | |
|---|---|---|---|
| Milestone: | To Be Determined | Component: | graph |
| Version: | Boost 1.48.0 | Severity: | Regression |
| Keywords: | reverse_graph | Cc: |
Description
The newly implemented data type for the edge descriptor of reversed_graph does not allow for the transparent sharing of edge property maps between the base and reversed graphs, which was possible with the old code (< 1.48), since its type is not related to the base graph. This is a very useful feature, and I think it should be preserved.
I'm sending attached a patch which makes the reversed edge descriptior a derived class from the base edge descriptor. This makes the type itself distinct from the base graph, avoiding any ambiguities, while at the same time preserving interchangeability with the base descriptor when required, e.g. when accessing property maps. This also makes the code a bit more elegant.
Attachments (1)
Change History (2)
by , 11 years ago
| Attachment: | reverse_graph_desc.patch added |
|---|
comment:1 by , 11 years ago
I also just hit this issue when trying to upgrade to boost 1.48.0 and tiago's patch made my code compile and work. I'm wondering if this patch is going to be accepted by boost or if I have to rewrite my code to work around this issue.
Thanks in advance
Proposed patch