Apply review comments

This commit is contained in:
Louis Dureuil 2024-07-15 12:43:29 +02:00
parent c35904d6e8
commit 20094eba06
No known key found for this signature in database
2 changed files with 3 additions and 5 deletions

View File

@ -27,7 +27,7 @@ use super::{
use crate::error::MeilisearchHttpError; use crate::error::MeilisearchHttpError;
use crate::routes::indexes::search::search_kind; use crate::routes::indexes::search::search_kind;
pub const DEFAULT_FEDERATED_WEIGHT: fn() -> f64 = || 1.0; pub const DEFAULT_FEDERATED_WEIGHT: f64 = 1.0;
#[derive(Debug, Default, Clone, Copy, PartialEq, deserr::Deserr)] #[derive(Debug, Default, Clone, Copy, PartialEq, deserr::Deserr)]
#[deserr(error = DeserrJsonError, rename_all = camelCase, deny_unknown_fields)] #[deserr(error = DeserrJsonError, rename_all = camelCase, deny_unknown_fields)]
@ -42,7 +42,7 @@ pub struct Weight(f64);
impl Default for Weight { impl Default for Weight {
fn default() -> Self { fn default() -> Self {
Weight(DEFAULT_FEDERATED_WEIGHT()) Weight(DEFAULT_FEDERATED_WEIGHT)
} }
} }
@ -50,8 +50,6 @@ impl std::convert::TryFrom<f64> for Weight {
type Error = InvalidMultiSearchWeight; type Error = InvalidMultiSearchWeight;
fn try_from(f: f64) -> Result<Self, Self::Error> { fn try_from(f: f64) -> Result<Self, Self::Error> {
// the suggested "fix" is: `!(0.0..=1.0).contains(&f)`` which is allegedly less readable
#[allow(clippy::manual_range_contains)]
if f < 0.0 { if f < 0.0 {
Err(InvalidMultiSearchWeight) Err(InvalidMultiSearchWeight)
} else { } else {

View File

@ -125,7 +125,7 @@ impl CompatibilityError {
| (RankingRuleKind::Relevancy, RankingRuleKind::DescendingGeoSort) | (RankingRuleKind::Relevancy, RankingRuleKind::DescendingGeoSort)
| (RankingRuleKind::AscendingGeoSort, RankingRuleKind::Relevancy) | (RankingRuleKind::AscendingGeoSort, RankingRuleKind::Relevancy)
| (RankingRuleKind::DescendingGeoSort, RankingRuleKind::Relevancy) => { | (RankingRuleKind::DescendingGeoSort, RankingRuleKind::Relevancy) => {
"cannot compare a relevancy rule with a sort rule" "cannot compare a relevancy rule with a geosort rule"
} }
(RankingRuleKind::AscendingSort, RankingRuleKind::DescendingSort) (RankingRuleKind::AscendingSort, RankingRuleKind::DescendingSort)