MantisBT

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0003480ardourfeaturespublic2010-09-25 14:442011-02-22 23:34
Reporterlincoln 
Assigned To 
PrioritynormalSeverityfeatureReproducibilityN/A
StatusnewResolutionopen 
PlatformOSOS Version
Product Version 
Target Version3.XFixed in Version 
Summary0003480: Add ability to remove unused regions from session
DescriptionPresently Ardour will hold on to regions even if they are not being used in any playlist. As a session evolves, more and more regions are created as editing happens and some operations will slow down due to the overhead of maintaining the region list. In such cases the session will also take a long time to load or save state.

The attached patch adds a menu entry in the region list context menu that allows the user to remove regions that are not being used.

The patch also moves the creation of automatic regions away from the audio_diskstream to the editor_regions add_region method. This reduces the amount of signals being emitted when the transport is stopped after a capture.

This is a destructive operation very similar to the clean sources function. History is cleared so it is suggested that the function is only used in cases where the region list overhead is slowing down the user.
Additional InformationNot sure if the regions need to be destroyed individually. The method may be leaky right now and require some work to address this.

Having a warning dialog appear before the operation is executed may be a good idea too.
TagsNo tags attached.
Attached Filespatch file icon remove-unused-regions-function.patch [^] (19,156 bytes) 2010-09-25 14:44 [Show Content]
patch file icon remove-unused-regions-function-2.patch [^] (27,542 bytes) 2010-10-31 17:11 [Show Content]

- Relationships
related to 0003619resolvedpaul Impossible to cleanup unused sources 

-  Notes
(0009334)
lincoln (reporter)
2010-10-31 17:12

Added an updated version of this patch. This one includes a fix to update region list entries when not displaying Automatic Regions.
(0009337)
dx9s (reporter)
2010-11-03 07:56

When I go home... I'll try the patch out (after updating to 7945.. or newer) and see how it works.
(0009338)
dx9s (reporter)
2010-11-03 16:54

I checked out 3.0 -r 7945 and tried to apply the patch and got lots of rejects
(0009339)
lincoln (reporter)
2010-11-03 17:03

Did you apply remove-unused-regions-function-2.patch? It applies against a clean pull for me.
(0009341)
dx9s (reporter)
2010-11-03 18:26

DUH... wrong patch level ... -p1 (I am used to doing patch one up .. not from inside the src (root) directory ... it works applies both to head and 7945 and seems to work.. (not extensively tested) and I like the remove unused menu within the 2nd-click context of the region list!
(0010176)
lincoln (reporter)
2011-02-22 23:34

This feature has been added to Ardour 3 as part of a patch for region list speed ups. It only caters for Audio regions for now.

- Issue History
Date Modified Username Field Change
2010-09-25 14:44 lincoln New Issue
2010-09-25 14:44 lincoln File Added: remove-unused-regions-function.patch
2010-09-27 11:59 cth103 cost => 0.00
2010-09-27 11:59 cth103 Target Version => 3.X
2010-10-31 17:11 lincoln File Added: remove-unused-regions-function-2.patch
2010-10-31 17:12 lincoln Note Added: 0009334
2010-11-03 07:56 dx9s Note Added: 0009337
2010-11-03 16:54 dx9s Note Added: 0009338
2010-11-03 17:03 lincoln Note Added: 0009339
2010-11-03 18:26 dx9s Note Added: 0009341
2010-12-21 17:08 cth103 Relationship added related to 0003619
2011-02-22 23:34 lincoln Note Added: 0010176


Copyright © 2000 - 2018 MantisBT Team
Powered by Mantis Bugtracker