Opened 5 years ago

Closed 5 years ago

#7338 closed Bugs (fixed)

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:


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 (1)

indirect_cmp.patch (823 bytes) - added by Alex Hagen-Zanker <ahh34@…> 5 years ago.

Download all attachments as: .zip

Change History (2)

Changed 5 years ago by Alex Hagen-Zanker <ahh34@…>

comment:1 Changed 5 years ago by jewillco

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

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

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain jewillco.
The resolution will be deleted. Next status will be 'reopened'.

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

Note: See TracTickets for help on using tickets.