espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-26 22:07:58 +08:00
Code Issues Packages Projects Releases Wiki Activity

[gdb/testsuite] Fix wrong return type in tests

Browse Source 
The following tests are marked untested with latest GCC due to a warning
being emitted for a mismatch between their return type and what the lack
of return statement:

* gdb.cp/breakpoint.exp
* gdb.cp/psymtab-parameter.exp
* gdb.cp/shadow.exp

This patch fix the return type to match the function definitions.

2017-11-29  Thomas Preud'homme  <thomas.preudhomme@arm.com>

gdb/testsuite/
	* gdb.cp/breakpoint.cc (bar): Set return type to void.
	* gdb.cp/psymtab-parameter.cc (func): Likewise.
	* gdb.cp/psymtab-parameter.exp: Update comment regarding prototype of
	func ().
	* gdb.cp/shadow.cc (B.func): Return 0.
This commit is contained in:
Thomas Preud'homme
2017-11-29 10:45:31 +00:00
parent ca39c2f4dd
commit f106e10e5e
5 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
gdb/testsuite/ChangeLog
View File

@ -1,3 +1,11 @@
2017-11-29 Thomas Preud'homme <thomas.preudhomme@arm.com>
* gdb.cp/breakpoint.cc (bar): Set return type to void.
* gdb.cp/psymtab-parameter.cc (func): Likewise.
* gdb.cp/psymtab-parameter.exp: Update comment regarding prototype of
func ().
* gdb.cp/shadow.cc (B.func): Return 0.
2017-11-27 Joel Brobecker <brobecker@adacore.com> 2017-11-27 Joel Brobecker <brobecker@adacore.com>
* gdb.ada/mi_catch_ex.exp (continue_to_exception): Adjust * gdb.ada/mi_catch_ex.exp (continue_to_exception): Adjust

2
gdb/testsuite/gdb.cp/breakpoint.cc
View File

@ -28,7 +28,7 @@ public:
return 1; // conditional breakpoint in method return 1; // conditional breakpoint in method
} }
int bar () void bar ()
{ {
for (int i = 0; i < 1; ++i) for (int i = 0; i < 1; ++i)
{ {

2
gdb/testsuite/gdb.cp/psymtab-parameter.cc
View File

@ -16,7 +16,7 @@
along with this program. If not, see <http://www.gnu.org/licenses/>. */ along with this program. If not, see <http://www.gnu.org/licenses/>. */
template <typename T> template <typename T>
long void
func () func ()
{ {
} }

2
gdb/testsuite/gdb.cp/psymtab-parameter.exp
View File

@ -31,6 +31,6 @@ gdb_test_no_output "set language c++"
# XFAIL than FAIL here. For example -readnow breaks it. # XFAIL than FAIL here. For example -readnow breaks it.
gdb_test_no_output "maintenance info symtabs" gdb_test_no_output "maintenance info symtabs"
# GDB has shown only the `long func<short>()' ELF symbol before, not the DWARF # GDB has shown only the `void func<short>()' ELF symbol before, not the DWARF
# symbol # symbol
gdb_test "complete p 'func<short>(" "p 'func<short>\\(\\)" gdb_test "complete p 'func<short>(" "p 'func<short>\\(\\)"

1
gdb/testsuite/gdb.cp/shadow.cc
View File

@ -36,6 +36,7 @@ public:
} }
} }
} }
return 0;
} }
}; };