rei-mirrors/Fornjot
1
0
Fork 0
mirror of https://github.com/hannobraun/Fornjot synced 2025-07-17 23:46:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove redundant struct field

Browse Source
This commit is contained in:
Hanno Braun 2024-12-16 21:15:58 +01:00
parent 784031af85
commit 3b0d56fb52
1 changed files with 1 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
experiments/2024-12-09/src/geometry/shape.rs
View File

@ -1,4 +1,4 @@
use std::{fmt, marker::PhantomData};
use std::fmt;
use tuples::CombinRight;
@ -86,16 +86,6 @@ pub struct ShapeExtender<'r, NewOps, T> {
store: &'r mut Store<T>,
sequence: &'r mut Vec<OperationInSequence>,
new_ops: NewOps,
// In principle, this isn't necessary, as `NewOps` and `T` are partially
// redundant. The way this struct is set up right now, it's only possible to
// add a single type of operation. So in theory, `NewOps` could be replaced
// with `[T; N]`, and we'd just need to track the value of `N` in addition
// to `T`.
//
// Unfortunately, the state of const expressions in Rust (at the time of
// writing) doesn't allow for this.
_t: PhantomData<T>,
}
impl<'r, T> ShapeExtender<'r, (), T> {
@ -107,7 +97,6 @@ impl<'r, T> ShapeExtender<'r, (), T> {
store,
sequence,
new_ops: (),
_t: PhantomData,
}
}
}
@ -129,7 +118,6 @@ impl<'r, NewOps, T> ShapeExtender<'r, NewOps, T> {
store: self.store,
sequence: self.sequence,
new_ops: self.new_ops.push_right(vertex),
_t: PhantomData,
}
}