commit a76fd68cf1c7042874eb6b6888ca9cc644d825e3
Author: Scott Kostyshak <skost...@lyx.org>
Date: Sat Oct 11 15:41:42 2025 +0200
Assert if adding existing branch in low level func
If I understand correctly, this should be handled higher up, as the
fix in 7d1eea03.
The debug message has been in for a few months (d8d7496e), and there
have been no reports.
---
src/BranchList.cpp | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/src/BranchList.cpp b/src/BranchList.cpp
index 2b74802822..f39fc7eec3 100644
--- a/src/BranchList.cpp
+++ b/src/BranchList.cpp
@@ -171,17 +171,14 @@ bool BranchList::add(docstring const & s)
// Is this name already in the list?
bool const already = find(name);
//
- // Not necessarily a problem, but these cases should probably
- // be handled in the GUI (e.g., 7d1eea03).
- // For now, I will put a terminal message. But eventually
- // I would like to use an assert instead:
- // LASSERT(!already, /**/);
- // \scott
if (already) {
LYXERR0("Error: attempt to add branch that already
exists: '"
<< name
<< "'. Please report this use case.");
}
+ // Not necessarily a problem, but these cases should probably
+ // be handled in the GUI (e.g., 7d1eea03). \scott
+ LASSERT(!already, /**/);
//
if (!already) {
added = true;
--
lyx-cvs mailing list
lyx-cvs@lists.lyx.org
https://lists.lyx.org/mailman/listinfo/lyx-cvs
