Re: [PATCH] libstdc++: std/ranges - Remove a duplicate define directive

2024-01-11 Thread Jonathan Wakely
On Wed, 10 Jan 2024 at 23:31, Jonathan Wakely  wrote:
>
> On Wed, 10 Jan 2024 at 21:28, Michael Levine (BLOOMBERG/ 120 PARK)
>  wrote:
> >
> > From a67cfd07ce27a62f764b381268502acb68b6bad9 Mon Sep 17 00:00:00 2001
> > From: Michael Levine 
> > Date: Wed, 10 Jan 2024 15:48:46 -0500
> > Subject: [PATCH 1/2] Removed a duplicate define directive for
> > __glibcxx_want_ranges_iota
> >
> > Signed-off-by: Michael Levine 
> > ---
> > libstdc++-v3/include/std/ranges | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/libstdc++-v3/include/std/ranges 
> > b/libstdc++-v3/include/std/ranges
> > index 81a857502e3..ae857f8c7fc 100644
> > --- a/libstdc++-v3/include/std/ranges
> > +++ b/libstdc++-v3/include/std/ranges
> > @@ -59,7 +59,6 @@
> > #define __glibcxx_want_ranges_chunk_by
> > #define __glibcxx_want_ranges_enumerate
> > #define __glibcxx_want_ranges_iota
> > -#define __glibcxx_want_ranges_iota
> > #define __glibcxx_want_ranges_join_with
> > #define __glibcxx_want_ranges_repeat
> > #define __glibcxx_want_ranges_slide
>
> Thanks for the patch, this looks obviously correct.
>
> Please note that all patches need to go to the gcc-patches list (as
> well as the more specific list, like this one, so e.g. add both to the
> email's To: header, or CC one of them).
>
> Also please don't touch the ChangeLog file in patches. We have been
> auto-generating the ChangeLog files nightly for some years now, so you
> should not edit them manually (and including it in the patch just
> means the patch won't apply cleanly after the next time the file is
> regen'd).
>
> But I can take care of that and apply the patch, there's no need for a
> corrected patch. Thanks again for the contribution.

Pushed to trunk now, thanks again.



Re: [PATCH] libstdc++: std/ranges - Remove a duplicate define directive

2024-01-10 Thread Jonathan Wakely
On Wed, 10 Jan 2024 at 21:28, Michael Levine (BLOOMBERG/ 120 PARK)
 wrote:
>
> From a67cfd07ce27a62f764b381268502acb68b6bad9 Mon Sep 17 00:00:00 2001
> From: Michael Levine 
> Date: Wed, 10 Jan 2024 15:48:46 -0500
> Subject: [PATCH 1/2] Removed a duplicate define directive for
> __glibcxx_want_ranges_iota
>
> Signed-off-by: Michael Levine 
> ---
> libstdc++-v3/include/std/ranges | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/libstdc++-v3/include/std/ranges b/libstdc++-v3/include/std/ranges
> index 81a857502e3..ae857f8c7fc 100644
> --- a/libstdc++-v3/include/std/ranges
> +++ b/libstdc++-v3/include/std/ranges
> @@ -59,7 +59,6 @@
> #define __glibcxx_want_ranges_chunk_by
> #define __glibcxx_want_ranges_enumerate
> #define __glibcxx_want_ranges_iota
> -#define __glibcxx_want_ranges_iota
> #define __glibcxx_want_ranges_join_with
> #define __glibcxx_want_ranges_repeat
> #define __glibcxx_want_ranges_slide

Thanks for the patch, this looks obviously correct.

Please note that all patches need to go to the gcc-patches list (as
well as the more specific list, like this one, so e.g. add both to the
email's To: header, or CC one of them).

Also please don't touch the ChangeLog file in patches. We have been
auto-generating the ChangeLog files nightly for some years now, so you
should not edit them manually (and including it in the patch just
means the patch won't apply cleanly after the next time the file is
regen'd).

But I can take care of that and apply the patch, there's no need for a
corrected patch. Thanks again for the contribution.


> --
> 2.25.1
>
>
> From 204b7ae1e403f86208e781ff2ca68df213d3104b Mon Sep 17 00:00:00 2001
> From: Michael Levine 
> Date: Wed, 10 Jan 2024 16:15:16 -0500
> Subject: [PATCH 2/2] Updated the changelog with the change from the previous
> commit
>
> Signed-off-by: Michael Levine 
> ---
> contrib/ChangeLog | 11 ---
> 1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/contrib/ChangeLog b/contrib/ChangeLog
> index 04bde02b65b..11c9c1178bc 100644
> --- a/contrib/ChangeLog
> +++ b/contrib/ChangeLog
> @@ -1,3 +1,8 @@
> +2024-01-10 Michael Levine 
> +
> + * libstdc++-v3/include/std/ranges: Remove a duplicate define directive
> + for __glibcxx_want_ranges_iota.
> +
> 2024-01-09 Jonathan Wakely 
>
> * unicode/gen_libstdcxx_unicode_data.py: Print out Gcb_property
> @@ -2324,8 +2329,8 @@
>
> * update-copyright.py (LibJavaFilter): Remove.
>
> -2017-01-17 Gerald Pfeifer 
> -
> +2017-01-17 Gerald Pfeifer 
> +
> * gcc_update: Remove entries related to GCJ and libgcj.
> Complete copyright years and adjust my e-mail address.
>
> @@ -2362,7 +2367,7 @@
>
> PR other/61439
> * download_prerequisites: Conditionally verify checksums of
> - downloaded pacakges. Add help text.
> + downloaded pacakges. Add help text.
> * prerequisites.md5: New file.
> * prerequisites.sha512: New file.
>
> --
> 2.25.1