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 ==================================================================
next 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: linkBe 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).