[cppcheck] convert several functions to use std::all_of

std::all_of becomes constexpr in C++20. I'm not sure it results in better
performance.

Found with useStlAlgorithm

Signed-off-by: Rosen Penev <rosenp@gmail.com>

src/command/FileCommands.cxx

@@ -127,12 +127,7 @@ gcc_pure
 static bool
 IsValidValue(const StringView s) noexcept
 {
-	for (const char ch : s) {
+	return std::none_of(s.begin(), s.end(), [](const auto &ch) { return (unsigned char)ch < 0x20; });
-		if ((unsigned char)ch < 0x20)
-			return false;
-	}
-
-	return true;
 }
 
 class PrintCommentHandler final : public NullTagHandler {

src/input/InputPlugin.cxx

@@ -33,11 +33,8 @@ InputPlugin::SupportsUri(const char *uri) const noexcept
 			if (StringStartsWithIgnoreCase(uri, *i))
 				return true;
 	} else {
-		for (const auto& schema : protocols()) {
+		return std::any_of(protocols().begin(), protocols().end(), [uri](const auto &schema)
-			if (StringStartsWithIgnoreCase(uri, schema.c_str())){
+			{ return StringStartsWithIgnoreCase(uri, schema.c_str()); } );
-				return true;
-			}
-		}
 	}
 	return false;
 }

src/lib/nfs/Cancellable.hxx

@@ -113,11 +113,7 @@ public:
 #ifndef NDEBUG
 	gcc_pure
 	bool IsEmpty() const noexcept {
-		for (const auto &c : list)
+		return std::all_of(list.begin(), list.end(), [](const auto &c) { return c.IsCancelled(); });
-			if (!c.IsCancelled())
-				return false;
-
-		return true;
 	}
 #endif
 

src/output/MultipleOutputs.cxx

@@ -258,11 +258,8 @@ MultipleOutputs::Open(const AudioFormat audio_format)
 bool
 MultipleOutputs::IsChunkConsumed(const MusicChunk *chunk) const noexcept
 {
-	for (const auto &ao : outputs)
+	return std::all_of(outputs.begin(), outputs.end(), [chunk](const auto &ao) {
-		if (!ao->LockIsChunkConsumed(*chunk))
+		return ao->LockIsChunkConsumed(*chunk); });
-			return false;
-
-	return true;
 }
 
 unsigned

src/output/MultipleOutputs.hxx

@@ -28,6 +28,7 @@
 #include "Chrono.hxx"
 #include "util/Compiler.h"
 
+#include <algorithm>
 #include <cassert>
 #include <memory>
 #include <vector>
@@ -106,11 +107,7 @@ public:
 	 */
 	gcc_pure
 	bool IsDummy() const noexcept {
-		for (const auto &i : outputs)
+		return std::all_of(outputs.begin(), outputs.end(), [](const auto &i) { return i->IsDummy(); });
-			if (!i->IsDummy())
-				return false;
-
-		return true;
 	}
 
 	/**

src/song/AndSongFilter.cxx

@@ -19,6 +19,8 @@
 
 #include "AndSongFilter.hxx"
 
+#include <algorithm>
+
 ISongFilterPtr
 AndSongFilter::Clone() const noexcept
 {
@@ -54,9 +56,5 @@ AndSongFilter::ToExpression() const noexcept
 bool
 AndSongFilter::Match(const LightSong &song) const noexcept
 {
-	for (const auto &i : items)
+	return std::all_of(items.begin(), items.end(), [&song](const auto &i) { return i->Match(song); });
-		if (!i->Match(song))
-			return false;
-
-	return true;
 }

src/tag/Builder.cxx

@@ -154,11 +154,7 @@ TagBuilder::CommitNew() noexcept
 bool
 TagBuilder::HasType(TagType type) const noexcept
 {
-	for (auto i : items)
+	return std::any_of(items.begin(), items.end(), [type](const auto &i) { return i->type == type; });
-		if (i->type == type)
-			return true;
-
-	return false;
 }
 
 void