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: Jeremiah Willcock
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@…>

Attachment: indirect_cmp.patch added

comment:1 Changed 5 years ago by Jeremiah Willcock

Resolution: fixed
Status: newclosed

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

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.