[vlc-devel] [PATCH] Load icons on runtime

Ilkka Ollakka ilkka.ollakka+vlc at oamk.fi
Tue Nov 24 09:59:21 CET 2009


Hi,

I don't think this one is sufficient enought to users having ability to  
change icons, atleast in linux that path is usually under prefix, and user  
may not have rights to put stuff there.

And also, what does QIcon do if you give it NULL? I didn't find answer to  
that on quick check from api.

So it could be better to do checking like this:
check config_GetUserDir(VLC_DATA_DIR)+DIR_SEP+icon
if not found
  check config_GetDataDir+DIR_SEP+icon
  if not found
   somekinda fallback


-- 
Ilkka Ollakka



More information about the vlc-devel mailing list