Code review comment for lp:~cjcurran/indicator-sound/remote-art-handling

Revision history for this message
Ted Gould (ted) wrote :

 * The temporary file needs to be built with mkstemp to create a random directory for the artwork.
 * It should be in the XDG temp directory instead of hardcoded to /tmp
 * You can use g_file_is_native() to determine whether it's local or not, just incase they're using something other than http://

review: Needs Fixing

« Back to merge proposal