/////////////////////////////////////////////////////////////////////////////
-// Name: file.cpp
+// Name: src/common/file.cpp
// Purpose: wxFile - encapsulates low-level "file descriptor"
// wxTempFile
// Author: Vadim Zeitlin
// Created: 29/01/98
// RCS-ID: $Id$
// Copyright: (c) 1998 Vadim Zeitlin <zeitlin@dptmaths.ens-cachan.fr>
-// Licence: wxWindows license
+// Licence: wxWindows licence
/////////////////////////////////////////////////////////////////////////////
// ----------------------------------------------------------------------------
// headers
// ----------------------------------------------------------------------------
-#ifdef __GNUG__
- #pragma implementation "file.h"
-#endif
-
// For compilers that support precompilation, includes "wx.h".
#include "wx/wxprec.h"
#pragma hdrstop
#endif
-#ifndef WX_PRECOMP
- #include "wx/defs.h"
-#endif
-
#if wxUSE_FILE
// standard
-#if defined(__WXMSW__) && !defined(__GNUWIN32__) && !defined(__WXWINE__)
- #include <io.h>
-
-#ifndef __SALFORDC__
- #define WIN32_LEAN_AND_MEAN
- #define NOSERVICE
- #define NOIME
- #define NOATOM
- #define NOGDI
- #define NOGDICAPMASKS
- #define NOMETAFILE
- #define NOMINMAX
- #define NOMSG
- #define NOOPENFILE
- #define NORASTEROPS
- #define NOSCROLL
- #define NOSOUND
- #define NOSYSMETRICS
- #define NOTEXTMETRIC
- #define NOWH
- #define NOCOMM
- #define NOKANJI
- #define NOCRYPT
- #define NOMCX
-#endif
-
- #include <windows.h> // for GetTempFileName
+#if defined(__WINDOWS__) && !defined(__GNUWIN32__) && !defined(__WXMICROWIN__) && !defined(__WXWINCE__)
+
+#define WIN32_LEAN_AND_MEAN
+#define NOSERVICE
+#define NOIME
+#define NOATOM
+#define NOGDI
+#define NOGDICAPMASKS
+#define NOMETAFILE
+#define NOMINMAX
+#define NOMSG
+#define NOOPENFILE
+#define NORASTEROPS
+#define NOSCROLL
+#define NOSOUND
+#define NOSYSMETRICS
+#define NOTEXTMETRIC
+#define NOWH
+#define NOCOMM
+#define NOKANJI
+#define NOCRYPT
+#define NOMCX
+
+#elif defined(__WINDOWS__) && defined(__WXWINCE__)
+ #include "wx/msw/missing.h"
+#elif (defined(__OS2__))
+ #include <io.h>
#elif (defined(__UNIX__) || defined(__GNUWIN32__))
#include <unistd.h>
+ #include <time.h>
+ #include <sys/stat.h>
#ifdef __GNUWIN32__
- #include <windows.h>
+ #include "wx/msw/wrapwin.h"
+ #endif
+#elif defined(__DOS__)
+ #if defined(__WATCOMC__)
+ #include <io.h>
+ #elif defined(__DJGPP__)
+ #include <io.h>
+ #include <unistd.h>
+ #include <stdio.h>
+ #else
+ #error "Please specify the header with file functions declarations."
#endif
#elif (defined(__WXSTUBS__))
// Have to ifdef this for different environments
#include <io.h>
#elif (defined(__WXMAC__))
+#if __MSL__ < 0x6000
int access( const char *path, int mode ) { return 0 ; }
+#else
+ int _access( const char *path, int mode ) { return 0 ; }
+#endif
char* mktemp( char * path ) { return path ;}
- #include <unistd.h>
- #include <unix.h>
- #define W_OK 2
- #define R_OK 4
+ #include <stat.h>
+ #include <unistd.h>
#else
#error "Please specify the header with file functions declarations."
#endif //Win/UNIX
#include <stdio.h> // SEEK_xxx constants
-#include <fcntl.h> // O_RDONLY &c
-
-#ifndef __MWERKS__
- #include <sys/types.h> // needed for stat
- #include <sys/stat.h> // stat
-#endif
-
-// Microsoft compiler loves underscores, feed them to it
-#ifdef __VISUALC__
- // functions
- #define open _open
- #define close _close
- #define read _read
- #define write _write
- #define lseek _lseek
- #define fsync _commit
- #define access _access
- #define eof _eof
-
- // types
- #define stat _stat
-
- // constants
-
- #define O_RDONLY _O_RDONLY
- #define O_WRONLY _O_WRONLY
- #define O_RDWR _O_RDWR
- #define O_EXCL _O_EXCL
- #define O_CREAT _O_CREAT
- #define O_BINARY _O_BINARY
-
- #define S_IFDIR _S_IFDIR
- #define S_IFREG _S_IFREG
-#else
- #define tell(fd) lseek(fd, 0, SEEK_CUR)
-#endif // VC++
-#if defined(__BORLANDC__) || defined(_MSC_VER)
- #define W_OK 2
- #define R_OK 4
+#ifndef __WXWINCE__
+ #include <errno.h>
#endif
-// there is no distinction between text and binary files under Unix
-#ifdef __UNIX__
+// Windows compilers don't have these constants
+#ifndef W_OK
+ enum
+ {
+ F_OK = 0, // test for existence
+ X_OK = 1, // execute permission
+ W_OK = 2, // write
+ R_OK = 4 // read
+ };
+#endif // W_OK
+
+// wxWidgets
+#ifndef WX_PRECOMP
+ #include "wx/string.h"
+ #include "wx/intl.h"
+ #include "wx/log.h"
+ #include "wx/crt.h"
+#endif // !WX_PRECOMP
+
+#include "wx/filename.h"
+#include "wx/file.h"
+#include "wx/filefn.h"
+
+// there is no distinction between text and binary files under Unix, so define
+// O_BINARY as 0 if the system headers don't do it already
+#if defined(__UNIX__) && !defined(O_BINARY)
#define O_BINARY (0)
#endif //__UNIX__
-#ifdef __SALFORDC__
- #include <unix.h>
+#ifdef __WINDOWS__
+ #include "wx/msw/mslu.h"
#endif
-// wxWindows
-#include <wx/string.h>
-#include <wx/intl.h>
-#include <wx/file.h>
-#include <wx/log.h>
+#ifdef __WXWINCE__
+ #include "wx/msw/private.h"
+#endif
#ifndef MAX_PATH
#define MAX_PATH 512
#endif
-#ifdef __WXMAC__
- char gwxMacFileName[ MAX_PATH ] ;
- char gwxMacFileName2[ MAX_PATH ] ;
- char gwxMacFileName3[ MAX_PATH ] ;
-#endif
-
-// some broken compilers don't have 3rd argument in open() and creat()
-#ifdef __SALFORDC__
- #define ACCESS(access)
- #define stat _stat
-#else // normal compiler
- #define ACCESS(access) , (access)
-#endif // Salford C
-
// ============================================================================
// implementation of wxFile
// ============================================================================
// ----------------------------------------------------------------------------
// static functions
// ----------------------------------------------------------------------------
-bool wxFile::Exists(const wxChar *name)
-{
- struct stat st;
-#if wxUSE_UNICODE && wxMBFILES
- wxCharBuffer fname = wxConvFile.cWC2MB(name);
- return !access(fname, 0) &&
- !stat(MBSTRINGCAST fname, &st) &&
- (st.st_mode & S_IFREG);
-#else
- return !access(name, 0) &&
- !stat((wxChar*) name, &st) &&
- (st.st_mode & S_IFREG);
-#endif
+bool wxFile::Exists(const wxString& name)
+{
+ return wxFileExists(name);
}
-bool wxFile::Access(const wxChar *name, OpenMode mode)
+bool wxFile::Access(const wxString& name, OpenMode mode)
{
- int how = 0;
+ int how;
+
+ switch ( mode )
+ {
+ default:
+ wxFAIL_MSG(wxT("bad wxFile::Access mode parameter."));
+ // fall through
- switch ( mode ) {
case read:
how = R_OK;
break;
how = W_OK;
break;
- default:
- wxFAIL_MSG(_T("bad wxFile::Access mode parameter."));
+ case read_write:
+ how = R_OK | W_OK;
+ break;
}
- return access(wxFNCONV(name), how) == 0;
+ return wxAccess(name, how) == 0;
}
// ----------------------------------------------------------------------------
// ----------------------------------------------------------------------------
// ctors
-wxFile::wxFile(const wxChar *szFileName, OpenMode mode)
+wxFile::wxFile(const wxString& fileName, OpenMode mode)
{
m_fd = fd_invalid;
- m_error = FALSE;
+ m_lasterror = 0;
- Open(szFileName, mode);
+ Open(fileName, mode);
}
-// dtor
-wxFile::~wxFile()
+bool wxFile::CheckForError(wxFileOffset rc) const
{
- Close();
+ if ( rc != -1 )
+ return false;
+
+ const_cast<wxFile *>(this)->m_lasterror =
+#ifndef __WXWINCE__
+ errno
+#else
+ ::GetLastError()
+#endif
+ ;
+
+ return true;
}
// create the file, fail if it already exists and bOverwrite
-bool wxFile::Create(const wxChar *szFileName, bool bOverwrite, int accessMode)
+bool wxFile::Create(const wxString& fileName, bool bOverwrite, int accessMode)
{
// if bOverwrite we create a new file or truncate the existing one,
// otherwise we only create the new file and fail if it already exists
- int fd = open(wxFNCONV(szFileName),
- O_WRONLY | O_CREAT | (bOverwrite ? O_TRUNC : O_EXCL)
- ACCESS(accessMode));
-
- if ( fd == -1 ) {
- wxLogSysError(_("can't create file '%s'"), szFileName);
- return FALSE;
- }
- else {
- Attach(fd);
- return TRUE;
+ int fildes = wxOpen( fileName,
+ O_BINARY | O_WRONLY | O_CREAT |
+ (bOverwrite ? O_TRUNC : O_EXCL),
+ accessMode );
+ if ( CheckForError(fildes) )
+ {
+ wxLogSysError(_("can't create file '%s'"), fileName);
+ return false;
}
+
+ Attach(fildes);
+ return true;
}
// open the file
-bool wxFile::Open(const wxChar *szFileName, OpenMode mode, int accessMode)
+bool wxFile::Open(const wxString& fileName, OpenMode mode, int accessMode)
{
int flags = O_BINARY;
- switch ( mode ) {
+ switch ( mode )
+ {
case read:
flags |= O_RDONLY;
break;
+ case write_append:
+ if ( wxFile::Exists(fileName) )
+ {
+ flags |= O_WRONLY | O_APPEND;
+ break;
+ }
+ //else: fall through as write_append is the same as write if the
+ // file doesn't exist
+
case write:
flags |= O_WRONLY | O_CREAT | O_TRUNC;
break;
- case write_append:
- flags |= O_WRONLY | O_APPEND;
+ case write_excl:
+ flags |= O_WRONLY | O_CREAT | O_EXCL;
break;
case read_write:
break;
}
- int fd = open(wxFNCONV(szFileName), flags ACCESS(accessMode));
+#ifdef __WINDOWS__
+ // only read/write bits for "all" are supported by this function under
+ // Windows, and VC++ 8 returns EINVAL if any other bits are used in
+ // accessMode, so clear them as they have at best no effect anyhow
+ accessMode &= wxS_IRUSR | wxS_IWUSR;
+#endif // __WINDOWS__
- if ( fd == -1 ) {
- wxLogSysError(_("can't open file '%s'"), szFileName);
- return FALSE;
- }
- else {
- Attach(fd);
- return TRUE;
+ int fildes = wxOpen( fileName, flags, accessMode);
+
+ if ( CheckForError(fildes) )
+ {
+ wxLogSysError(_("can't open file '%s'"), fileName);
+ return false;
}
+
+ Attach(fildes);
+ return true;
}
// close
bool wxFile::Close()
{
if ( IsOpened() ) {
- if ( close(m_fd) == -1 ) {
+ if ( CheckForError(wxClose(m_fd)) )
+ {
wxLogSysError(_("can't close file descriptor %d"), m_fd);
m_fd = fd_invalid;
- return FALSE;
+ return false;
}
else
m_fd = fd_invalid;
}
- return TRUE;
+ return true;
}
// ----------------------------------------------------------------------------
// read/write
// ----------------------------------------------------------------------------
+bool wxFile::ReadAll(wxString *str, const wxMBConv& conv)
+{
+ wxCHECK_MSG( str, false, wxS("Output string must be non-NULL") );
+
+ ssize_t length = Length();
+ wxCHECK_MSG( (wxFileOffset)length == Length(), false, wxT("huge file not supported") );
+
+ wxCharBuffer buf(length);
+ char* p = buf.data();
+ for ( ;; )
+ {
+ static const ssize_t READSIZE = 4096;
+
+ ssize_t nread = Read(p, length > READSIZE ? READSIZE : length);
+ if ( nread == wxInvalidOffset )
+ return false;
+
+ p += nread;
+ if ( length <= nread )
+ break;
+
+ length -= nread;
+ }
+
+ *p = 0;
+
+ wxString strTmp(buf, conv);
+ str->swap(strTmp);
+
+ return true;
+}
+
// read
-off_t wxFile::Read(void *pBuf, off_t nCount)
+ssize_t wxFile::Read(void *pBuf, size_t nCount)
{
wxCHECK( (pBuf != NULL) && IsOpened(), 0 );
-#ifdef __MWERKS__
- int iRc = ::read(m_fd, (char*) pBuf, nCount);
-#else
- int iRc = ::read(m_fd, pBuf, nCount);
-#endif
- if ( iRc == -1 ) {
+ ssize_t iRc = wxRead(m_fd, pBuf, nCount);
+
+ if ( CheckForError(iRc) )
+ {
wxLogSysError(_("can't read from file descriptor %d"), m_fd);
return wxInvalidOffset;
}
- else
- return (size_t)iRc;
+
+ return iRc;
}
// write
{
wxCHECK( (pBuf != NULL) && IsOpened(), 0 );
-#ifdef __MWERKS__
- int iRc = ::write(m_fd, (const char*) pBuf, nCount);
-#else
- int iRc = ::write(m_fd, pBuf, nCount);
-#endif
- if ( iRc == -1 ) {
+ ssize_t iRc = wxWrite(m_fd, pBuf, nCount);
+
+ if ( CheckForError(iRc) )
+ {
wxLogSysError(_("can't write to file descriptor %d"), m_fd);
- m_error = TRUE;
- return 0;
+ iRc = 0;
}
- else
- return iRc;
+
+ return iRc;
+}
+
+bool wxFile::Write(const wxString& s, const wxMBConv& conv)
+{
+ const wxWX2MBbuf buf = s.mb_str(conv);
+ if ( !buf )
+ return false;
+
+#if wxUSE_UNICODE
+ const size_t size = buf.length();
+#else
+ const size_t size = s.length();
+#endif
+
+ return Write(buf, size) == size;
}
// flush
bool wxFile::Flush()
{
- if ( IsOpened() ) {
-#if defined(__VISUALC__) || wxHAVE_FSYNC
- if ( fsync(m_fd) == -1 )
+#ifdef HAVE_FSYNC
+ // fsync() only works on disk files and returns errors for pipes, don't
+ // call it then
+ if ( IsOpened() && GetKind() == wxFILE_KIND_DISK )
+ {
+ if ( CheckForError(wxFsync(m_fd)) )
{
wxLogSysError(_("can't flush file descriptor %d"), m_fd);
- return FALSE;
+ return false;
}
-#else // no fsync
- // just do nothing
-#endif // fsync
}
+#endif // HAVE_FSYNC
- return TRUE;
+ return true;
}
// ----------------------------------------------------------------------------
// ----------------------------------------------------------------------------
// seek
-off_t wxFile::Seek(off_t ofs, wxSeekMode mode)
+wxFileOffset wxFile::Seek(wxFileOffset ofs, wxSeekMode mode)
{
- wxASSERT( IsOpened() );
+ wxASSERT_MSG( IsOpened(), wxT("can't seek on closed file") );
+ wxCHECK_MSG( ofs != wxInvalidOffset || mode != wxFromStart,
+ wxInvalidOffset,
+ wxT("invalid absolute file offset") );
- int flag = -1;
+ int origin;
switch ( mode ) {
+ default:
+ wxFAIL_MSG(wxT("unknown seek origin"));
+
case wxFromStart:
- flag = SEEK_SET;
+ origin = SEEK_SET;
break;
case wxFromCurrent:
- flag = SEEK_CUR;
+ origin = SEEK_CUR;
break;
case wxFromEnd:
- flag = SEEK_END;
+ origin = SEEK_END;
break;
-
- default:
- wxFAIL_MSG(_("unknown seek origin"));
}
- int iRc = lseek(m_fd, ofs, flag);
- if ( iRc == -1 ) {
+ wxFileOffset iRc = wxSeek(m_fd, ofs, origin);
+ if ( CheckForError(iRc) )
+ {
wxLogSysError(_("can't seek on file descriptor %d"), m_fd);
- return wxInvalidOffset;
}
- else
- return (off_t)iRc;
+
+ return iRc;
}
-// get current off_t
-off_t wxFile::Tell() const
+// get current file offset
+wxFileOffset wxFile::Tell() const
{
wxASSERT( IsOpened() );
- int iRc = tell(m_fd);
- if ( iRc == -1 ) {
+ wxFileOffset iRc = wxTell(m_fd);
+ if ( CheckForError(iRc) )
+ {
wxLogSysError(_("can't get seek position on file descriptor %d"), m_fd);
- return wxInvalidOffset;
}
- else
- return (off_t)iRc;
+
+ return iRc;
}
// get current file length
-off_t wxFile::Length() const
+wxFileOffset wxFile::Length() const
{
wxASSERT( IsOpened() );
-#ifdef __VISUALC__
- int iRc = _filelength(m_fd);
-#else // !VC++
- int iRc = tell(m_fd);
- if ( iRc != -1 ) {
- // @ have to use const_cast :-(
- int iLen = ((wxFile *)this)->SeekEnd();
- if ( iLen != -1 ) {
+ // we use a special method for Linux systems where files in sysfs (i.e.
+ // those under /sys typically) return length of 4096 bytes even when
+ // they're much smaller -- this is a problem as it results in errors later
+ // when we try reading 4KB from them
+#ifdef __LINUX__
+ struct stat st;
+ if ( fstat(m_fd, &st) == 0 )
+ {
+ // returning 0 for the special files indicates to the caller that they
+ // are not seekable
+ return st.st_blocks ? st.st_size : 0;
+ }
+ //else: failed to stat, try the normal method
+#endif // __LINUX__
+
+ wxFileOffset iRc = Tell();
+ if ( iRc != wxInvalidOffset ) {
+ wxFileOffset iLen = const_cast<wxFile *>(this)->SeekEnd();
+ if ( iLen != wxInvalidOffset ) {
// restore old position
- if ( ((wxFile *)this)->Seek(iRc) == -1 ) {
+ if ( ((wxFile *)this)->Seek(iRc) == wxInvalidOffset ) {
// error
- iLen = -1;
+ iLen = wxInvalidOffset;
}
}
iRc = iLen;
}
-#endif // VC++
- if ( iRc == -1 ) {
+ if ( iRc == wxInvalidOffset )
+ {
+ // last error was already set by Tell()
wxLogSysError(_("can't find length of file on file descriptor %d"), m_fd);
- return wxInvalidOffset;
}
- else
- return (off_t)iRc;
+
+ return iRc;
}
// is end of file reached?
{
wxASSERT( IsOpened() );
- int iRc;
+ wxFileOffset iRc;
-#if defined(__UNIX__) || defined(__GNUWIN32__) || defined( __MWERKS__ ) || defined(__SALFORDC__)
+#if defined(__DOS__) || defined(__UNIX__) || defined(__GNUWIN32__)
// @@ this doesn't work, of course, on unseekable file descriptors
- off_t ofsCur = Tell(),
+ wxFileOffset ofsCur = Tell(),
ofsMax = Length();
if ( ofsCur == wxInvalidOffset || ofsMax == wxInvalidOffset )
- iRc = -1;
+ iRc = wxInvalidOffset;
else
iRc = ofsCur == ofsMax;
#else // Windows and "native" compiler
- iRc = eof(m_fd);
+ iRc = wxEof(m_fd);
#endif // Windows/Unix
- switch ( iRc ) {
- case 1:
- break;
-
- case 0:
- return FALSE;
-
- case -1:
- wxLogSysError(_("can't determine if the end of file is reached on \
- descriptor %d"), m_fd);
- break;
+ if ( iRc == 0 )
+ return false;
- default:
- wxFAIL_MSG(_("invalid eof() return value."));
+ if ( iRc == wxInvalidOffset )
+ {
+ wxLogSysError(_("can't determine if the end of file is reached on descriptor %d"), m_fd);
}
- return TRUE;
+ return true;
}
// ============================================================================
// ----------------------------------------------------------------------------
// construction
// ----------------------------------------------------------------------------
+
wxTempFile::wxTempFile(const wxString& strName)
{
Open(strName);
bool wxTempFile::Open(const wxString& strName)
{
- m_strName = strName;
-
- // we want to create the file in the same directory as strName because
- // otherwise rename() in Commit() might not work (if the files are on
- // different partitions for example). Unfortunately, the only standard
- // (POSIX) temp file creation function tmpnam() can't do it.
-#if defined(__UNIX__) || defined(__WXSTUBS__)|| defined( __WXMAC__ )
- static const wxChar *szMktempSuffix = _T("XXXXXX");
- m_strTemp << strName << szMktempSuffix;
- mktemp(MBSTRINGCAST m_strTemp.mb_str()); // will do because length doesn't change
-#else // Windows
- wxString strPath;
- wxSplitPath(strName, &strPath, NULL, NULL);
- if ( strPath.IsEmpty() )
- strPath = _T('.'); // GetTempFileName will fail if we give it empty string
-#ifdef __WIN32__
- if ( !GetTempFileName(strPath, _T("wx_"),0, m_strTemp.GetWriteBuf(MAX_PATH)) )
-#else
- // Not sure why MSVC++ 1.5 header defines first param as BYTE - bug?
- if ( !GetTempFileName((BYTE) (const wxChar*) strPath, _T("wx_"),0, m_strTemp.GetWriteBuf(MAX_PATH)) )
-#endif
- wxLogLastError(_T("GetTempFileName"));
- m_strTemp.UngetWriteBuf();
-#endif // Windows/Unix
+ // we must have an absolute filename because otherwise CreateTempFileName()
+ // would create the temp file in $TMP (i.e. the system standard location
+ // for the temp files) which might be on another volume/drive/mount and
+ // wxRename()ing it later to m_strName from Commit() would then fail
+ //
+ // with the absolute filename, the temp file is created in the same
+ // directory as this one which ensures that wxRename() may work later
+ wxFileName fn(strName);
+ if ( !fn.IsAbsolute() )
+ {
+ fn.Normalize(wxPATH_NORM_ABSOLUTE);
+ }
+
+ m_strName = fn.GetFullPath();
+
+ m_strTemp = wxFileName::CreateTempFileName(m_strName, &m_file);
+
+ if ( m_strTemp.empty() )
+ {
+ // CreateTempFileName() failed
+ return false;
+ }
- int access = wxS_DEFAULT;
#ifdef __UNIX__
- // create the file with the same mode as the original one under Unix
- mode_t umaskOld = 0; // just to suppress compiler warning
- bool changedUmask;
+ // the temp file should have the same permissions as the original one
+ mode_t mode;
- struct stat st;
- if ( stat(strName.fn_str(), &st) == 0 )
+ wxStructStat st;
+ if ( stat( (const char*) m_strName.fn_str(), &st) == 0 )
{
- // this assumes that only lower bits of st_mode contain the access
- // rights, but it's true for at least all Unices which have S_IXXXX()
- // macros, so should not be less portable than using (not POSIX)
- // S_IFREG &c
- access = st.st_mode & 0777;
-
- // we want to create the file with exactly the same access rights as
- // the original one, so disable the user's umask for the moment
- umaskOld = umask(0);
- changedUmask = TRUE;
+ mode = st.st_mode;
}
else
{
- // file probably didn't exist, just create with default mode _using_
- // user's umask (new files creation should respet umask)
- changedUmask = FALSE;
+ // file probably didn't exist, just give it the default mode _using_
+ // user's umask (new files creation should respect umask)
+ mode_t mask = umask(0777);
+ mode = 0666 & ~mask;
+ umask(mask);
}
-#endif // Unix
- bool ok = m_file.Open(m_strTemp, wxFile::write, access);
-
-#ifdef __UNIX__
- if ( changedUmask )
+ if ( chmod( (const char*) m_strTemp.fn_str(), mode) == -1 )
{
- // restore umask now that the file is created
- (void)umask(umaskOld);
+#ifndef __OS2__
+ wxLogSysError(_("Failed to set temporary file permissions"));
+#endif
}
#endif // Unix
- return ok;
+ return true;
}
// ----------------------------------------------------------------------------
{
m_file.Close();
- if ( wxFile::Exists(m_strName) && remove(m_strName.fn_str()) != 0 ) {
+ if ( wxFile::Exists(m_strName) && wxRemove(m_strName) != 0 ) {
wxLogSysError(_("can't remove file '%s'"), m_strName.c_str());
- return FALSE;
+ return false;
}
- if ( rename(m_strTemp.fn_str(), m_strName.fn_str()) != 0 ) {
+ if ( !wxRenameFile(m_strTemp, m_strName) ) {
wxLogSysError(_("can't commit changes to file '%s'"), m_strName.c_str());
- return FALSE;
+ return false;
}
- return TRUE;
+ return true;
}
void wxTempFile::Discard()
{
m_file.Close();
- if ( remove(m_strTemp.fn_str()) != 0 )
+ if ( wxRemove(m_strTemp) != 0 )
+ {
wxLogSysError(_("can't remove temporary file '%s'"), m_strTemp.c_str());
+ }
}
-#endif
\ No newline at end of file
+#endif // wxUSE_FILE
+
projects / wxWidgets.git / blobdiff