[Patch, Fortran] PR fortran/37193: USE m, ONLY: i, j => i

Tobias Burnus burnus@net-b.de
Mon Sep 1 07:37:00 GMT 2008


Daniel Kraft wrote:
> Paul Richard Thomas wrote:
>> Your patch looks OK
Ditto from my side.

>> Why does the bit of code in module.c(load_needed)....
>>   mio_symbol (sym);
>>   sym->attr.use_assoc = 1;
>>   if (only_flag)
>>     sym->attr.use_only = 1;
>>   if (p->u.rsym.renamed)
>>     sym->attr.use_rename = 1;
>>
>>   return 1;
>>
>> not do the job?  Is it needed now, with your patch?
> If you want I can replace the second initialization (the one in your 
> snippet) by an assertion and see if the test-suite still passes and if 
> it does leave the assertion there.
I think one should do this: For the trunk, such a patch is OK.

For 4.3 I would wait a week or so and add then only the new line without 
replacing the old lines with the assert.

Tobias



More information about the Fortran mailing list