Modify

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:

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 (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
Action
as closed The owner will remain jewillco.
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.