20.01.2013 22:14, Max Kellermann пишет:
>
> - FileSystem.hxx includes all relevant system headers, and I would
> like to reduce header dependencies as much as possible; include
> special things like dirent.h only if you really want to read
> directory contents
Agree, I think we could extract DirectoryReader to the separate header
and also introduce new subdir in source tree, i.e.:
src/
fs/
FileSystem.hxx
FileSystem.cxx
DirectoryReader.hxx
>
> - ReadLink() should better return a "Path" object, and Path::Null() on
> error
I've done it so initially, but what would happen in the following situation:
// Initializing p1 with some non-null value
Path p1 = ...;
// Reading the link and suppose ReadLink returns Path::Null()
p1 = ReadLink(...);
In this case old value of p1 would destroyed via free(), which might
under some condition overwrite our errno.
> - CheckIsRegular(): for me, a name like IsRegularFile() sounds better
Agree, those names are invented under effects of reading g_file_test()
documentation :-)
Couldn't it be something like the following?
PathExists()/FileExists()/DirectoryExists()
Looks clean and symmetric.
> - slightly disagree with Ben (but thanks for pointing out that
> potential problem): there should be some way to specify whether
> symbolic links shall be followed. Some callers may want to follow
> symlinks, some do not, and the database update has that configurable
> at runtime.
Approach with extra parameter (i.e. follow_symlinks) seems OK for me.
However I'm not sure which variant should be default.
--
Denis
------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. ON SALE this month only -- learn more at:
http://p.sf.net/sfu/learnmore_123012
_______________________________________________
Musicpd-dev-team mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/musicpd-dev-team
