X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FBiblioInfo.cpp;h=44321bcad3f90a62a7e25489757a7da6603a6f75;hb=f56ee1875c9bf2665ab7377fdc3ff3ec80d9268f;hp=70127798522effae211a946b48d67658c7600892;hpb=9c55af4a223ce4db29d643251109e245665344bd;p=lyx.git

diff --git a/src/BiblioInfo.cpp b/src/BiblioInfo.cpp
index 7012779852..44321bcad3 100644
--- a/src/BiblioInfo.cpp
+++ b/src/BiblioInfo.cpp
@@ -756,8 +756,7 @@ docstring BibTeXInfo::getValueForKey(string const & oldkey, Buffer const & buf,
 		ret = html::cleanAttr(ret);
 
 	// make sure it is not too big
-	if (ret.size() > maxsize)
-		ret = ret.substr(0, maxsize - 3) + from_ascii("...");
+	support::truncateWithEllipsis(ret, maxsize);
 	return ret;
 }
 
@@ -928,14 +927,9 @@ docstring const BiblioInfo::getLabel(vector<docstring> keys,
 			before, after, dialog, key + 1 != ken);
 	}
 
-	if (ret.size() > max_size) {
-		ret.resize(max_size - 3);
-		ret += "...";
-	} else if (too_many_keys) {
-		if (ret.size() > max_size - 3)
-			ret.resize(max_size - 3);
-		ret += "...";
-	}
+	if (too_many_keys)
+		ret.push_back(0x2026);//HORIZONTAL ELLIPSIS
+	support::truncateWithEllipsis(ret, max_size);
 	return ret;
 }
 
@@ -1005,9 +999,9 @@ void BiblioInfo::collectCitedEntries(Buffer const & buf)
 	// FIXME We may want to collect these differently, in the first case,
 	// so that we might have them in order of appearance.
 	set<docstring> citekeys;
-	Toc const & toc = buf.tocBackend().toc("citation");
-	Toc::const_iterator it = toc.begin();
-	Toc::const_iterator const en = toc.end();
+	shared_ptr<Toc const> toc = buf.tocBackend().toc("citation");
+	Toc::const_iterator it = toc->begin();
+	Toc::const_iterator const en = toc->end();
 	for (; it != en; ++it) {
 		if (it->str().empty())
 			continue;
@@ -1066,6 +1060,11 @@ void BiblioInfo::makeCitationLabels(Buffer const & buf)
 			docstring const num = convert<docstring>(++keynumber);
 			entry.setCiteNumber(num);
 		} else {
+			// coverity complains about our derefercing the iterator last,
+			// which was not initialized above. but it does get initialized
+			// after the first time through the loop, which is the point of
+			// the first test.
+			// coverity[FORWARD_NULL]
 			if (it != cited_entries_.begin()
 			    && entry.getAbbreviatedAuthor() == last->second.getAbbreviatedAuthor()
 			    // we access the year via getYear() so as to get it from the xref,