Shouldn't G_USER_DIRECTORY_DOWNLOAD take priority over G_USER_DIRECTORY_DESKTOP? It just so happens that both of these are set to ~/Desktop by default on Ubuntu, so will not matter to most people. However, I manually changed my XDG download directory to ~/Download, and would expect Transmission to choose this instead of my desktop when I first run it. The patch suggests that it will still choose my desktop as opposed to my download directory, unless I've misunderstood it.
I'll package this later on today. Just a quick question about this bit of the patch though charles:
#ifdef HAVE_GIO special_ dir( G_USER_ DIRECTORY_ DESKTOP ); special_ dir( G_USER_ DIRECTORY_ DOWNLOAD ); wnloadDir( ); string_ set_default ( PREF_KEY_DIR_WATCH, str ); flag_set_ default ( PREF_KEY_ DIR_WATCH_ ENABLED, FALSE );
str = NULL;
if( !str ) str = g_get_user_
if( !str ) str = g_get_user_
if( !str ) str = tr_getDefaultDo
pref_
pref_
#endif
Shouldn't G_USER_ DIRECTORY_ DOWNLOAD take priority over G_USER_ DIRECTORY_ DESKTOP? It just so happens that both of these are set to ~/Desktop by default on Ubuntu, so will not matter to most people. However, I manually changed my XDG download directory to ~/Download, and would expect Transmission to choose this instead of my desktop when I first run it. The patch suggests that it will still choose my desktop as opposed to my download directory, unless I've misunderstood it.