Modify

Ticket #7338 (closed Bugs: fixed)

Opened 20 months ago

Last modified 20 months ago

pending/indirect_cmp has wrong return type

Reported by: Alex Hagen-Zanker <ahh34@…> Owned by: jewillco
Milestone: To Be Determined Component: graph
Version: Boost 1.52.0 Severity: Problem
Keywords: pending Cc:

Description

The return_type for indirect_cmp now is PropertyMap::value_type it should be Compare::result_type (or simply bool).

The return type for the operator(a,b) should then be return_type instead of bool.

Also, the PropertyMap? may be an LValuePropertyMap and then it might be avoided to copy the values before comparison.

Patch attached

Attachments

indirect_cmp.patch Download (823 bytes) - added by Alex Hagen-Zanker <ahh34@…> 20 months ago.

Change History

Changed 20 months ago by Alex Hagen-Zanker <ahh34@…>

comment:1 Changed 20 months ago by jewillco

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

(In [80420]) Fixed issues with indirect_cmp; fixes #7338

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.