mirrors/Graphite
1
0
Fork 0
mirror of https://github.com/GraphiteEditor/Graphite.git synced 2025-08-04 05:18:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Remove checks from append_subpath to improve vector editing performance (#2190)

Browse source 
Remove checks from append_subpath

Co-authored-by: Keavon Chambers <keavon@keavon.com>
This commit is contained in:
Dennis Kobert 2025-03-01 12:45:49 +01:00 • committed by GitHub
parent 1510ad820c
commit 8d0c5d305d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 8 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

28
node-graph/gcore/src/vector/vector_data/attributes.rs
View file

@ -129,10 +129,7 @@ impl PointDomain {
}
pub fn push(&mut self, id: PointId, position: DVec2) {
if self.id.contains(&id) {
warn!("Duplicate point");
return;
}
debug_assert!(!self.id.contains(&id));
self.id.push(id);
self.positions.push(position);
}
@ -292,22 +289,13 @@ impl SegmentDomain {
}
pub(crate) fn push(&mut self, id: SegmentId, start: usize, end: usize, handles: bezier_rs::BezierHandles, stroke: StrokeId) {
if self.ids.contains(&id) {
return;
}
// Attempt to keep line joins?
let after = self.end_point.iter().copied().position(|other_end| other_end == start);
let before = self.start_point.iter().copied().position(|other_start| other_start == end);
let index = match (before, after) {
(_, Some(after)) => after + 1,
(Some(before), _) => before,
(None, None) => self.ids.len(),
};
self.ids.insert(index, id);
self.start_point.insert(index, start);
self.end_point.insert(index, end);
self.handles.insert(index, handles);
self.stroke.insert(index, stroke);
debug_assert!(!self.ids.contains(&id), "Tried to push an existing point to a point domain");
self.ids.push(id);
self.start_point.push(start);
self.end_point.push(end);
self.handles.push(handles);
self.stroke.push(stroke);
}
pub(crate) fn start_point_mut(&mut self) -> impl Iterator<Item = (SegmentId, &mut usize)> {