COIN-OR::LEMON - Graph Library

Opened 10 years ago

Last modified 8 years ago

#37 assigned enhancement

operator= for RangeMap and SparseMap

Reported by: kpeter Owned by: kpeter
Priority: major Milestone:
Component: core Version: hg main
Keywords: Cc: deba
Revision id:

Description

Now operator= function is private (without implementation) in RangeMap and SparseMap classes. Do we need public operator=?

Change History (10)

comment:1 in reply to: ↑ description ; follow-up: Changed 10 years ago by alpar

  • Cc deba added

Replying to kpeter:

Now operator= function is private (without implementation) in RangeMap and SparseMap classes. Do we need public operator=?

Generally, maps are not assignable. Should RangeMap and SparseMap be exceptions?

comment:2 in reply to: ↑ 1 Changed 10 years ago by kpeter

Replying to alpar:

Generally, maps are not assignable. Should RangeMap and SparseMap be exceptions?

According to the map concepts a map class needn't be assignable (e.g. maps created with map adaptors are usually not assignable).
However the basic maps except for RangeMap and SparseMap (NullMap, ConstMap, IdentityMap) are assignable, see test/maps_test.cc. So RangeMap and SparseMap would not be exceptions and operator= would be a useful feature for them.

comment:3 Changed 10 years ago by alpar

  • Owner changed from alpar to kpeter

comment:4 Changed 10 years ago by kpeter

  • Status changed from new to assigned

comment:5 Changed 10 years ago by kpeter

What is your opinion? Should the basic map classes (not Adaptors!) be assignable?

comment:6 Changed 10 years ago by alpar

  • Milestone changed from LEMON 1.0 release to Post 1.0

comment:7 Changed 10 years ago by alpar

  • Version set to hg main

comment:8 Changed 9 years ago by kpeter

This question is related to #137 and #146.

comment:9 Changed 9 years ago by alpar

  • Milestone changed from LEMON 1.1 release to LEMON 1.2 release

The deadline for release 1.1 is rapidly approaching and we still have a couple of more important things to do till then. Thus I postpone this ticket.

comment:10 Changed 8 years ago by kpeter

  • Milestone LEMON 1.2 release deleted
Note: See TracTickets for help on using tickets.