Show song length in the library #2

Closed
opened 3 years ago by p · 0 comments
p commented 3 years ago
Owner

Similar to the "Current" tab. For the "Current" tab we use time but it is marked as deprecated, so prefer and parse the duration field from the lsinfo response, which may contain a fractional part.

Consider replacing the '?' for unknown times with a dash or removing it altogether.

The items array is not sufficient to store this, we need to abandon struct strv. Consider using ARRAY macros—library_tab_add() is only used to append items and the array is only reset in one place.

Similar to the "Current" tab. For the "Current" tab we use `time` but it is marked as deprecated, so prefer and parse the `duration` field from the `lsinfo` response, which may contain a fractional part. Consider replacing the '?' for unknown times with a dash or removing it altogether. The `items` array is not sufficient to store this, we need to abandon `struct strv`. Consider using `ARRAY` macros—`library_tab_add()` is only used to append items and the array is only reset in one place.
p added this to the v1.0.0 milestone 3 years ago
p self-assigned this 3 years ago
p added the
easy
label 3 years ago
p closed this issue 3 years ago
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: p/nncmpp#2
Loading…
There is no content yet.