Test results for commit 800eb1cebe736f6867d13e5df40a2c463a4b23ad on branch master

sergiodj@redhat.com sergiodj@redhat.com
Wed Feb 11 14:56:00 GMT 2015


*** TEST RESULTS FOR COMMIT 800eb1cebe736f6867d13e5df40a2c463a4b23ad ***

Author: Jan Kratochvil <jan.kratochvil@redhat.com>
Branch: master
Commit: 800eb1cebe736f6867d13e5df40a2c463a4b23ad

framefilter quit: Code cleanup: Avoid gotos
goto error patters are sometimes AFAIK used in C for the cases like:
	int retval=-1;
	if (!(a=malloc())) goto error;
	if (!(b=malloc())) goto error_a;
	if (!(c=malloc())) goto error_b;
	retval=0;
	error_c: free(c);
	error_b: free(b);
	error_a: free(a);
	error: return retval;

But here there is single error label with one do_cleanups() which I do not find
it worth the goto complication.  Without goto one can then furher merge code in
the exit paths in the next patches and ... after all it is all the same, just
without a goto.

gdb/ChangeLog
2015-02-11  Jan Kratochvil  <jan.kratochvil@redhat.com>

	* python/py-framefilter.c (py_print_frame): Substitute goto error.
	Remove the error label.



More information about the Gdb-testers mailing list