openshot/libopenshot
0
0
Fork 0
mirror of https://github.com/OpenShot/libopenshot.git synced 2026-03-02 08:53:52 -08:00
Code Issues Packages Projects Releases Wiki Activity

Keyframe::RemovePoint() only set needs_update if a point was removed

Browse Source
This commit is contained in:
Daniel Jour
2019-11-19 20:59:47 +01:00
parent 6226e9d8e9
commit 5f7766e49e
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/KeyFrame.cpp
View File

@@ -550,9 +550,6 @@ int64_t Keyframe::GetCount() {
// Remove a point by matching a coordinate
void Keyframe::RemovePoint(Point p) {
// mark as dirty
needs_update = true;
// loop through points, and find a matching coordinate
for (int64_t x = 0; x < Points.size(); x++) {
// Get each point
@@ -562,6 +559,8 @@ void Keyframe::RemovePoint(Point p) {
if (p.co.X == existing_point.co.X && p.co.Y == existing_point.co.Y) {
// Remove the matching point, and break out of loop
Points.erase(Points.begin() + x);
// mark as dirty
needs_update = true;
return;
}
}
@@ -572,12 +571,11 @@ void Keyframe::RemovePoint(Point p) {
// Remove a point by index
void Keyframe::RemovePoint(int64_t index) {
// mark as dirty
needs_update = true;
// Is index a valid point?
if (index >= 0 && index < Points.size())
{
// mark as dirty
needs_update = true;
// Remove a specific point by index
Points.erase(Points.begin() + index);
}