Re: *ping* [patch, fortran] Fix PR 60355, missing error for BIND(C) outside module scope

2017-08-11 Thread Thomas Koenig

Hi Paul,


It looks fine to me. OK for trunk.



Thanks, committed.


Paul

PS What about PR34640 ?


Your patch is OK if you bump the library version.

Regards

Thomas


Re: *ping* [patch, fortran] Fix PR 60355, missing error for BIND(C) outside module scope

2017-08-10 Thread Paul Richard Thomas
Hi Thomas,

It looks fine to me. OK for trunk.

Thanks

Paul

PS What about PR34640 ?


On 9 August 2017 at 20:44, Thomas Koenig  wrote:
> Am 02.08.2017 um 15:19 schrieb Thomas Koenig:
>>
>> the attached patch is a bit smaller than it looks, because most of
>> it is due to reformatting a large comment.  It is rather simple -
>> checking for an incorrectly placed BIND(C) variable was sometimes not
>> done because the test was mixed in with other tests where implicitly
>> typed variables were excluded.
>>
>> Regression-tested. OK for trunk?
>
>
> Ping.
>
> Patch at:
>
> https://gcc.gnu.org/ml/fortran/2017-08/msg00010.html
>
> Regards
>
> Thomas



-- 
"If you can't explain it simply, you don't understand it well enough"
- Albert Einstein


*ping* [patch, fortran] Fix PR 60355, missing error for BIND(C) outside module scope

2017-08-09 Thread Thomas Koenig

Am 02.08.2017 um 15:19 schrieb Thomas Koenig:

the attached patch is a bit smaller than it looks, because most of
it is due to reformatting a large comment.  It is rather simple -
checking for an incorrectly placed BIND(C) variable was sometimes not
done because the test was mixed in with other tests where implicitly
typed variables were excluded.

Regression-tested. OK for trunk?


Ping.

Patch at:

https://gcc.gnu.org/ml/fortran/2017-08/msg00010.html

Regards

Thomas