fix small clang warnings

This commit is contained in:
Jean-Francois Dockes 2021-04-21 03:51:46 -07:00
parent 3ac4349c64
commit 900f718074
3 changed files with 5 additions and 5 deletions

View File

@ -128,7 +128,7 @@
#define PACKAGE_NAME "Recoll" #define PACKAGE_NAME "Recoll"
/* Define to the full name and version of this package. */ /* Define to the full name and version of this package. */
#define PACKAGE_STRING "Recoll 1.28.6pre4" #define PACKAGE_STRING "Recoll 1.30.2"
/* Define to the one symbol short name of this package. */ /* Define to the one symbol short name of this package. */
#define PACKAGE_TARNAME "recoll" #define PACKAGE_TARNAME "recoll"
@ -137,7 +137,7 @@
#define PACKAGE_URL "" #define PACKAGE_URL ""
/* Define to the version of this package. */ /* Define to the version of this package. */
#define PACKAGE_VERSION "1.28.6pre4" #define PACKAGE_VERSION "1.30.2"
/* putenv parameter is const */ /* putenv parameter is const */
/* #undef PUTENV_ARG_CONST */ /* #undef PUTENV_ARG_CONST */

View File

@ -269,7 +269,7 @@ static bool matchesSkipped(
string topdir; string topdir;
for (;;) { // Used to test not root here, but root may be in topdirs ! for (;;) { // Used to test not root here, but root may be in topdirs !
for (const auto tdlent : tdl) { for (const auto& tdlent : tdl) {
// the topdirs members are already canonized. // the topdirs members are already canonized.
LOGDEB1("matchesSkipped: comparing ancestor [" << mpath << LOGDEB1("matchesSkipped: comparing ancestor [" << mpath <<
"] to topdir [" << tdlent << "]\n"); "] to topdir [" << tdlent << "]\n");

View File

@ -66,7 +66,7 @@ string make_abstract(Rcl::Doc& doc, Rcl::Query& query, bool asSnippets,
std::vector<Rcl::Snippet> snippets; std::vector<Rcl::Snippet> snippets;
std::ostringstream str; std::ostringstream str;
if (query.makeDocAbstract(doc, snippets, snipcount, -1, true)) { if (query.makeDocAbstract(doc, snippets, snipcount, -1, true)) {
for (const auto snippet : snippets) { for (const auto& snippet : snippets) {
str << snippet.page << " : " << snippet.snippet << endl; str << snippet.page << " : " << snippet.snippet << endl;
} }
} }
@ -467,7 +467,7 @@ endopts:
<< doc.fbytes << "\tbytes" << "\t" << doc.fbytes << "\tbytes" << "\t"
<< endl; << endl;
if (op_flags & OPT_m) { if (op_flags & OPT_m) {
for (const auto ent : doc.meta) { for (const auto& ent : doc.meta) {
cout << ent.first << " = " << ent.second << endl; cout << ent.first << " = " << ent.second << endl;
} }
} }