Modify

Opened 4 years ago

Closed 4 years ago

#8690 closed Patches (fixed)

duration_units_default - lost return types constness in overridden methods

Reported by: tmmikolajczyk@… Owned by: viboes
Milestone: Boost 1.54.0 Component: chrono
Version: Boost 1.53.0 Severity: Problem
Keywords: Cc:

Description

When implementing custom chrono duration units by deriving from duration_units_default<> I noticed the "get_valid_units_start" and "get_valid_units_end" is declared to return mutable string_type pointer however the base class (duration_units<>) declares them as immutable. The attached patch has been generated with the "git diff --no-prefix" command (I have boost svn repo cloned with the git-svn tool).

Attachments (1)

duration_unit_return_type.patch (825 bytes) - added by tmmikolajczyk@… 4 years ago.

Download all attachments as: .zip

Change History (6)

Changed 4 years ago by tmmikolajczyk@…

comment:1 Changed 4 years ago by tmmikolajczyk@…

It concerns the 1.53 version and the latest trunk as well.

comment:2 Changed 4 years ago by viboes

Milestone: To Be DeterminedBoost 1.54.0
Status: newassigned

comment:3 Changed 4 years ago by viboes

Thanks for catching this issue. I will try to fix it for 1.54.

Last edited 4 years ago by viboes (previous) (diff)

comment:4 Changed 4 years ago by viboes

Committed revision [84751].

comment:5 Changed 4 years ago by viboes

Resolution: fixed
Status: assignedclosed

(In [84767]) Chrono: fix #8690.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain viboes.
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.