Path: utzoo!utgpu!jarvis.csri.toronto.edu!rutgers!tut.cis.ohio-state.edu!G.OSWEGO.EDU!dl
From: dl@G.OSWEGO.EDU (Doug Lea)
Newsgroups: gnu.g++.lib.bug
Subject: Plex.hP / Plex.ccP inconsistency
Message-ID: <8911111238.AA01470@g.oswego.edu>
Date: 11 Nov 89 12:38:48 GMT
References: <469@ucl-cs.UUCP>
Sender: daemon@tut.cis.ohio-state.edu
Reply-To: dl@oswego.oswego.edu
Distribution: gnu
Organization: GNUs Not Usenet
Lines: 28


> void <T>Plex::fill(<T&> x, int lo, int hi)
> {...
> 
> should have ``const'' inserted before <T&> to match the
> description in the class header.
> 
> In fact this seems to be a problem in ALL the Plex prototype classes.

I removed `const' argument specifications from container classes
member functions where I thought these would give people problems -- I
didn't catch a few cases where I did this inconsistently between
headers and definitions. Sorry.

I removed const for two reasons:

In g++-1.36 prereleases, g++ was partially, but incompletely supporting
const member functions. (g++-1.36.1 now seems to do this correctly.)

Many people use containers for classes that have not been revised to
obey C++-2.0 const rules (This includes several libg++ classes!). I
decided to remove const so that everyone could have a chance to do this
without all of their code breaking.

const  member functions and arguments WILL be introduced/re-introduced
in these containers soon.

-Doug