[Bug go/90669] go/gofrontend/types.cc:2805 contains range-based ‘for’ loops which are not C++98

2019-05-30 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90669

--- Comment #4 from Jonathan Wakely  ---
shabits.assign(digest.begin(), digest.end()) would've worked too, and not
required any vector reallocations as it grows.

[Bug go/90669] go/gofrontend/types.cc:2805 contains range-based ‘for’ loops which are not C++98

2019-05-29 Thread ian at airs dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90669

Ian Lance Taylor  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #3 from Ian Lance Taylor  ---
Thanks.  Fixed.

[Bug go/90669] go/gofrontend/types.cc:2805 contains range-based ‘for’ loops which are not C++98

2019-05-29 Thread ian at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90669

--- Comment #2 from ian at gcc dot gnu.org  ---
Author: ian
Date: Thu May 30 00:57:11 2019
New Revision: 271761

URL: https://gcc.gnu.org/viewcvs?rev=271761=gcc=rev
Log:
PR go/90669
compiler: remove range-based 'for' loop

Fix for GCC PR/90669: remove range-based 'for' loop to preserve
buildability with g++ version 4.X.

Reviewed-on: https://go-review.googlesource.com/c/gofrontend/+/179397

Modified:
trunk/gcc/go/gofrontend/MERGE
trunk/gcc/go/gofrontend/types.cc

[Bug go/90669] go/gofrontend/types.cc:2805 contains range-based ‘for’ loops which are not C++98

2019-05-29 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90669

Jakub Jelinek  changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #1 from Jakub Jelinek  ---
Range for is just syntactic sugar, so shouldn't be hard to rewrite that into
valid C++98 using explicit iterator.