Re: [wwwdocs] [PATCH] Add "id" attributes to the headings in the GCC 6 porting guide

2016-03-02 Thread Gerald Pfeifer
On Wed, 2 Mar 2016, David Malcolm wrote:
> Committed to CVS, and it appears to be live on the website, so links
> like the following now work:
> https://gcc.gnu.org/gcc-6/porting_to.html#Wmisleading-indentation

Yes.  Updates to gcc.gnu.org happen instantaneously, whereas
the site at www.gnu.org/software/gcc/ syncs once a day.

Gerald


Re: [wwwdocs] [PATCH] Add "id" attributes to the headings in the GCC 6 porting guide

2016-03-02 Thread David Malcolm
On Wed, 2016-03-02 at 00:10 +0100, Gerald Pfeifer wrote:
> Hi David,
> 
> On Tue, 1 Mar 2016, David Malcolm wrote:
> > The attached patch adds "id" attributes to every h2, h3 and h4
> > element
> > in the page, so that people can create URLs that reference specific
> > subsections of the guide.
> 
> this is a great idea!
> 
> Allow me to suggest some potential tweaks wrt to the actual choices.
> 
> The patch is fine if you consider these (which is not the same is
> necessarily implementing all of them though I might be disappointed
> if you discard all of them ;-).
> 
> +C++ language issues
> 
> Here I believe "cxx" is more natural/common.
Indeed, this is what the changes.html document uses.

Changed.

> +Default standard is now GNU++14
> 
> "gxx14", perhaps?  Consider this a weaker recommendation than "cxx"
> above, though.

Changed.

> +Call of overloaded 'abs(unsigned
> int)' is ambiguous
> 
> Somehow I couldn't help thinking about a six packs here. ;-)
> 
> +Deprecation of
> std::auto_ptr
> 
> Here, how about just using "auto_ptr"?

I didn't bother with the other two tweaks.

Thanks.

Committed to CVS, and it appears to be live on the website, so links
like the following now work:
https://gcc.gnu.org/gcc-6/porting_to.html#Wmisleading-indentation



Re: [wwwdocs] [PATCH] Add "id" attributes to the headings in the GCC 6 porting guide

2016-03-01 Thread Jeff Law

On 03/01/2016 12:41 PM, David Malcolm wrote:

htdocs/gcc-6/porting_to.html is now multiple screens of text.

The attached patch adds "id" attributes to every h2, h3 and h4 element
in the page, so that people can create URLs that reference specific
subsections of the guide.

Validates.

OK to commit?

OK.
jeff


Re: [wwwdocs] [PATCH] Add "id" attributes to the headings in the GCC 6 porting guide

2016-03-01 Thread Gerald Pfeifer
Hi David,

On Tue, 1 Mar 2016, David Malcolm wrote:
> The attached patch adds "id" attributes to every h2, h3 and h4 element
> in the page, so that people can create URLs that reference specific
> subsections of the guide.

this is a great idea!

Allow me to suggest some potential tweaks wrt to the actual choices.

The patch is fine if you consider these (which is not the same is
necessarily implementing all of them though I might be disappointed
if you discard all of them ;-).

+C++ language issues

Here I believe "cxx" is more natural/common.

+Default standard is now GNU++14

"gxx14", perhaps?  Consider this a weaker recommendation than "cxx"
above, though.

+Call of overloaded 'abs(unsigned 
int)' is ambiguous

Somehow I couldn't help thinking about a six packs here. ;-)

+Deprecation of std::auto_ptr

Here, how about just using "auto_ptr"?

Gerald


[wwwdocs] [PATCH] Add "id" attributes to the headings in the GCC 6 porting guide

2016-03-01 Thread David Malcolm
htdocs/gcc-6/porting_to.html is now multiple screens of text.

The attached patch adds "id" attributes to every h2, h3 and h4 element
in the page, so that people can create URLs that reference specific
subsections of the guide.

Validates.

OK to commit?Index: htdocs/gcc-6/porting_to.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/porting_to.html,v
retrieving revision 1.19
diff -u -p -r1.19 porting_to.html
--- htdocs/gcc-6/porting_to.html	1 Mar 2016 19:22:05 -	1.19
+++ htdocs/gcc-6/porting_to.html	1 Mar 2016 19:36:49 -
@@ -25,15 +25,15 @@ manner. Additions and suggestions for im
 
 
 
-Preprocessor issues
+Preprocessor issues
 
 
-C language issues
+C language issues
 
 
-C++ language issues
+C++ language issues
 
