Change start/stop times

Bug #1310827 reported by Paul Loschak
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Exaile
Fix Released
Wishlist
Unassigned

Bug Description

This is a wishlist item. It would be helpful if I could go into the properties (similar to in itunes) and change the time in a song that I want the song to start or stop at. For example, some songs have long obnoxious intros, and I would prefer if the playback simply started at 0:30.00 instead of 0:00.00. Similar request for the endings of songs.

Related branches

Changed in exaile:
status: New → In Progress
importance: Undecided → Wishlist
milestone: none → 3.4.0
assignee: nobody → Dustin Spicuzza (dustin-virtualroadside)
Revision history for this message
Dustin Spicuzza (dustin-virtualroadside) wrote :

Implemented in r4492. Try it out and let me know if it works.

Currently, the editing for the start/stop offsets are just integer seconds in the properties that aren't validated. I'll put something better in at a later date.

Changed in exaile:
assignee: Dustin Spicuzza (dustin-virtualroadside) → nobody
status: In Progress → Fix Committed
Revision history for this message
Rocco Aliberti (eri.trabiccolo) wrote :

Stop doesn't really work. It stops almost one second after the start.
Debug:
start = 15
stop = 30

print of some values in ._base._monitor_for_stop:
get_position() => 15673446712
stop_offset*1000 => 30000

You should use get_time() instead of get_position() and then just stop_offset instead of stop_offset*1000.

Patch against r4492 attached

Revision history for this message
Dustin Spicuzza (dustin-virtualroadside) wrote :

Ah, looks like the documentation for get_position was wrong. It just so happened that the file I was testing with was with a one second stop condition… oops.

I've updated the documentation and the code, available as of r4493.

Revision history for this message
Rocco Aliberti (eri.trabiccolo) wrote :

Eheh sorry, didn't read the documentation. Once I've found that nanoseconds issue I went into get_position and seen right below that the get_time method which already did what "we" wanted, sloppy me :D

Changed in exaile:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.