diff options
author | Yao Qi <yao.qi@linaro.org> | 2015-11-27 14:21:47 +0000 |
---|---|---|
committer | Yao Qi <yao.qi@linaro.org> | 2015-11-27 14:21:47 +0000 |
commit | 88e8ec1b3e372dc1ea1cd0be44c59ca7760e9008 (patch) | |
tree | 0904902b1ae741f629851be3a17b4b5d17a1db4e /gdb/breakpoint.h | |
parent | bfde72c275bff9d5be21cf51ba790cf38ecd0d59 (diff) | |
download | gdb-88e8ec1b3e372dc1ea1cd0be44c59ca7760e9008.zip gdb-88e8ec1b3e372dc1ea1cd0be44c59ca7760e9008.tar.gz gdb-88e8ec1b3e372dc1ea1cd0be44c59ca7760e9008.tar.bz2 |
Allow multiple occurrences of the frames-invalid annotation in gdb.cp/annota2.exp
Hi,
I see one fail on aarch64-linux testing,
FAIL: gdb.cp/annota2.exp: watch triggered on a.x (timeout)
because GDB prints two frames-invalid annotation but the test expects
only one.
next^M
^M
^Z^Zpost-prompt^M
^M
^Z^Zstarting^M
^M
^Z^Zframes-invalid^M
^M
^Z^Zframes-invalid^M
^M
Note I also see the fail on Debian-s390x-m64 too.
https://sourceware.org/ml/gdb-testers/2015-q4/msg07291.html
The test shouldn't only expect one frames-invalid annotation, because
there can be multiple times of stop/resume before the user visible
stop. Ulrich did something similar before
https://www.sourceware.org/ml/gdb-patches/2009-06/msg00118.html
This patch only changes ${frames_invalid} to \(${frames_invalid}\)*
in the regexp pattern.
The patch below fixes the fail on aarch64-linux.
gdb/testsuite:
2015-11-27 Yao Qi <yao.qi@linaro.org>
* gdb.cp/annota2.exp: Allow multiple occurrences of the
frames-invalid annotation.
Diffstat (limited to 'gdb/breakpoint.h')
0 files changed, 0 insertions, 0 deletions