From 69421c158424dd75fba9c3804ebf6d0d89262aea Mon Sep 17 00:00:00 2001
From: Christian Jaeger <ch@christianjaeger.ch>
Date: Thu, 18 Apr 2024 16:12:56 +0200
Subject: [PATCH] fix: nodiscard (silence warnings)

Refs: #390
---
 include/silo/query_engine/actions/fasta.h   | 8 +++++---
 include/silo/query_engine/operator_result.h | 2 +-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/include/silo/query_engine/actions/fasta.h b/include/silo/query_engine/actions/fasta.h
index e8feaa915..7d93c6eea 100644
--- a/include/silo/query_engine/actions/fasta.h
+++ b/include/silo/query_engine/actions/fasta.h
@@ -24,10 +24,12 @@ class Fasta : public Action {
 
    std::vector<std::string> sequence_names;
 
-   [[nodiscard]] void validateOrderByFields(const Database& database) const override;
+   void validateOrderByFields(const Database& database) const override;
 
-   QueryResult execute(const Database& database, std::vector<OperatorResult> bitmap_filter)
-      const override;
+   [[nodiscard]] QueryResult execute(
+      const Database& database,
+      std::vector<OperatorResult> bitmap_filter
+   ) const override;
 
    void addSequencesToResultsForPartition(
       QueryResult& results,
diff --git a/include/silo/query_engine/operator_result.h b/include/silo/query_engine/operator_result.h
index 4ac5b916b..ce13a3f6b 100644
--- a/include/silo/query_engine/operator_result.h
+++ b/include/silo/query_engine/operator_result.h
@@ -28,7 +28,7 @@ class OperatorResult {
    roaring::Roaring* operator->();
    const roaring::Roaring* operator->() const;
 
-   bool isMutable() const;
+   [[nodiscard]] bool isMutable() const;
 };
 
 }  // namespace silo::query_engine