-Default standard is now GNU++14
+Default standard is now GNU++14
 
 
 GCC 6 defaults to -std=gnu++14 instead of -std=gnu++98:
@@ -51,7 +51,7 @@ in CXXFLAGS or similar vari
 changes but not the C++14 ones.  If so, use the -std=gnu++11
 command-line option.
 
-Narrowing conversions
+Narrowing conversions
 
 
 The C++11 standard does not allow "narrowing conversions" inside braced
@@ -71,7 +71,7 @@ conversion. Narrowing conversions can be
 e.g. (char)i.
 
 
-Invalid literal suffixes
+Invalid literal suffixes
 
 
 The C++11 "user-defined literals" feature allows custom suffixes to be added
@@ -84,7 +84,7 @@ the code to compile in C++11 add whitesp
 macro: printf("%" PRIu64, uint64_value).
 
 
-Cannot convert 'bool' to 'T*'
+Cannot convert 'bool' to 'T*'
 
 
 The current C++ standard only allows integer literals to be used as null
@@ -94,7 +94,7 @@ fails to compile with this error should 
 or 0, or NULL.
 
 
-Cannot convert 'std::ostream' to 'bool'
+Cannot convert 'std::ostream' to 'bool'
 
 
 As of C++11, iostream classes are no longer implicitly convertible to
@@ -111,7 +111,7 @@ or
 return static_castbool(os);
 
 
-Lvalue required as left operand of assignment with complex numbers
+Lvalue required as left operand of assignment with complex numbers
 
 
 Since C++11 (as per DR#387) the member functions real() and 
@@ -132,7 +132,7 @@ following should be used instead:
   f.real (val);
 
 
-Destructors are noexcept by default
+Destructors are noexcept by default
 
 
 As of C++11, destructors have an implicit noexcept
@@ -168,7 +168,7 @@ behavior when defining the destructor li
 ~S() noexcept(false) { throw std::runtime_error ("oops"); }
 
 
-Header dependency changes
+Header dependency changes
 
 
 The algorithm header has been changed to reduce the
@@ -179,7 +179,7 @@ As such, C++ programs that used componen
 will no longer compile.
 
 
-Header cmath changes
+Header cmath changes
 
 
 Some C libraries declare obsolete int isinf(double) or
@@ -191,7 +191,7 @@ will use them and import them into names
 instead of defining the correct signatures.
 
 
-Header math.h changes
+Header math.h changes
 
 
 The C++ library now provides its own math.h header that
@@ -202,7 +202,7 @@ Code which assumes that sin
 isfinite etc. are macros may no longer compile.
 
 
-Header stdlib.h changes
+Header stdlib.h changes
 
 
 The C++ library now provides its own stdlib.h header that
@@ -220,7 +220,7 @@ responsible for ensuring their headers w
 the C++ standard library.
 
 
-Call of overloaded 'abs(unsigned int)' is ambiguous
+Call of overloaded 'abs(unsigned int)' is ambiguous
 
 
 The additional overloads can cause the compiler to reject invalid code that
@@ -241,7 +241,7 @@ Since calling abs() on an u
 this code will become explicitly invalid as per discussion in the LWG.
 
 
-Optimizations remove null pointer checks for this
+Optimizations remove null pointer checks for this
 
 
 When optimizing, GCC now assumes the this pointer can never be
@@ -260,7 +260,7 @@ this optimization. That option also disa
 pointers, not only those involving this.
 
 
-Deprecation of std::auto_ptr
+Deprecation of std::auto_ptr
 
 
 The std::auto_ptr class template was deprecated in C++11, so GCC
@@ -269,7 +269,7 @@ now warns about its usage.  This warning
 to port the code to use C++11's std::unique_ptr instead.
 
 
-'constexpr' needed for in-class initialization of static data member
+'constexpr' needed for in-class initialization of static data member
 
 
 Since C++11, the constexpr keyword is needed when initializing a
@@ -290,7 +290,7 @@ C++11 or later.  Programs relying on the
 error.  The fix is to use constexpr instead of const.
 
 
-Stricter flexible array member rules
+Stricter flexible array member rules
 
 
 As of this release, the C++ compiler is now more strict about flexible array
@@ -324,7 +324,7 @@ to T[0]).  This is a silent ABI
 -fabi-version or -Wabi option to disable or warn about.
 
 
-More aggressive optimization of -flifetime-dse
+More aggressive optimization of -flifetime-dse
 
 
 The C++ compiler (with enabled -flifetime-dse)
@@ -373,7 +373,7 @@ the option -flifetime-dse=1
 this optimization.
 
 
--Wmisleading-indentation