Enlightenment CVS wrote:
> Enlightenment CVS committal
> 
> Author  : devilhorns
> Project : e17
> Module  : apps/e
> 
> Dir     : e17/apps/e/src/bin
> 
> 
> Modified Files:
>       e_fm_main.c 
> 
> 
> Log Message:
> Use mkpath, thanks Seb :)

You should still check the return value from mkpath. If mkpath returns 
0, it couldn't create the dir for some reason.

> 
> ===================================================================
> RCS file: /cvs/e/e17/apps/e/src/bin/e_fm_main.c,v
> retrieving revision 1.31
> retrieving revision 1.32
> diff -u -3 -r1.31 -r1.32
> --- e_fm_main.c       10 Oct 2007 17:54:04 -0000      1.31
> +++ e_fm_main.c       10 Oct 2007 19:04:55 -0000      1.32
> @@ -1634,16 +1634,13 @@
>     fop = (E_Fop *)data;
>     if (!fop) return 0;
>  
> -   /* Check that 'home trash' exists, create if not */
> +   /* Check that 'home trash' and subsequesnt dirs exists, create if not */
>     snprintf(buf, sizeof(buf), "%s/Trash", efreet_data_home_get());
>     trash_dir = evas_stringshare_add(buf);
> -   if (!ecore_file_exists(trash_dir)) ecore_file_mkdir(trash_dir);
> -
> -   /* Create required 'info' and 'files' subdirs if needed */
>     snprintf(buf, sizeof(buf), "%s/files", trash_dir);
> -   if (!ecore_file_exists(buf)) ecore_file_mkdir(buf);
> +   ecore_file_mkpath(buf);
>     snprintf(buf, sizeof(buf), "%s/info", trash_dir);
> -   if (!ecore_file_exists(buf)) ecore_file_mkdir(buf);
> +   ecore_file_mkpath(buf);
>  
>     filename = evas_stringshare_add(strrchr(fop->src, '/'));
>     escname = ecore_file_escape_name(filename);

-- 
Sebastian

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to