[Webkit-unassigned] [Bug 60361] [EFL] EWebLauncher creates only one settings databases directory at startup.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=60361


Kenneth Rohde Christiansen <kenneth at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kenneth at webkit.org




--- Comment #15 from Kenneth Rohde Christiansen <kenneth at webkit.org>  2011-05-17 13:25:42 PST ---
(In reply to comment #14)
> Just based on the name makeAllDirectories it is not possible to conclude what should it return in case the path already existed and if it is or not a directory, but I would say that permissions are not related to makeAllDirectories. There is no documentation and it seems that different platform implementations differ a bit from each other, but none of the ones I've seen are checking permissions.
> 
> This patch just considers that makeAllDirectories(path) will not return false if path already is a directory.
> 
> I like the patch as it is.

I looked at the patch and it is definitely an improvement. I think the above should be considered and can easily be fixed/improved in another patch. Thanks Luiz

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.