arkq / cmusfm

Last.fm standalone scrobbler for the cmus music player
GNU General Public License v3.0
232 stars 5 forks source link

Add show album cover in image CUE file #46

Closed andgera closed 2 years ago

andgera commented 2 years ago

Doesn't show album art images in popup notification when listening to disc image using

I read a book on the C language, and how I could implement the ability to display album covers in cue files. I'm not sure if the following design is optimal in terms of performance, but it works.


for(j = 0; j <= n; j++) {
    memmove(&location_t[j], &location_t[j + 6], 6);
}
andgera commented 2 years ago

Add show album cover in image CUE file (new)

delete:

for(j = 0; j <= n; j++) {
    memmove(&location_t[j], &location_t[j + 6], 6);
}

replaced:

    strcpy(location_t, &location[6]);
arkq commented 2 years ago

Why have you closed this PR. It is preferable to have one PR per one feature. Currently PR #47 contains changes not related to Debian.