Skip to content

Conversation

@YichiZhang0613
Copy link

The assertion's predicate is amount >= 0.0 while message requires amount > 0.0, the message should be changed into "The loosening margin must be non-negative." based on semantic. The same issue elsewhere should also be fixed.

/// # Arguments
    ///
    /// * `amount` - The amount to increase the radius (must be non-negative)
    ///
    /// # Panics
    ///
    /// Panics if `amount` is negative.
#[inline]
    fn loosened(&self, amount: Real) -> BoundingSphere {
        assert!(amount >= 0.0, "The loosening margin must be positive.");
        BoundingSphere::new(self.center, self.radius + amount)
    }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant