This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] Testsuite: Fortran: 6.8 regression + gfortran updates
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- Cc: gdb-patches at sourceware dot org, Markus Deuling <deuling at de dot ibm dot com>
- Date: Wed, 7 Jan 2009 09:14:04 +0400
- Subject: Re: [patch] Testsuite: Fortran: 6.8 regression + gfortran updates
- References: <20081208181523.GA23831@host0.dyn.jankratochvil.net> <20081229034845.GF4216@adacore.com> <20090106140516.GA16916@host0.dyn.jankratochvil.net>
> gdb/
> 2009-01-06 Jan Kratochvil <jan.kratochvil@redhat.com>
>
> * f-typeprint.c (f_type_print_base <TYPE_CODE_STRUCT>): Fix output
> spacing, a regression from 2008-04-22.
I am not sure about this part - the use of the "indented" printf seems
to be pretty deliberate, and we find this use elsewhere for the other
kinds of types. Looks like Fortran developers like indented type names?
I think it'd be interesting to get Markus' opinion.
This, in turn, conditions the changes to the testsuite, but it should
only be minimal.
> gdb/testsuite/
> 2009-01-06 Jan Kratochvil <jan.kratochvil@redhat.com>
>
> * gdb.fortran/derived-type.exp: New variables int4 and real4. Match
> now the output exactly, using less wildcards. Expect also the output
> of gfortran-4.3. Update for the f-valprint.c modification from
> 2008-04-22.
> * gdb.fortran/subarray.exp: Expect also the output of gfortran-4.3.
> Remove gdb_test test names where matching the command.
Overall, the changes look OK to me. The only adjustment that might be
needed is if we keep (f_type_print_base <TYPE_CODE_STRUCT>) as is.
--
Joel