Opened 12 years ago

Closed 12 years ago

#467 closed request (fixed)

disable implicit conversion between Vector and its View siblings

Reported by: Peter Owned by: Peter
Priority: critical Milestone: yat 0.5
Component: utility Version: trunk
Keywords: Cc:

Description

implicit conversion is dangerous and should be avoided. This was unfortunately allowed in yat 0.4 but should be removed. As the change will break with 0.4, it must be documented in NEWS.

Change History (3)

comment:1 Changed 12 years ago by Peter

Resolution: fixed
Status: newclosed

(In [1651]) fixes #467 - implicit conversion between Vector and its Views are no longer allowed

comment:2 Changed 12 years ago by Peter

Resolution: fixed
Status: closedreopened

I think we should allow implicit conversion from VectorView? to VectorConstView?. It corresponds to normal const conversion. With Vector it is a different story because ownership is different in Vector and its views.

Conversion from a temporary VectorView? will probably be a bit tricky. See how the proxy class is used in VectorView?.

comment:3 Changed 12 years ago by Peter

Resolution: fixed
Status: reopenedclosed

Conversion from a temporary VectorView seems very tricky (see ticket #474). Without that working I don't think we should allow implicit conversion.

The initial issue has been resolved, so I close this ticket (and make a note about implicit conversion in #474).

Note: See TracTickets for help on using tickets.