public inbox for fortran@gcc.gnu.org
 help / color / mirror / Atom feed
* gfortran online docs: small typo in RESULT_IMAGE description
@ 2021-01-21 12:22 Jorge D'Elia
  2021-01-21 13:28 ` [Patch, committed] gcc/fortran/intrinsic.texi: Fix typo (was: gfortran online docs: small typo in RESULT_IMAGE description) Tobias Burnus
  0 siblings, 1 reply; 4+ messages in thread
From: Jorge D'Elia @ 2021-01-21 12:22 UTC (permalink / raw)
  To: Gfortran List

Hi all,

A small typo in https://gcc.gnu.org/onlinedocs/gfortran/CO_005fREDUCE.html#CO_005fREDUCE

at the RESULT_IMAGE description:

"(optional) a scalar integer expression; if present, it shall have the same the same value on all images ..."
                                                                   ^^^^^^^^ ^^^^^^^^
Regards.
JorgeD.
--

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Patch, committed] gcc/fortran/intrinsic.texi: Fix typo (was: gfortran online docs: small typo in RESULT_IMAGE description)
  2021-01-21 12:22 gfortran online docs: small typo in RESULT_IMAGE description Jorge D'Elia
@ 2021-01-21 13:28 ` Tobias Burnus
  2021-01-25 10:42   ` Jorge D'Elia
  0 siblings, 1 reply; 4+ messages in thread
From: Tobias Burnus @ 2021-01-21 13:28 UTC (permalink / raw)
  To: Jorge D'Elia, Gfortran List, gcc-patches

[-- Attachment #1: Type: text/plain, Size: 561 bytes --]

On 21.01.21 13:22, Jorge D'Elia via Fortran wrote:

> A small typo in https://gcc.gnu.org/onlinedocs/gfortran/CO_005fREDUCE.html#CO_005fREDUCE

Thanks for the report – I regarded it as (almost) patch and, hence, used
you as author :-)

Committed as r11-6835-g9be0a89c95cc30089786faa26b89e8d7444c879e

Thanks,

Tobias

PS: The bug existed since 2014 ...

-----------------
Mentor Graphics (Deutschland) GmbH, Arnulfstraße 201, 80634 München / Germany
Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Alexander Walter

[-- Attachment #2: committed.diff --]
[-- Type: text/x-patch, Size: 1122 bytes --]

commit 9be0a89c95cc30089786faa26b89e8d7444c879e
Author:     Jorge D'Elia <jdelia@cimec.unl.edu.ar>
AuthorDate: Thu Jan 21 14:23:28 2021 +0100
Commit:     Tobias Burnus <tobias@codesourcery.com>
CommitDate: Thu Jan 21 14:24:27 2021 +0100

    gcc/fortran/intrinsic.texi: Fix typo
    
    gcc/fortran/ChangeLog:
    
            * intrinsic.texi (CO_MAX): Fix typo.

diff --git a/gcc/fortran/intrinsic.texi b/gcc/fortran/intrinsic.texi
index acf1cae3921..6e6aa26c25c 100644
--- a/gcc/fortran/intrinsic.texi
+++ b/gcc/fortran/intrinsic.texi
@@ -3753,7 +3753,7 @@ Collective subroutine
 @item @var{A}            @tab shall be an integer, real or character variable,
 which has the same type and type parameters on all images of the team.
 @item @var{RESULT_IMAGE} @tab (optional) a scalar integer expression; if
-present, it shall have the same the same value on all images and refer to an
+present, it shall have the same value on all images and refer to an
 image of the current team.
 @item @var{STAT}         @tab (optional) a scalar integer variable
 @item @var{ERRMSG}       @tab (optional) a scalar character variable

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Patch, committed] gcc/fortran/intrinsic.texi: Fix typo (was: gfortran online docs: small typo in RESULT_IMAGE description)
  2021-01-21 13:28 ` [Patch, committed] gcc/fortran/intrinsic.texi: Fix typo (was: gfortran online docs: small typo in RESULT_IMAGE description) Tobias Burnus
@ 2021-01-25 10:42   ` Jorge D'Elia
  2021-01-25 13:48     ` Tobias Burnus
  0 siblings, 1 reply; 4+ messages in thread
From: Jorge D'Elia @ 2021-01-25 10:42 UTC (permalink / raw)
  To: Tobias Burnus; +Cc: Gfortran List, GCC Patches

Hi Tobias,

----- Mensaje original -----
> De: "Tobias Burnus" 
> Para: "jdelia" <jdelia@cimec.unl.edu.ar>, "Gfortran List" , "GCC Patches" 
> CC: "Jorge D'Elia" <jdelia@intec.unl.edu.ar>
> Enviado: Jueves, 21 de Enero 2021 10:28:02
> Asunto: [Patch, committed] gcc/fortran/intrinsic.texi: Fix typo (was: gfortran online docs: small typo in RESULT_IMAGE
> description)
>
> On 21.01.21 13:22, Jorge D'Elia via Fortran wrote:
> 
>> A small typo in
>> https://gcc.gnu.org/onlinedocs/gfortran/CO_005fREDUCE.html#CO_005fREDUCE
> 
> Thanks for the report – I regarded it as (almost) patch and, hence, used
> you as author :-)

Ok, non problem.

> Committed as r11-6835-g9be0a89c95cc30089786faa26b89e8d7444c879e
>
> Thanks,
> 
> Tobias
> 
> PS: The bug existed since 2014 ...

Yes, I remember that typo for a while. 
By the way, when will the update be available in https://gcc.gnu.org/onlinedocs ? 
e.g. at least until today it is still not seen...

