public inbox for gdb-testers@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Import strchrnul from gnulib and use it
@ 2016-01-28 15:47 sergiodj+buildbot
  2016-01-28 15:47 ` Failures on Fedora-i686, branch master sergiodj+buildbot
                   ` (12 more replies)
  0 siblings, 13 replies; 14+ messages in thread
From: sergiodj+buildbot @ 2016-01-28 15:47 UTC (permalink / raw)
  To: gdb-testers

*** TEST RESULTS FOR COMMIT 8424cc978c8c76aca7945d50408762de65646095 ***

Author: Simon Marchi <simon.marchi@ericsson.com>
Branch: master
Commit: 8424cc978c8c76aca7945d50408762de65646095

Import strchrnul from gnulib and use it

For a forthcoming patch, I need a "skip_to_colon" function.  I noticed
there are two skip_to_semicolon (one in gdb and one in gdbserver). I
thought we could put it in common/, and generalize it for any character.
It turns out that the strchrnul function does exactly that.  I imported
the corresponding module from gnulib, for those systems that do not have
it.

There are probably more places where this function can be used instead
of doing the work by hand (I am looking at
remote-utils.c::look_up_one_symbol).

gdb/ChangeLog:

	* remote.c (skip_to_semicolon): Remove.
	(remote_parse_stop_reply): Use strchrnul instead of
	skip_to_semicolon.
	* gnulib/update-gnulib.sh (IMPORTED_GNULIB_MODULES): Add
	strchrnul.
	* gnulib/aclocal.m4: Regenerate.
	* gnulib/config.in: Regenerate.
	* gnulib/configure: Regenerate.
	* gnulib/import/Makefile.am: Regenerate.
	* gnulib/import/Makefile.in: Regenerate.
	* gnulib/import/m4/gnulib-cache.m4: Regenerate.
	* gnulib/import/m4/gnulib-comp.m4: Regenerate.
	* gnulib/import/m4/rawmemchr.m4: New file.
	* gnulib/import/m4/strchrnul.m4: New file.
	* gnulib/import/rawmemchr.c: New file.
	* gnulib/import/rawmemchr.valgrind: New file.
	* gnulib/import/strchrnul.c: New file.
	* gnulib/import/strchrnul.valgrind: New file.

gdb/gdbserver/ChangeLog:

	* server.c (skip_to_semicolon):  Remove.
	(process_point_options): Use strchrnul instead of
	skip_to_semicolon.


^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2016-01-28 19:17 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-28 15:47 [binutils-gdb] Import strchrnul from gnulib and use it sergiodj+buildbot
2016-01-28 15:47 ` Failures on Fedora-i686, branch master sergiodj+buildbot
2016-01-28 15:49 ` Failures on Fedora-x86_64-native-extended-gdbserver-m32, " sergiodj+buildbot
2016-01-28 15:50 ` Failures on Fedora-x86_64-m32, " sergiodj+buildbot
2016-01-28 16:02 ` Failures on Fedora-s390x-m64, " sergiodj+buildbot
2016-01-28 16:02 ` Failures on Fedora-x86_64-native-extended-gdbserver-m64, " sergiodj+buildbot
2016-01-28 16:15 ` Failures on AIX-POWER7-plain, " sergiodj+buildbot
2016-01-28 16:26 ` Failures on Debian-s390x-native-gdbserver-m64, " sergiodj+buildbot
2016-01-28 16:50 ` Failures on Debian-i686, " sergiodj+buildbot
2016-01-28 17:10 ` Failures on Fedora-ppc64be-cc-with-index, " sergiodj+buildbot
2016-01-28 17:13 ` Failures on Debian-s390x-native-extended-gdbserver-m64, " sergiodj+buildbot
2016-01-28 17:30 ` Failures on Fedora-ppc64be-m64, " sergiodj+buildbot
2016-01-28 17:47 ` Failures on Fedora-ppc64be-native-gdbserver-m64, " sergiodj+buildbot
2016-01-28 19:17 ` Failures on Fedora-ppc64le-m64, " sergiodj+buildbot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).