public inbox for fortran@gcc.gnu.org
 help / color / mirror / Atom feed
* build warnings
@ 2018-09-25 10:25 Bernhard Reutner-Fischer
  2018-09-25 19:32 ` Thomas Koenig
  0 siblings, 1 reply; 2+ messages in thread
From: Bernhard Reutner-Fischer @ 2018-09-25 10:25 UTC (permalink / raw)
  To: gfortran

Hi!

trunk@264126 JFYI
Would be nice if someone familiar with CAF could take a look at 2)

1)
fortran/gfortran.texi:2743: warning: @ref node name should not contain `:'

2)
libgfortran/caf/single.c:2813:57: warning: passing argument 3 of
‘_gfortran_caf_get_by_ref’ from incompatible pointer type
[-Wincompatible-pointer-types]
2813 |   _gfortran_caf_get_by_ref (src_token, src_image_index, &temp, src_refs,
     |                                                         ^~~~~
     |                                                         |
     |                                                         struct
<anonymous> *
libgfortran/caf/single.c:1544:24: note: expected ‘gfc_descriptor_t *’
{aka ‘struct <anonymous> *’} but argument is of type ‘struct
<anonymous> *’
1544 |      gfc_descriptor_t *dst, caf_reference_t *refs,
     |      ~~~~~~~~~~~~~~~~~~^~~
libgfortran/caf/single.c:2820:58: warning: passing argument 3 of
‘_gfortran_caf_send_by_ref’ from incompatible pointer type
[-Wincompatible-pointer-types]
2820 |   _gfortran_caf_send_by_ref (dst_token, dst_image_index, &temp, dst_refs,
     |                                                          ^~~~~
     |                                                          |
     |                                                          struct
<anonymous> *
libgfortran/caf/single.c:2434:25: note: expected ‘gfc_descriptor_t *’
{aka ‘struct <anonymous> *’} but argument is of type ‘struct
<anonymous> *’
2434 |       gfc_descriptor_t *src, caf_reference_t *refs,
     |       ~~~~~~~~~~~~~~~~~~^~~

3)
libgfortran/generated/minloc0_4_i1.c:138:5: warning: this ‘else’
clause does not guard... [-Wmisleading-indentation]
138 |     else
    |     ^~~~
../../../../src/gcc-9.0.mine/libgfortran/generated/minloc0_4_i1.c:152:7:
note: ...this statement, but the latter is misleadingly indented as if
it were guarded by the ‘else’
152 |       n = 0;
    |       ^
(likewise for the other expansion points, elided here)

4)
libgfortran/io/transfer.c:2451:35: warning: operand of ?: changes
signedness from ‘long int’ to ‘size_t’ {aka ‘long unsigned int’} due
to unsignedness of other operand [-Wsign-compare]
2451 |   size = iotype == BT_CHARACTER ? (index_type) charlen :
GFC_DESCRIPTOR_SIZE (desc);
     |                                   ^~~~~~~~~~~~~~~~~~~~

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

* Re: build warnings
  2018-09-25 10:25 build warnings Bernhard Reutner-Fischer
@ 2018-09-25 19:32 ` Thomas Koenig
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Koenig @ 2018-09-25 19:32 UTC (permalink / raw)
  To: Bernhard Reutner-Fischer, gfortran

Hi Bernhard,

> 3)
> libgfortran/generated/minloc0_4_i1.c:138:5: warning: this ‘else’
> clause does not guard... [-Wmisleading-indentation]

Ugh. Since these are generated files (by m4, no less) fixing the
indentation is not something that I would do voluntarily.

However, volunteers are welcome :-)

Regards

	Thomas

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

end of thread, other threads:[~2018-09-25 19:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-25 10:25 build warnings Bernhard Reutner-Fischer
2018-09-25 19:32 ` Thomas Koenig

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