This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] testsuite/gdb.dwarf2: Enable dw2-error.exp tests for arm targets


>>>>> "Abid" == Abid, Hafiz <Hafiz_Abid@mentor.com> writes:

>> +if  {[gdb_compile [list ${binfile}1.o ${binfile}2.o ] \
>> +      "${binfile}" executable {}] != ""} {
>> +    return -1
>> +}
>> +
>> +clean_restart ${testfile}
>> +
>> gdb_exit
>> gdb_start
>> gdb_reinitialize_dir $srcdir/$subdir

Abid> This gdb_exit, gdb_start, gdb_reinitialize_dir sequence right after
Abid> clean_restart tells me that perhaps one of them is redundant.

Good catch.
Given the rest of the test I think the clean_restart is not needed.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]