Modify

Opened 5 years ago

Closed 5 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@…> 5 years ago.
Add 3 overloads of add_edge and missing includes.

Download all attachments as: .zip

Change History (4)

Changed 5 years ago by Louis Dionne <louis.dionne92@…>

Attachment: named_graph.patch added

Add 3 overloads of add_edge and missing includes.

comment:1 Changed 5 years ago by Jeremiah Willcock

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

comment:2 Changed 5 years ago by Louis Dionne <louis.dionne92@…>

Yes, of course.

comment:3 Changed 5 years ago by Jeremiah Willcock

Resolution: fixed
Status: newclosed

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

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jeremiah Willcock.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.