Date: Thu, 14 Jan 2016 07:27:43 +0000 (UTC) From: Ed Schouten <ed@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r293879 - head/lib/libc/stdlib Message-ID: <201601140727.u0E7Rhll089486@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: ed Date: Thu Jan 14 07:27:42 2016 New Revision: 293879 URL: https://svnweb.freebsd.org/changeset/base/293879 Log: Remove an unneeded assignment of the return value. tdelete() is supposed to return the address of the parent node that has been deleted. We already keep track of this node in the loop between lines 94-107. The GO_LEFT()/GO_RIGHT() macros are used later on as well, so we must make sure not to change it to something else. Modified: head/lib/libc/stdlib/tdelete.c Modified: head/lib/libc/stdlib/tdelete.c ============================================================================== --- head/lib/libc/stdlib/tdelete.c Thu Jan 14 07:27:14 2016 (r293878) +++ head/lib/libc/stdlib/tdelete.c Thu Jan 14 07:27:42 2016 (r293879) @@ -62,7 +62,6 @@ __FBSDID("$FreeBSD$"); base = leaf; \ path_init(&path); \ } \ - result = &(*leaf)->key; \ path_taking_right(&path); \ leaf = &(*leaf)->rlink; \ } while (0)
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201601140727.u0E7Rhll089486>