public inbox for gcc-prs@sourceware.org
help / color / mirror / Atom feed
From: Eric Allamanche <alm@iis.fhg.de>
To: nobody@gcc.gnu.org
Cc: gcc-prs@gcc.gnu.org
Subject: Re: c++/3745: bug in basic_string.tcc
Date: Mon, 23 Jul 2001 00:56:00 -0000	[thread overview]
Message-ID: <20010723075602.20250.qmail@sourceware.cygnus.com> (raw)

The following reply was made to PR c++/3745; it has been noted by GNATS.

From: Eric Allamanche <alm@iis.fhg.de>
To: Phil Edwards <pedwards@disaster.jaj.com>
Cc: gcc-gnats@gcc.gnu.org, spz@iis.fhg.de
Subject: Re: c++/3745: bug in basic_string.tcc
Date: Mon, 23 Jul 2001 09:54:18 +0200

 Phil,
 
 thanks for your answer. The "serious" qualifier was due to the fact that
 I did not really paid attention to this field (sorry for that). Anyhow,
 we had some code which could not get compiled because of the mentionned
 issue. I will check if we can reproduce this error with a reduced code
 fragment.
 
 Eric
 
 Phil Edwards wrote:
 > 
 > On Fri, Jul 20, 2001 at 09:04:42AM -0000, alm@iis.fhg.de wrote:
 > > >Severity:       serious
 > > >Description:
 > > On line 630 of the basic_string.tcc file:
 > >   __pos = std::min(__size - __n ,__pos);
 > >
 > > should be:
 > >  __pos = min(__size - __n ,__pos);
 > >
 > > since the entire block is in the "namespace std {" scope.
 > > This can lead to a compilation error.
 > 
 > Perhaps I'm being dense, but I do not see how.  I have tried to make this
 > function call fail to compile, but cannot.  Please give an example of such
 > source code, showing why this is "serious".
 > 
 > We could probably remove "std::" just to make it match with the rest of the
 > file, but I would want to know why the qualifier is there in the first place.
 > 
 > Thanks,
 > Phil
 > 
 > --
 > Would I had phrases that are not known, utterances that are strange, in
 > new language that has not been used, free from repetition, not an utterance
 > which has grown stale, which men of old have spoken.
 >                                      - anonymous Egyptian scribe, c.1700 BC
 
 -- 
 ==================================================================
  Eric Allamanche                        mailto:alm@iis.fhg.de
  Fraunhofer Institut 
  Integrierte Schaltungen                Phone :+49 9131 776-322
  Dept. Audio-Multimedia                 Fax   :+49 9131 776-398
  Am Weichselgarten 3                    http://www.iis.fhg.de/amm  
  D-91058 Erlangen, Germany		
 ==================================================================


             reply	other threads:[~2001-07-23  0:56 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-07-23  0:56 Eric Allamanche [this message]
  -- strict thread matches above, loose matches on Subject: below --
2001-08-10  7:58 lerdsuwa
2001-07-21 16:36 Phil Edwards
2001-07-20  2:06 alm

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20010723075602.20250.qmail@sourceware.cygnus.com \
    --to=alm@iis.fhg.de \
    --cc=gcc-prs@gcc.gnu.org \
    --cc=nobody@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).