ambroisie/pathtracer
ambroisie/pathtracer
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

library: core: linearcolor: fix must_use attributes

Browse source
This commit is contained in:
Bruno BELANYI 2020-03-26 23:43:22 +01:00
parent 965fda7395
commit 88478ebecf
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
pathtracer/src/core/color.rs
View file

@ -49,6 +49,7 @@ impl LinearColor {
/// } /// }
/// ); /// );
/// ``` /// ```
#[must_use]
pub fn black() -> Self { pub fn black() -> Self {
LinearColor { LinearColor {
r: 0., r: 0.,
@ -66,11 +67,11 @@ impl LinearColor {
/// # /// #
/// let color = LinearColor::new(1.0, 0.0, 0.0); // bright red! /// let color = LinearColor::new(1.0, 0.0, 0.0); // bright red!
/// ``` /// ```
#[must_use]
pub fn new(r: f32, g: f32, b: f32) -> Self { pub fn new(r: f32, g: f32, b: f32) -> Self {
LinearColor { r, g, b } LinearColor { r, g, b }
} }
#[must_use]
/// Clamps the color's RGB components between 0.0 and 1.0. /// Clamps the color's RGB components between 0.0 and 1.0.
/// ///
/// # Examples /// # Examples
@ -81,6 +82,7 @@ impl LinearColor {
/// let color = LinearColor::new(1.5, -1.0, 0.5); /// let color = LinearColor::new(1.5, -1.0, 0.5);
/// assert_eq!(color.clamp(), LinearColor::new(1.0, 0.0, 0.5)) /// assert_eq!(color.clamp(), LinearColor::new(1.0, 0.0, 0.5))
/// ``` /// ```
#[must_use]
pub fn clamp(self) -> Self { pub fn clamp(self) -> Self {
fn clamp(v: f32) -> f32 { fn clamp(v: f32) -> f32 {
if v > 1. { if v > 1. {