Opened 10 years ago

Closed 10 years ago

#7766 closed Patches (fixed)

missing add_edge overloads in named_graph

Reported by: Louis Dionne <louis.dionne92@…> Owned by: Jeremiah Willcock
Milestone: Boost 1.53.0 Component: graph
Version: Boost Development Trunk Severity: Problem
Keywords: named_graph graph add_edge Cc:

Description

Hello,

The named_graph is missing 3 overloads for the add_edge function. The overloads are required if the adapted graph models the EdgeMutablePropertyGraph concept.

An EdgeMutablePropertyGraph allows:

add_edge(vertex_descriptor1, vertex_descriptor2, property, graph)

A named_graph whose adapted graph models the concept should allow:

add_edge(vertex_name1, vertex_name2, property, graph)

add_edge(vertex_name, vertex_descriptor, property, graph)

add_edge(vertex_descriptor, vertex_name, property, graph)

A patch containing the 3 missing overloads is attached. The patch also adds some includes that I found to be missing to the named_graph.hpp header.

Regards,

Louis Dionne

Attachments (1)

named_graph.patch (2.7 KB ) - added by Louis Dionne <louis.dionne92@…> 10 years ago.
Add 3 overloads of add_edge and missing includes.

Download all attachments as: .zip

Change History (4)

by Louis Dionne <louis.dionne92@…>, 10 years ago

Attachment: named_graph.patch added

Add 3 overloads of add_edge and missing includes.

comment:1 by Jeremiah Willcock, 10 years ago

Since this is a decent amount of code, are you willing to license your additions under the Boost license?

comment:2 by Louis Dionne <louis.dionne92@…>, 10 years ago

Yes, of course.

comment:3 by Jeremiah Willcock, 10 years ago

Resolution: fixed
Status: newclosed

(In [81772]) Applied patch from #7766; fixes #7766

Note: See TracTickets for help on using tickets.