| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
| |
Move the portion of screen_get_mouse_event() which handles list_window
clicks to list_window.c. The code contained a NULL pointer
dereference, which is now fixed.
|
| |
|
|
| |
Don't call filelist_get() with an invalid index.
|
| |
|
|
|
| |
Since we do a lot of indexed accesses to the filelist, a linked list
doesn't seem appropriate. Use GPtrArray instead.
|
| |
|
|
|
| |
browser_get_selected() checks if there is a file list and a valid
selection, and returns the selected entry.
|
| |
|
|
| |
Avoid direct accesses to the filelist struct, provide an API for that.
|
| |
|
|
|
| |
The separate filelist library does not depend on mpdclient, so the
prefix is superfluous.
|
| |
|
|
|
| |
A song should not be twice in the browser. Simplify set_highlight()
by finding the entry with mpdclient_filelist_find_song().
|
| |
|
|
|
| |
The caller must not pass NULL to set_highlight(). Replaced check with
assertion.
|
| |
|
|
|
| |
It is guaranteed that the "song" attribute is not NULL, remove the
check.
|
| |
|
|
|
|
| |
browser_select_entry() checked the entry's "highlight" flag twice: the
first time, it was toggled, and the second time, action was performed
based on the flag. Merge the two checks.
|
| |
|
|
|
| |
The function browser_select_entry() returns early when the entity type
is not "SONG". No need to double check that.
|
| |
|
|
|
|
| |
Moved code from browser_handle_select(). Call this new function also
from browser_handle_select_all(). This fixes a bug in select_all():
it actually did not select all, but stopped after the first song.
|
| | |
|
| |
|
|
|
| |
Merge code from all 3 browser screens into one generic browser
function.
|
|
|
Isolate the generic browser code from the directory browser.
|
