* [PATCH] libiberty: fix URL for demangler ABI
@ 2018-03-02 16:38 David Malcolm
2018-03-02 16:41 ` Jeff Law
0 siblings, 1 reply; 2+ messages in thread
From: David Malcolm @ 2018-03-02 16:38 UTC (permalink / raw)
To: gcc-patches; +Cc: David Malcolm
The comment at the top of cp-demangle.c lists:
http://www.codesourcery.com/cxx-abi/abi.html#mangling
which redirects to:
https://mentorembedded.github.io/cxx-abi/abi.html#mangling
but this is a 404: the latter site is empty.
I looked on archive.org; the final version before the redirect
seems to be:
https://web.archive.org/web/20110903075217/http://www.codesourcery.com:80/public/cxx-abi/abi.html
Some searching showed up:
https://itanium-cxx-abi.github.io/cxx-abi/abi.html#mangling
I don't know how canonical this page is, but:
https://itanium-cxx-abi.github.io/cxx-abi/
has "Revised March 14, 2017", which seems relatively up-to-date, so this
patch updates the comment to point to that site.
OK for trunk?
libiberty/ChangeLog:
* cp-demangle.c: Update URL for g++ V3 ABI.
---
libiberty/cp-demangle.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libiberty/cp-demangle.c b/libiberty/cp-demangle.c
index 1d5b855..3f2a097 100644
--- a/libiberty/cp-demangle.c
+++ b/libiberty/cp-demangle.c
@@ -30,7 +30,7 @@
/* This code implements a demangler for the g++ V3 ABI. The ABI is
described on this web page:
- http://www.codesourcery.com/cxx-abi/abi.html#mangling
+ https://itanium-cxx-abi.github.io/cxx-abi/abi.html#mangling
This code was written while looking at the demangler written by
Alex Samuel <samuel@codesourcery.com>.
--
1.8.5.3
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] libiberty: fix URL for demangler ABI
2018-03-02 16:38 [PATCH] libiberty: fix URL for demangler ABI David Malcolm
@ 2018-03-02 16:41 ` Jeff Law
0 siblings, 0 replies; 2+ messages in thread
From: Jeff Law @ 2018-03-02 16:41 UTC (permalink / raw)
To: David Malcolm, gcc-patches
On 03/02/2018 09:44 AM, David Malcolm wrote:
> The comment at the top of cp-demangle.c lists:
> http://www.codesourcery.com/cxx-abi/abi.html#mangling
> which redirects to:
> https://mentorembedded.github.io/cxx-abi/abi.html#mangling
> but this is a 404: the latter site is empty.
>
> I looked on archive.org; the final version before the redirect
> seems to be:
> https://web.archive.org/web/20110903075217/http://www.codesourcery.com:80/public/cxx-abi/abi.html
>
> Some searching showed up:
> https://itanium-cxx-abi.github.io/cxx-abi/abi.html#mangling
>
> I don't know how canonical this page is, but:
> https://itanium-cxx-abi.github.io/cxx-abi/
> has "Revised March 14, 2017", which seems relatively up-to-date, so this
> patch updates the comment to point to that site.
>
> OK for trunk?
>
> libiberty/ChangeLog:
> * cp-demangle.c: Update URL for g++ V3 ABI.
OK.
jeff
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-03-02 16:41 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-02 16:38 [PATCH] libiberty: fix URL for demangler ABI David Malcolm
2018-03-02 16:41 ` Jeff Law
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).