Topic: Constructor can subvert constness (was ~const)


Author: ngo@tammy.harvard.edu (Tom Ngo)
Date: 22 Mar 91 03:22:44 GMT
Raw View
Twenty days ago, Reid Ellis <rae@utcs.toronto.edu> wrote:

rae> Tom Ngo <ngo@tammy.harvard.edu> writes:
rae> [Are there any ways to accomplish what casting away const does,
rae> without doing so explicitly?]
rae>
rae> How about this?
rae>
rae> struct foo {
rae>     foo();
rae>     int sometimes() const;
rae>     int nonconst();
rae>   private:
rae>     foo & fRef;
rae> };
rae>
rae> foo::foo() : fRef(*this) {}
rae>
rae> Then, if say "sometimes()" needs to do a non-const thing, it can
rae> simply say "fRef.nonconst()".
rae>
rae> I guess this is essentially casting the const away, but the compiler
rae> won't make a peep about it.

Interesting!  This must be a violation of ARM 8.4.3 [References],
which states:

    A reference to a plain T can be initialized only with a plain T.

Assuming this rule is intended to apply to the initialization of a
reference in a constructor, it seems you have uncovered an issue that
really ought to be addressed in the ARM:  what is that status of a
constructor with regard to constness?  Should it be made possible, as
suggested by Jim Adcock <jimad@microsoft.UUCP>, for constructors to be
specified as const?  If so, how would a const constructor be permitted
to change data members?

When a const object is constructed, I suppose *this is considered
non-const until the constructor is done.  This provides a window of
opportunity to initialize a non-const pointer or reference that is
durable, i.e. will still be alive after the constructor is done.

Perhaps the ARM needs to specify explicit rules regarding the
initialization of such durable entities.  Clearly one should not be
able to initialize a non-const pointer or reference from this.  But
what about components of this?  Can someone come up with a concise
rule that covers all of the ways that the constness of an object might
be subverted by assignments to non-const pointers and references
during construction?
--
  Tom Ngo
  ngo@harvard.harvard.edu
  617/495-1768 lab number, leave message




Author: vaughan@puma.cad.mcc.com (Paul Vaughan)
Date: 25 Mar 91 15:41:27 GMT
Raw View
   From: ngo@tammy.harvard.edu (Tom Ngo)

   Twenty days ago, Reid Ellis <rae@utcs.toronto.edu> wrote:

   rae> Tom Ngo <ngo@tammy.harvard.edu> writes:
   rae> [Are there any ways to accomplish what casting away const does,
   rae> without doing so explicitly?]
   rae>
   rae> How about this?
   rae>
   rae> struct foo {
   rae>     foo();
   rae>     int sometimes() const;
   rae>     int nonconst();
   rae>   private:
   rae>     foo & fRef;
   rae> };
   rae>
   rae> foo::foo() : fRef(*this) {}
   rae>
   rae> Then, if say "sometimes()" needs to do a non-const thing, it can
   rae> simply say "fRef.nonconst()".
   rae>
   rae> I guess this is essentially casting the const away, but the compiler
   rae> won't make a peep about it.

   Interesting!  This must be a violation of ARM 8.4.3 [References],
   which states:

       A reference to a plain T can be initialized only with a plain T.

   Assuming this rule is intended to apply to the initialization of a
   reference in a constructor, it seems you have uncovered an issue that
   really ought to be addressed in the ARM:  what is that status of a
   constructor with regard to constness?  Should it be made possible, as
   suggested by Jim Adcock <jimad@microsoft.UUCP>, for constructors to be
   specified as const?  If so, how would a const constructor be permitted
   to change data members?

Note that "const" in this context is a property of a pointer, not of
an object.  For instance

class Foo {
public:
  void bar() const;
  void nonconst();
};

Foo* global;

void Foo::bar() const {
 global->nonconst();
}

main() {
  global = new Foo();
  const Foo* f = global;
  f->bar();
}

It doesn't really make any difference that one example uses a global
and the other one uses a member. The point is, when the pointer (or
reference) was initialized, it was initialized with a non-const.  When
you realize that the Foo object isn't intrinsically const, you see
that nothing is really subverted, and no rules are violated. Also note
that the only way you can have intrinsically const objects (that is,
ones that might be stored in read only memory) is if they have no
constructor.


--

 Paul Vaughan, MCC CAD Program | ARPA: vaughan@mcc.com | Phone: [512] 338-3639
 Box 200195, Austin, TX 78720  | UUCP: ...!cs.utexas.edu!milano!cadillac!vaughan
----------------------------------------------------------------------------------
I spent from $3 to $10 today to pay interest on the national debt.  How about you?
----------------------------------------------------------------------------------