From f6727b38d515e1fd38ef47af37e8d850782c9566 Mon Sep 17 00:00:00 2001 From: Richard Barnes Date: Fri, 8 Dec 2023 18:19:46 -0800 Subject: [PATCH] Remove extra semi colon from faiss/impl/IDSelector.h Summary: `-Wextra-semi` or `-Wextra-semi-stmt` If the code compiles, this is safe to land. Reviewed By: palmje Differential Revision: D51995085 fbshipit-source-id: c0a4f134f9361143f337521759793d1cd86e145c --- faiss/impl/IDSelector.h | 6 +++--- faiss/impl/ResidualQuantizer.h | 2 +- faiss/utils/sorting.cpp | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/faiss/impl/IDSelector.h b/faiss/impl/IDSelector.h index dd56cff662..7760d2a9fa 100644 --- a/faiss/impl/IDSelector.h +++ b/faiss/impl/IDSelector.h @@ -140,7 +140,7 @@ struct IDSelectorAnd : IDSelector { : lhs(lhs), rhs(rhs) {} bool is_member(idx_t id) const final { return lhs->is_member(id) && rhs->is_member(id); - }; + } virtual ~IDSelectorAnd() {} }; @@ -153,7 +153,7 @@ struct IDSelectorOr : IDSelector { : lhs(lhs), rhs(rhs) {} bool is_member(idx_t id) const final { return lhs->is_member(id) || rhs->is_member(id); - }; + } virtual ~IDSelectorOr() {} }; @@ -166,7 +166,7 @@ struct IDSelectorXOr : IDSelector { : lhs(lhs), rhs(rhs) {} bool is_member(idx_t id) const final { return lhs->is_member(id) ^ rhs->is_member(id); - }; + } virtual ~IDSelectorXOr() {} }; diff --git a/faiss/impl/ResidualQuantizer.h b/faiss/impl/ResidualQuantizer.h index 95677d888b..004f7cabd4 100644 --- a/faiss/impl/ResidualQuantizer.h +++ b/faiss/impl/ResidualQuantizer.h @@ -155,4 +155,4 @@ struct ResidualQuantizer : AdditiveQuantizer { std::vector cent_norms; }; -}; // namespace faiss +} // namespace faiss diff --git a/faiss/utils/sorting.cpp b/faiss/utils/sorting.cpp index 67dd51bf73..76fa5e6201 100644 --- a/faiss/utils/sorting.cpp +++ b/faiss/utils/sorting.cpp @@ -123,7 +123,7 @@ void parallel_merge( } } -}; // namespace +} // namespace void fvec_argsort(size_t n, const float* vals, size_t* perm) { for (size_t i = 0; i < n; i++) {