621. non-const copy assignment operators of helper arrays

Section: 29.7 [numarray] Status: CD1 Submitter: Martin Sebor Opened: 2007-01-20 Last modified: 2016-02-10

Priority: Not Prioritized

View all other issues in [numarray].

View all issues with CD1 status.

Discussion:

The computed and "fill" assignment operators of valarray helper array class templates (slice_array, gslice_array, mask_array, and indirect_array) are const member functions of each class template (the latter by the resolution of 123 since they have reference semantics and thus do not affect the state of the object on which they are called. However, the copy assignment operators of these class templates, which also have reference semantics, are non-const. The absence of constness opens the door to speculation about whether they really are intended to have reference semantics (existing implementations vary widely).

Pre-Kona, Martin adds:

I realized that adding the const qualifier to the functions as I suggested would break the const correctness of the classes. A few possible solutions come to mind:

  1. Add the const qualifier to the return types of these functions.
  2. Change the return type of all the functions to void to match the signatures of all the other assignment operators these classes define.
  3. Prohibit the copy assignment of these classes by declaring the copy assignment operators private (as is done and documented by some implementations).

Proposed resolution:

Declare the copy assignment operators of all four helper array class templates const.

Specifically, make the following edits:

Change the signature in 29.7.5 [template.slice.array] and 29.7.5.2 [slice.arr.assign] as follows:


const slice_array& operator= (const slice_array&) const;

        

Change the signature in 29.7.7 [template.gslice.array] and 29.7.7.2 [gslice.array.assign] as follows:


const gslice_array& operator= (const gslice_array&) const;

        

Change the signature in 29.7.8 [template.mask.array] and 29.7.8.2 [mask.array.assign] as follows:


const mask_array& operator= (const mask_array&) const;

        

Change the signature in 29.7.9 [template.indirect.array] and 29.7.9.2 [indirect.array.assign] as follows:


const indirect_array& operator= (const indirect_array&) const;

        

[ Kona (2007) Added const qualification to the return types and set to Ready. ]