public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jonathan Wakely <jwakely@redhat.com>
To: Jason Merrill <jason@redhat.com>
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH] wwwdocs: Add ADL to C++ non-bugs
Date: Thu, 5 Oct 2023 10:11:58 +0100	[thread overview]
Message-ID: <CACb0b4kOffy=kksAL3BaBJ5PQ+Aokw1Rx0hrNC04eEOEV7nCcg@mail.gmail.com> (raw)
In-Reply-To: <b7daf709-3f72-472b-b3e3-d104d138f73e@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 606 bytes --]

On Wed, 4 Oct 2023 at 20:17, Jason Merrill <jason@redhat.com> wrote:
>
> On 10/3/23 10:45, Jonathan Wakely wrote:
> > We have a long history of INVALID bugs about std functions being
> > available in the global namespace (PRs 27846, 67566, 82619, 99865,
> > 110602, 111553, probably others). Let's document it.
> >
> > Also de-prioritize the C++98-only bugs, which are unlikely to affect
> > anybody nowadays.
> >
> > OK for wwwdocs?
>
> OK, thanks.
>
> Jason


After pushing it I realised the formatting looks bad compared to the
other items in the list, so I've pushed the attached follow-up as
obvious.

[-- Attachment #2: patch.txt --]
[-- Type: text/plain, Size: 1026 bytes --]

commit 1b1a0cf29826ce9287a203cde00fd1512918fc17
Author: Jonathan Wakely <jwakely@redhat.com>
Date:   Thu Oct 5 10:09:54 2023 +0100

    Add <p> to new item in C++ non-bugs list

diff --git a/htdocs/bugs/index.html b/htdocs/bugs/index.html
index 41edc561..da3d4c0d 100644
--- a/htdocs/bugs/index.html
+++ b/htdocs/bugs/index.html
@@ -541,12 +541,14 @@ for details.
 <dl>
 <dt>Functions can be called without qualifying them with their namespace.</dt>
 <dd>
+<p>
 Argument Dependent Lookup (ADL) means that functions can be found in namespaces
 associated with their arguments. This means that <code>move(arg)</code> can
 call <code>std::move</code> if <code>arg</code> is a type defined in namespace
 <code>std</code>, such as <code>std::string</code> or <code>std::vector</code>.
 If <code>std::move</code> is not the function you intended to call, use a
 qualified name such as <code>::move(arg)</code> or <code>foo::move(arg)</code>.
+</p>
 </dd>
 
 <dt>Nested classes can access private members and types of the containing

      reply	other threads:[~2023-10-05  9:12 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-03 14:45 Jonathan Wakely
2023-10-04 19:17 ` Jason Merrill
2023-10-05  9:11   ` Jonathan Wakely [this message]

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='CACb0b4kOffy=kksAL3BaBJ5PQ+Aokw1Rx0hrNC04eEOEV7nCcg@mail.gmail.com' \
    --to=jwakely@redhat.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=jason@redhat.com \
    /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).