From: Patrick Steinhardt Date: Wed, 30 Apr 2025 12:44:58 +0000 (+0200) Subject: builtin/mv: convert assert(3p) into `BUG()` X-Git-Tag: v2.50.0-rc0~59^2 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=974f0d46645604ac45b8a5ce0b90e2b2a56ca764;p=thirdparty%2Fgit.git builtin/mv: convert assert(3p) into `BUG()` The use of asserts is discouraged in our codebase because they lead to different behaviour depending on how Git is built. When being unsure enough whether a condition always holds so that one adds the assert, then the assert should probably trigger regardless of how Git is being built. Drop the call to assert(3p) in git-mv(1) and instead use `BUG()`. Signed-off-by: Patrick Steinhardt Signed-off-by: Junio C Hamano --- diff --git a/builtin/mv.c b/builtin/mv.c index c17e14cee6..4b4e1fce2e 100644 --- a/builtin/mv.c +++ b/builtin/mv.c @@ -560,7 +560,8 @@ remove_entry: continue; pos = index_name_pos(the_repository->index, src, strlen(src)); - assert(pos >= 0); + if (pos < 0) + BUG("could not find source in index: '%s'", src); if (!(mode & SPARSE) && !lstat(src, &st)) sparse_and_dirty = ie_modified(the_repository->index, the_repository->index->cache[pos],