Modify

Ticket #7766 (closed Patches: fixed)

Opened 17 months ago

Last modified 17 months ago

missing add_edge overloads in named_graph

Reported by: Louis Dionne <louis.dionne92@…> Owned by: jewillco
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

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

Change History

Changed 17 months ago by Louis Dionne <louis.dionne92@…>

Add 3 overloads of add_edge and missing includes.

comment:1 Changed 17 months ago by jewillco

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

comment:2 Changed 17 months ago by Louis Dionne <louis.dionne92@…>

Yes, of course.

comment:3 Changed 17 months ago by jewillco

  • Status changed from new to closed
  • Resolution set to fixed

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

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


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

 
Note: See TracTickets for help on using tickets.