Plex.hP / Plex.ccP inconsistency

Thread View: gnu.g++.lib.bug

2 messages

[T] Back to Threads [V] Tree View [G] Group Articles

2 total messages Started by T.Day Wed, 08 Nov 1989 19:37

#177

Author: T.Day
Date: Wed, 08 Nov 1989 19:37

22 lines
1037 bytes

From: Tim Day <T.Day@uk.ac.ucl.cs>

In libg++-1.36.0 in Plex.hP / Plex.ccP

void <T>Plex::fill(<T&> x)
{...

and

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.

+-----------------------------------------------------------------------------+
Tim Day                           | Meet every second in life as challenge;
Department of Photogrammetry      | Respond fully to whatever happens
UCL, Gower St., London  WC1E 6BT  |  without anxiety, or complaint, or clinging
+-----------------------------------------------------------------------------+

Plex.hP / Plex.ccP inconsistency

#178

Author: Doug Lea
Date: Sat, 11 Nov 1989 12:38

28 lines
1451 bytes

> 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

Thread Navigation

This is a paginated view of messages in the thread with full content displayed inline.

Messages are displayed in chronological order, with the original post highlighted in green.

Use pagination controls to navigate through all messages in large threads.

Back to All Threads