[wwwdocs] Use whitespace between sourceware.org and gcc.gnu.org

2019-07-05 Thread Shakthi Kannan
Use "sourceware.org / gcc.gnu.org" for more clarity (bug #90334).

Shakthi Kannan

Index: htdocs/svnwrite.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/svnwrite.html,v
retrieving revision 1.43
diff -u -p -r1.43 svnwrite.html
--- htdocs/svnwrite.html12 Jan 2019 16:27:27 -  1.43
+++ htdocs/svnwrite.html5 Jul 2019 10:27:54 -
@@ -34,7 +34,7 @@ maintainers and significant developers.<
 be sponsored by an existing maintainer (someone with "write after approval"
 is not sufficient).

-If you already have an account on sourceware.org/gcc.gnu.org, ask
+If you already have an account on sourceware.org / gcc.gnu.org, ask
 overse...@gcc.gnu.org to add access to the GCC repository.
 Include the name of your sponsor and CC: her.
 If you do not have an account yet, use 

[PATCH, wwwdocs] Fix PR 50642

2013-04-30 Thread Shakthi Kannan
Hi,

The following patch overrides the default @smallexample attributes defined by 
makeinfo. It includes a html.css file where the smallexample attributes are 
defined to use a medium font size with a light blue background for readability 
and clarity.

2013-04-30 Shakthi Kannan 

PR wwwdocs/50642
* Makefile.in: Use --css-include with makeinfo.
* gcc/doc/include/html.css: New CSS file for HTML docs.

---
 Makefile.in  | 2 +-
 gcc/doc/include/html.css | 6 ++
 2 files changed, 7 insertions(+), 1 deletion(-)
 create mode 100644 gcc/doc/include/html.css

diff --git a/Makefile.in b/Makefile.in
index bfbaf03..ea70c0d 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -381,7 +381,7 @@ RUNTEST = @RUNTEST@
 # sub-makes.  It lets flags be given on the command line while still
 # using the makeinfo from the object tree.
 # (Default to avoid splitting info files by setting the threshold high.)
-MAKEINFOFLAGS = --split-size=500
+MAKEINFOFLAGS = --split-size=500 
--css-include=$$r/$(HOST_SUBDIR)/../gcc/doc/include/html.css
 
 # -
 # Programs producing files for the HOST machine
diff --git a/gcc/doc/include/html.css b/gcc/doc/include/html.css
new file mode 100644
index 000..6cc0284
--- /dev/null
+++ b/gcc/doc/include/html.css
@@ -0,0 +1,6 @@
+pre.smallexample { 
+font-family: sans-serif; 
+font-size: medium; 
+padding: 4px; 
+background: #f2f2f9 
+}
\ No newline at end of file
-- 
1.7.11.7


Re: [PATCH, wwwdocs] Fix PR 50642

2013-04-30 Thread Shakthi Kannan
Hi,

- Original Message -
| From: "Joseph S. Myers" 
|
| onlinedocs copies of manuals are generated by 
| maintainer-scripts/update_web_docs_svn, which doesn't use any makefiles.
\--

Could you please list the pre-requisite steps needed to run the above?

I had used "configure; make; make html; make install-html" to test the patch 
and the CSS.

Thanks for your reply.

SK

-- 
Shakthi Kannan 
skannan at redhat dot com


Re: [PATCH, wwwdocs] Fix PR 50642

2013-05-06 Thread Shakthi Kannan
Hi,

I am re-submitting the patch to use html.css from the update_web_docs_svn 
script present in the maintainer-scripts folder. The smallexample attributes 
are defined to use a medium font size with a light blue background for 
readability and clarity.

2013-05-06 Shakthi Kannan 

PR wwwdocs/50642
* maintainer-scripts/update_web_docs_svn: Use --css-include with makeinfo.
* gcc/doc/include/html.css: New CSS file for HTML docs.

---
 gcc/doc/include/html.css   | 6 ++
 maintainer-scripts/update_web_docs_svn | 4 +++-
 2 files changed, 9 insertions(+), 1 deletion(-)
 create mode 100644 gcc/doc/include/html.css

diff --git a/gcc/doc/include/html.css b/gcc/doc/include/html.css
new file mode 100644
index 000..6cc0284
--- /dev/null
+++ b/gcc/doc/include/html.css
@@ -0,0 +1,6 @@
+pre.smallexample { 
+font-family: sans-serif; 
+font-size: medium; 
+padding: 4px; 
+background: #f2f2f9 
+}
\ No newline at end of file
diff --git a/maintainer-scripts/update_web_docs_svn 
b/maintainer-scripts/update_web_docs_svn
index cfc33c4..2cea078 100755
--- a/maintainer-scripts/update_web_docs_svn
+++ b/maintainer-scripts/update_web_docs_svn
@@ -36,6 +36,8 @@ WWWBASE=/www/gcc/htdocs
 WWWBASE_PREFORMATTED=/www/gcc/htdocs-preformatted
 WWWPREPROCESS='/www/gcc/bin/preprocess -r'
 
+HTMLCSS=gcc/doc/include/html.css
+
 # Process options -rrelease and -ddirectory
 RELEASE=""
 SUBDIR=""
@@ -164,7 +166,7 @@ for file in $MANUALS; do
 if [ "$file" = "gnat_ugn_unw" ]; then
   includes="$includes -I gcc/gcc/ada"
 fi
-makeinfo --html $includes -o ${file} ${filename}
+makeinfo --css-include=../$HTMLCSS --html $includes -o ${file} ${filename}
 tar cf ${file}-html.tar ${file}/*.html
 texi2dvi $includes -o ${file}.dvi ${filename} /dev/null && 
dvips -o ${file}.ps ${file}.dvi
 texi2pdf $includes -o ${file}.pdf ${filename} 

[PATCH, PR preprocessor/42014] Added LAST_SOURCE_COLUMN in while loop

2013-05-09 Thread Shakthi Kannan
Hi,

The attached patch adds LAST_SOURCE_COLUMN to pp_verbatim
function in the while loop present in
diagnostic_report_current_module(). This makes the output
consistent for any error parsing program as stated in the bug.

2013-05-10 Shakthi Kannan 

PR preprocessor/42014
* gcc/diagnostic.c: Added LAST_SOURCE_COLUMN in while loop.

---
 gcc/diagnostic.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gcc/diagnostic.c b/gcc/diagnostic.c
index f9a236b..2addbf0 100644
--- a/gcc/diagnostic.c
+++ b/gcc/diagnostic.c
@@ -528,8 +528,9 @@ diagnostic_report_current_module (diagnostic_context 
*context, location_t where)
{
  map = INCLUDED_FROM (line_table, map);
  pp_verbatim (context->printer,
-  ",\n from %s:%d",
-  LINEMAP_FILE (map), LAST_SOURCE_LINE (map));
+  ",\n from %s:%d:%d",
+  LINEMAP_FILE (map),
+  LAST_SOURCE_LINE (map), LAST_SOURCE_COLUMN (map));
}
  pp_verbatim (context->printer, ":");
  pp_newline (context->printer);
-- 
1.7.11.7


Re: [PATCH, PR preprocessor/42014] Added LAST_SOURCE_COLUMN in while loop

2013-05-10 Thread Shakthi Kannan
Hi,

- Original Message -
| From: "Marek Polacek" 
|
| Maybe
|   PR preprocessor/42014
|   * diagnostic.c: Print LAST_SOURCE_COLUMN as well.
| ?
\--

Sure!

SK

-- 
Shakthi Kannan 
skannan at redhat dot com


Re: [PATCH, PR preprocessor/42014] Added LAST_SOURCE_COLUMN in while loop

2013-05-15 Thread Shakthi Kannan
Hi,

- Original Message -
| From: "Tom Tromey" 
|
| Does this cause test suite regressions?
\--

I built gcc-4.8.0 with and without the patch, and ran the test suite for both 
instances using:

  $ make -k check

There was no difference in the test result output. Is this the same as the test 
suite regression?

---
| If so then the patch needs fixes there.
| If not then the patch needs a new test.
\--

Sorry, where should this new test be written, and what does it need to test?

Thanks for your reply.

SK

-- 
Shakthi Kannan 
skannan at redhat dot com


Re: [PATCH, PR preprocessor/42014] Added LAST_SOURCE_COLUMN in while loop

2013-05-15 Thread Shakthi Kannan
Hi,

- Original Message -
| From: "Mike Stump" 
|
| The output of make -k, or the contents of all the .sum and .log files, 
\--

The output of "make -k". I did a "grep '^#'" on the output and the following 
results were the same, with and without the patch applied:

  # of expected passes  96818
  # of unexpected failures  19
  # of unexpected successes 39
  # of expected failures267
  # of unsupported tests1400
  # of expected passes  54561
  # of expected failures294
  # of unsupported tests916
  # of expected passes  44247
  # of unexpected failures  6
  # of expected failures56
  # of unresolved testcases 6
  # of unsupported tests71
  # of expected passes  2988
  # of expected failures6
  # of unsupported tests74
  # of expected passes  9340
  # of unexpected failures  4
  # of expected failures45
  # of unsupported tests212
  # of expected passes  1433
  # of unexpected failures  3
  # of expected passes  1819
  # of unsupported tests55
  # of expected passes  2582
  # of unexpected failures  4
  # of expected passes  12
  # of unsupported tests1
  # of expected passes  2998
  # of expected passes  26
  # of expected failures3
  # of unsupported tests1
  # of expected passes  54

---
| I like using ~/contrib/compare_tests gcc-before.sum gcc-after.sum to 
| determine if there are regressions.  You can also use that script to
| check for regressions between two build trees as well.  
\--

I ran the the script between the two build trees. Here is the output:

  $ ./without/gcc/contrib/compare_tests ./without/build ./with/build
  # Comparing directories
  ## Dir1=./without/build: 12 sum files
  ## Dir2=./with/build: 12 sum files

  # Comparing 12 common sum files
  ## /bin/sh ./without/gcc/contrib/compare_tests  /tmp/gxx-sum1.2065 
/tmp/gxx-sum2.2065
  # No differences found in 12 common sum files

Regards,

SK

-- 
Shakthi Kannan 
skannan at redhat dot com


*ping* Re: [PATCH, wwwdocs] Fix PR 50642

2013-05-21 Thread Shakthi Kannan
Hi,

Is the following patch okay for trunk?

SK

- Original Message -
From: "Shakthi Kannan" 
To: "Joseph S. Myers" 
Cc: gcc-patches@gcc.gnu.org
Sent: Monday, May 6, 2013 4:00:06 PM
Subject: Re: [PATCH, wwwdocs] Fix PR 50642

Hi,

I am re-submitting the patch to use html.css from the update_web_docs_svn 
script present in the maintainer-scripts folder. The smallexample attributes 
are defined to use a medium font size with a light blue background for 
readability and clarity.

2013-05-06 Shakthi Kannan 

PR wwwdocs/50642
* maintainer-scripts/update_web_docs_svn: Use --css-include with makeinfo.
* gcc/doc/include/html.css: New CSS file for HTML docs.

---
 gcc/doc/include/html.css   | 6 ++
 maintainer-scripts/update_web_docs_svn | 4 +++-
 2 files changed, 9 insertions(+), 1 deletion(-)
 create mode 100644 gcc/doc/include/html.css

diff --git a/gcc/doc/include/html.css b/gcc/doc/include/html.css
new file mode 100644
index 000..6cc0284
--- /dev/null
+++ b/gcc/doc/include/html.css
@@ -0,0 +1,6 @@
+pre.smallexample { 
+font-family: sans-serif; 
+font-size: medium; 
+padding: 4px; 
+background: #f2f2f9 
+}
\ No newline at end of file
diff --git a/maintainer-scripts/update_web_docs_svn 
b/maintainer-scripts/update_web_docs_svn
index cfc33c4..2cea078 100755
--- a/maintainer-scripts/update_web_docs_svn
+++ b/maintainer-scripts/update_web_docs_svn
@@ -36,6 +36,8 @@ WWWBASE=/www/gcc/htdocs
 WWWBASE_PREFORMATTED=/www/gcc/htdocs-preformatted
 WWWPREPROCESS='/www/gcc/bin/preprocess -r'
 
+HTMLCSS=gcc/doc/include/html.css
+
 # Process options -rrelease and -ddirectory
 RELEASE=""
 SUBDIR=""
@@ -164,7 +166,7 @@ for file in $MANUALS; do
 if [ "$file" = "gnat_ugn_unw" ]; then
   includes="$includes -I gcc/gcc/ada"
 fi
-makeinfo --html $includes -o ${file} ${filename}
+makeinfo --css-include=../$HTMLCSS --html $includes -o ${file} ${filename}
 tar cf ${file}-html.tar ${file}/*.html
 texi2dvi $includes -o ${file}.dvi ${filename} /dev/null && 
dvips -o ${file}.ps ${file}.dvi
 texi2pdf $includes -o ${file}.pdf ${filename} 

*ping* Re: [PATCH, PR preprocessor/42014] Added LAST_SOURCE_COLUMN in while loop

2013-05-21 Thread Shakthi Kannan
Hi,

Is the following patch okay for trunk?

SK

- Original Message -
From: "Shakthi Kannan" 
To: gcc-patches@gcc.gnu.org
Sent: Friday, May 10, 2013 12:25:07 PM
Subject: [PATCH, PR preprocessor/42014] Added LAST_SOURCE_COLUMN in while loop

Hi,

The attached patch adds LAST_SOURCE_COLUMN to pp_verbatim
function in the while loop present in
diagnostic_report_current_module(). This makes the output
consistent for any error parsing program as stated in the bug.

2013-05-10 Shakthi Kannan 

PR preprocessor/42014
* gcc/diagnostic.c: Added LAST_SOURCE_COLUMN in while loop.

---
 gcc/diagnostic.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gcc/diagnostic.c b/gcc/diagnostic.c
index f9a236b..2addbf0 100644
--- a/gcc/diagnostic.c
+++ b/gcc/diagnostic.c
@@ -528,8 +528,9 @@ diagnostic_report_current_module (diagnostic_context 
*context, location_t where)
{
  map = INCLUDED_FROM (line_table, map);
  pp_verbatim (context->printer,
-  ",\n from %s:%d",
-  LINEMAP_FILE (map), LAST_SOURCE_LINE (map));
+  ",\n from %s:%d:%d",
+  LINEMAP_FILE (map),
+  LAST_SOURCE_LINE (map), LAST_SOURCE_COLUMN (map));
}
  pp_verbatim (context->printer, ":");
  pp_newline (context->printer);
-- 
1.7.11.7

-- 
Shakthi Kannan 
skannan at redhat dot com


[PATCH] Fix PR 55473

2013-03-05 Thread Shakthi Kannan
The libquadmath/quadmath.h file cannot be used with C++. The
following patch allows inclusion and use of the quadmath.h header
file.

2013-03-06 Shakthi Kannan 

PR libquadmath/55473
* quadmath.h: Add ifdef __cplusplus macros.

---
 libquadmath/quadmath.h | 8 
 1 file changed, 8 insertions(+)

diff --git a/libquadmath/quadmath.h b/libquadmath/quadmath.h
index 863fe44..aa9ef51 100644
--- a/libquadmath/quadmath.h
+++ b/libquadmath/quadmath.h
@@ -23,6 +23,10 @@ Boston, MA 02110-1301, USA.  */

 #include 

+#ifdef __cplusplus
+extern "C" {
+#endif
+
 /* Define the complex type corresponding to __float128
("_Complex __float128" is not allowed) */
 typedef _Complex float __attribute__((mode(TC))) __complex128;
@@ -189,4 +193,8 @@ __quadmath_nth (conjq (__complex128 __z))
   return __extension__ ~__z;
 }

+#ifdef __cplusplus
+}
+#endif
+
 #endif
-- 
1.7.11.7