X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/353327844e319db755c2e7a07a4cf5b53cef35c4..6f026b5b63fe7ccb025e84509886f74772b9df13:/src/unix/dir.cpp diff --git a/src/unix/dir.cpp b/src/unix/dir.cpp index 13b8d2a68a..f88a594935 100644 --- a/src/unix/dir.cpp +++ b/src/unix/dir.cpp @@ -1,12 +1,11 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: unix/dir.cpp +// Name: src/unix/dir.cpp // Purpose: wxDir implementation for Unix/POSIX systems // Author: Vadim Zeitlin // Modified by: // Created: 08.12.99 -// RCS-ID: $Id$ // Copyright: (c) 1999 Vadim Zeitlin -// Licence: wxWindows license +// Licence: wxWindows licence ///////////////////////////////////////////////////////////////////////////// // ============================================================================ @@ -17,10 +16,6 @@ // headers // ---------------------------------------------------------------------------- -#ifdef __GNUG__ - #pragma implementation "dir.h" -#endif - // For compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" @@ -35,8 +30,11 @@ #include "wx/dir.h" #include "wx/filefn.h" // for wxMatchWild +#include "wx/filename.h" #include +#include +#include #include @@ -93,7 +91,7 @@ wxDirData::wxDirData(const wxString& dirname) // throw away the trailing slashes size_t n = m_dirname.length(); - wxCHECK_RET( n, _T("empty dir name in wxDir") ); + wxCHECK_RET( n, wxT("empty dir name in wxDir") ); while ( n > 0 && m_dirname[--n] == '/' ) ; @@ -110,21 +108,34 @@ wxDirData::~wxDirData() { if ( closedir(m_dir) != 0 ) { - wxLogLastError(_T("closedir")); + wxLogLastError(wxT("closedir")); } } } bool wxDirData::Read(wxString *filename) { - dirent *de = (dirent *)NULL; // just to silent compiler warnings - bool matches = FALSE; + dirent *de = NULL; // just to silence compiler warnings + bool matches = false; + + // speed up string concatenation in the loop a bit + wxString path = m_dirname; + path += wxT('/'); + path.reserve(path.length() + 255); + + wxString de_d_name; while ( !matches ) { de = readdir(m_dir); if ( !de ) - return FALSE; + return false; + +#if wxUSE_UNICODE + de_d_name = wxString(de->d_name, *wxConvFileName); +#else + de_d_name = de->d_name; +#endif // don't return "." and ".." unless asked for if ( de->d_name[0] == '.' && @@ -133,45 +144,53 @@ bool wxDirData::Read(wxString *filename) { if ( !(m_flags & wxDIR_DOTDOT) ) continue; + + // we found a valid match + break; } - // check the type now - if ( !(m_flags & wxDIR_FILES) && - !wxDir::Exists(m_dirname + _T('/') + de->d_name) ) + // check the type now: notice that we may want to check the type of + // the path itself and not whatever it points to in case of a symlink + wxFileName fn = wxFileName::DirName(path + de_d_name); + if ( m_flags & wxDIR_NO_FOLLOW ) + { + fn.DontFollowLink(); + } + + if ( !(m_flags & wxDIR_FILES) && !fn.DirExists() ) { // it's a file, but we don't want them continue; } - else if ( !(m_flags & wxDIR_DIRS) && - wxDir::Exists(m_dirname + _T('/') + de->d_name) ) + else if ( !(m_flags & wxDIR_DIRS) && fn.DirExists() ) { // it's a dir, and we don't want it continue; } // finally, check the name - if ( !m_filespec ) + if ( m_filespec.empty() ) { - matches = m_flags & wxDIR_HIDDEN ? TRUE : de->d_name[0] != '.'; + matches = m_flags & wxDIR_HIDDEN ? true : de->d_name[0] != '.'; } else { // test against the pattern - matches = wxMatchWild(m_filespec, de->d_name, + matches = wxMatchWild(m_filespec, de_d_name, !(m_flags & wxDIR_HIDDEN)); } } - *filename = de->d_name; + *filename = de_d_name; - return TRUE; + return true; } #else // old VMS (TODO) wxDirData::wxDirData(const wxString& WXUNUSED(dirname)) { - wxFAIL_MSG(_T("not implemented")); + wxFAIL_MSG(wxT("not implemented")); } wxDirData::~wxDirData() @@ -180,21 +199,11 @@ wxDirData::~wxDirData() bool wxDirData::Read(wxString * WXUNUSED(filename)) { - return FALSE; + return false; } #endif // not or new VMS/old VMS -// ---------------------------------------------------------------------------- -// wxDir helpers -// ---------------------------------------------------------------------------- - -/* static */ -bool wxDir::Exists(const wxString& dir) -{ - return wxPathExists(dir); -} - // ---------------------------------------------------------------------------- // wxDir construction/destruction // ---------------------------------------------------------------------------- @@ -213,16 +222,13 @@ bool wxDir::Open(const wxString& dirname) if ( !M_DIR->IsOk() ) { - wxLogSysError(_("Can not enumerate files in directory '%s'"), - dirname.c_str()); - delete M_DIR; m_data = NULL; - return FALSE; + return false; } - return TRUE; + return true; } bool wxDir::IsOpened() const @@ -236,19 +242,26 @@ wxString wxDir::GetName() const if ( m_data ) { name = M_DIR->GetName(); - if ( !name.empty() && (name.Last() == _T('/')) ) + + // Notice that we need to check for length > 1 as we shouldn't remove + // the last slash from the root directory! + if ( name.length() > 1 && (name.Last() == wxT('/')) ) { - // chop off the last (back)slash - name.Truncate(name.length() - 1); + // chop off the last slash + name.RemoveLast(); } } return name; } -wxDir::~wxDir() +void wxDir::Close() { - delete M_DIR; + if ( m_data ) + { + delete m_data; + m_data = NULL; + } } // ---------------------------------------------------------------------------- @@ -259,7 +272,7 @@ bool wxDir::GetFirst(wxString *filename, const wxString& filespec, int flags) const { - wxCHECK_MSG( IsOpened(), FALSE, _T("must wxDir::Open() first") ); + wxCHECK_MSG( IsOpened(), false, wxT("must wxDir::Open() first") ); M_DIR->Rewind(); @@ -271,9 +284,55 @@ bool wxDir::GetFirst(wxString *filename, bool wxDir::GetNext(wxString *filename) const { - wxCHECK_MSG( IsOpened(), FALSE, _T("must wxDir::Open() first") ); + wxCHECK_MSG( IsOpened(), false, wxT("must wxDir::Open() first") ); - wxCHECK_MSG( filename, FALSE, _T("bad pointer in wxDir::GetNext()") ); + wxCHECK_MSG( filename, false, wxT("bad pointer in wxDir::GetNext()") ); return M_DIR->Read(filename); } + +bool wxDir::HasSubDirs(const wxString& spec) const +{ + wxCHECK_MSG( IsOpened(), false, wxT("must wxDir::Open() first") ); + + if ( spec.empty() ) + { + // faster check for presence of any subdirectory: normally each subdir + // has a hard link to the parent directory and so, knowing that there + // are at least "." and "..", we have a subdirectory if and only if + // links number is > 2 - this is just a guess but it works fairly well + // in practice + // + // note that we may guess wrongly in one direction only: i.e. we may + // return true when there are no subdirectories but this is ok as the + // caller will learn it soon enough when it calls GetFirst(wxDIR) + // anyhow + wxStructStat stBuf; + if ( wxStat(M_DIR->GetName(), &stBuf) == 0 ) + { + switch ( stBuf.st_nlink ) + { + case 2: + // just "." and ".." + return false; + + case 0: + case 1: + // weird filesystem, don't try to guess for it, use dumb + // method below + break; + + default: + // assume we have subdirs - may turn out to be wrong if we + // have other hard links to this directory but it's not + // that bad as explained above + return true; + } + } + } + + // just try to find first directory + wxString s; + return GetFirst(&s, spec, wxDIR_DIRS | wxDIR_HIDDEN); +} +