Regards.
JorgeD.

> -----------------
> Mentor Graphics (Deutschland) GmbH, Arnulfstraße 201, 80634 München / Germany
> Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Alexander
> Walter

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Patch, committed] gcc/fortran/intrinsic.texi: Fix typo (was: gfortran online docs: small typo in RESULT_IMAGE description)
  2021-01-25 10:42   ` Jorge D'Elia
@ 2021-01-25 13:48     ` Tobias Burnus
  0 siblings, 0 replies; 4+ messages in thread
From: Tobias Burnus @ 2021-01-25 13:48 UTC (permalink / raw)
  To: Jorge D'Elia; +Cc: Gfortran List, GCC Patches

[-- Attachment #1: Type: text/plain, Size: 843 bytes --]

Hi Jorge, hi all,

On 25.01.21 11:42, Jorge D'Elia wrote:

>> Committed as r11-6835-g9be0a89c95cc30089786faa26b89e8d7444c879e
> By the way, when will the update be available in https://gcc.gnu.org/onlinedocs ?
> e.g. at least until today it is still not seen...

It turned out that the issue was in multiple intrinsics – I thought I
had correctly grepped but seemingly, I haven't. (CO_MAX was fixed.) — I
also fixed another copy'n'paste error (one 'co_min' instead of 'co_sum').

Thus, I now fixed the others – and it hopefully shows up tomorrow.

Thanks for the initial report and insisting that it should be fixed ...

Tobias

-----------------
Mentor Graphics (Deutschland) GmbH, Arnulfstraße 201, 80634 München / Germany
Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Alexander Walter

[-- Attachment #2: committed.diff --]
[-- Type: text/x-patch, Size: 3110 bytes --]

commit 10c83fb7131779ee27fbaa9091065ca6cfedc0c5
Author:     Tobias Burnus <tobias@codesourcery.com>
AuthorDate: Mon Jan 25 14:40:33 2021 +0100
Commit:     Tobias Burnus <tobias@codesourcery.com>
CommitDate: Mon Jan 25 14:43:52 2021 +0100

    gcc/fortran/intrinsic.texi: Fix typos
    
    gcc/fortran/ChangeLog:
    
            * intrinsic.texi (CO_BROADCAST, CO_MIN, CO_REDUCE, CO_SUM): Fix typos.

diff --git a/gcc/fortran/intrinsic.texi b/gcc/fortran/intrinsic.texi
index 6e6aa26c25c..5debe94773b 100644
--- a/gcc/fortran/intrinsic.texi
+++ b/gcc/fortran/intrinsic.texi
@@ -3696,7 +3696,7 @@ Collective subroutine
 dynamic type and type parameters on all images of the current team. If it
 is an array, it shall have the same shape on all images.
 @item @var{SOURCE_IMAGE} @tab a scalar integer expression.
-It shall have the same the same value on all images and refer to an
+It shall have the same value on all images and refer to an
 image of the current team.
 @item @var{STAT}         @tab (optional) a scalar integer variable
 @item @var{ERRMSG}       @tab (optional) a scalar character variable
@@ -3810,7 +3810,7 @@ Collective subroutine
 @item @var{A}            @tab shall be an integer, real or character variable,
 which has the same type and type parameters on all images of the team.
 @item @var{RESULT_IMAGE} @tab (optional) a scalar integer expression; if
-present, it shall have the same the same value on all images and refer to an
+present, it shall have the same value on all images and refer to an
 image of the current team.
 @item @var{STAT}         @tab (optional) a scalar integer variable
 @item @var{ERRMSG}       @tab (optional) a scalar character variable
@@ -3881,7 +3881,7 @@ all images and with regards to the arguments mathematically commutative and
 associative.  Note that @var{OPERATOR} may not be an elemental function, unless
 it is an intrisic function.
 @item @var{RESULT_IMAGE} @tab (optional) a scalar integer expression; if
-present, it shall have the same the same value on all images and refer to an
+present, it shall have the same value on all images and refer to an
 image of the current team.
 @item @var{STAT}         @tab (optional) a scalar integer variable
 @item @var{ERRMSG}       @tab (optional) a scalar character variable
@@ -3943,14 +3943,14 @@ Technical Specification (TS) 18508 or later
 Collective subroutine
 
 @item @emph{Syntax}:
-@code{CALL CO_MIN(A [, RESULT_IMAGE, STAT, ERRMSG])}
+@code{CALL CO_SUM(A [, RESULT_IMAGE, STAT, ERRMSG])}
 
 @item @emph{Arguments}:
 @multitable @columnfractions .20 .65
 @item @var{A}            @tab shall be an integer, real or complex variable,
 which has the same type and type parameters on all images of the team.
 @item @var{RESULT_IMAGE} @tab (optional) a scalar integer expression; if
-present, it shall have the same the same value on all images and refer to an
+present, it shall have the same value on all images and refer to an
 image of the current team.
 @item @var{STAT}         @tab (optional) a scalar integer variable
 @item @var{ERRMSG}       @tab (optional) a scalar character variable

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-01-25 13:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-21 12:22 gfortran online docs: small typo in RESULT_IMAGE description Jorge D'Elia
2021-01-21 13:28 ` [Patch, committed] gcc/fortran/intrinsic.texi: Fix typo (was: gfortran online docs: small typo in RESULT_IMAGE description) Tobias Burnus
2021-01-25 10:42   ` Jorge D'Elia
2021-01-25 13:48     ` Tobias Burnus

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).