RE: [PATCH, FORTRAN] Class Name Clash

2014-04-10 Thread Bernd Edlinger
Hi, On Thu, 10 Apr 2014 17:15:16, Tobias Burnus wrote: > > Hi Bernd, > > On Thu, Apr 10, 2014 at 11:54:52AM +0200, Tobias Burnus wrote: >> No re-reading the orginal patch, I think it is okay. Thanks! >> http://gcc.gnu.org/ml/gcc-patches/2014-04/msg00376.html > > > Thanks for the commit. However, c

Re: [PATCH, FORTRAN] Class Name Clash

2014-04-10 Thread Tobias Burnus
Hi Bernd, On Thu, Apr 10, 2014 at 11:54:52AM +0200, Tobias Burnus wrote: > No re-reading the orginal patch, I think it is okay. Thanks! > http://gcc.gnu.org/ml/gcc-patches/2014-04/msg00376.html Thanks for the commit. However, can you also fix the ChangeLog? You used gcc/ChangeLog but the proper

Re: [PATCH, FORTRAN] Class Name Clash

2014-04-10 Thread Tobias Burnus
Hi Bernd, On Thu, Apr 10, 2014 at 11:20:47AM +0200, Bernd Edlinger wrote: > I thought about that, too. But it is likely that the pattern for array of > target class > "__class_%s_%d_%d" will eventually clash with the non-array target class > "__class_%s" which proves that I am too sleep depriev

RE: [PATCH, FORTRAN] Class Name Clash

2014-04-10 Thread Bernd Edlinger
Hi Tobias, On Thu, 10 Apr 2014 10:50:24, Tobias Burnus wrote: > > Bernd Edlinger wrote: >> this patch fixes a recently discovered name-clash in gfc_build_class_symbol. >> >> Fortunately it is quite easy to fix: just make sure that the class names of >> target >> classes end with "_t", and target