]> git.lyx.org Git - lyx.git/commitdiff
* support/FileName:
authorAbdelrazak Younes <younes@lyx.org>
Mon, 26 Nov 2007 10:29:04 +0000 (10:29 +0000)
committerAbdelrazak Younes <younes@lyx.org>
Mon, 26 Nov 2007 10:29:04 +0000 (10:29 +0000)
- simplify copyTo(), the second argument was never used.
- move string manipulations to FileName::Private
- replace platform specific code with Qt equivalent.

git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@21793 a592a061-630c-0410-9148-cb99ea01b6c8

src/Buffer.cpp
src/EmbeddedFiles.cpp
src/support/FileName.cpp
src/support/FileName.h

index e068bc9d914c691565a95a3b0ffb3543decb6c74..af30fa6d265245c2af30a91164828b5216afe51c 100644 (file)
@@ -853,7 +853,7 @@ bool Buffer::save() const
                        backupName = FileName(addName(lyxrc.backupdir_path,
                                                      mangledName));
                }
-               if (fileName().copyTo(backupName, false)) {
+               if (fileName().copyTo(backupName)) {
                        madeBackup = true;
                } else {
                        Alert::error(_("Backup failure"),
index bf2e5fbd358c3089ca5335d350c2c64d0568b24f..2d2aef8c37b842a1c93d4f2340528fc9191529d2 100644 (file)
@@ -156,7 +156,7 @@ bool EmbeddedFile::extract(Buffer const * buf) const
        FileName path = ext.onlyPath();
        if (!path.isDirectory())
                makedir(const_cast<char*>(path.absFilename().c_str()), 0755);
-       if (emb.copyTo(ext, false))
+       if (emb.copyTo(ext))
                return true;
        Alert::error(_("Copy file failure"),
                 bformat(_("Cannot copy file %1$s to %2$s.\n"
@@ -198,7 +198,7 @@ bool EmbeddedFile::updateFromExternalFile(Buffer const * buf) const
        FileName path = emb.onlyPath();
        if (!path.isDirectory())
                makedir(const_cast<char*>(path.absFilename().c_str()), 0755);
-       if (ext.copyTo(emb, false))
+       if (ext.copyTo(emb))
                return true;
        Alert::error(_("Copy file failure"),
                 bformat(_("Cannot copy file %1$s to %2$s.\n"
@@ -301,7 +301,7 @@ bool EmbeddedFiles::writeFile(DocFileName const & filename)
 
        ::zipFiles(zipfile.toFilesystemEncoding(), filenames);
        // copy file back
-       if (!zipfile.copyTo(filename, false)) {
+       if (!zipfile.copyTo(filename)) {
                Alert::error(_("Save failure"),
                                 bformat(_("Cannot create file %1$s.\n"
                                           "Please check whether the directory exists and is writeable."),
index 7f06ee793021eae566a9617f28aa83ac643c1917..170b3823b95b8dd53fc4ed3b238dd935c053c48b 100644 (file)
@@ -26,9 +26,6 @@
 #include <QList>
 
 #include <boost/assert.hpp>
-#include <boost/filesystem/config.hpp>
-#include <boost/detail/workaround.hpp>
-#include <boost/throw_exception.hpp>
 
 #include <map>
 #include <sstream>
@@ -51,141 +48,97 @@ using std::ifstream;
 using std::ostringstream;
 using std::endl;
 
+namespace lyx {
+namespace support {
 
-// FIXME: merge this
-//
-// BOOST_POSIX or BOOST_WINDOWS specify which API to use.
-# if !defined( BOOST_WINDOWS ) && !defined( BOOST_POSIX )
-#   if defined(_WIN32) || defined(__WIN32__) || defined(WIN32) || defined(__CYGWIN__)
-#     define BOOST_WINDOWS
-#   else
-#     define BOOST_POSIX
-#   endif
-# endif
-
-#if defined (BOOST_WINDOWS)
-# define WIN32_LEAN_AND_MEAN
-# include <windows.h>
-# undef min
-# undef max
-#endif
 
+/////////////////////////////////////////////////////////////////////
+//
+// FileName::Private
+//
+/////////////////////////////////////////////////////////////////////
 
-static bool copy_file(std::string const & source, std::string const & target, bool noclobber)
+struct FileName::Private
 {
+       Private() {}
 
-#ifdef BOOST_POSIX
-       int const infile = ::open(source.c_str(), O_RDONLY);
-       if (infile == -1)
-               return false;
-
-       struct stat source_stat;
-       int const ret = ::fstat(infile, &source_stat);
-       if (ret == -1) {
-               //int err = errno;
-               ::close(infile);
-       }
-
-       int const flags = O_WRONLY | O_CREAT | (noclobber ? O_EXCL : O_TRUNC);
-
-       int const outfile = ::open(target.c_str(), flags, source_stat.st_mode);
-       if (outfile == -1) {
-               //int err = errno;
-               ::close(infile);
-               return false;
-       }
-
-       std::size_t const buf_sz = 32768;
-       char buf[buf_sz];
-       ssize_t in = -1;
-       ssize_t out = -1;
+       Private(string const & abs_filename) : fi(toqstr(abs_filename))
+       {}
+       ///
+       QFileInfo fi;
+};
 
-       while (true) {
-               in = ::read(infile, buf, buf_sz);
-               if (in == -1) {
-                       break;
-               } else if (in == 0) {
-                       break;
-               } else {
-                       out = ::write(outfile, buf, in);
-                       if (out == -1) {
-                               break;
-                       }
-               }
-       }
+/////////////////////////////////////////////////////////////////////
+//
+// FileName
+//
+/////////////////////////////////////////////////////////////////////
 
-       //int err = errno;
 
-       ::close(infile);
-       ::close(outfile);
+FileName::FileName() : d(new Private)
+{}
 
-       if (in == -1 || out == -1)
-               return false;
+FileName::FileName(string const & abs_filename)
+       : d(abs_filename.empty() ? new Private : new Private(abs_filename))
+{
+       BOOST_ASSERT(empty() || d->fi.isAbsolute());
+#if defined(_WIN32)
+       BOOST_ASSERT(!contains(abs_filename, '\\'));
 #endif
+}
 
-#ifdef BOOST_WINDOWS
-       if (::CopyFile(source.c_str(), target.c_str(), noclobber) == 0) {
-               // CopyFile is probably not setting errno so this is most
-               // likely wrong.
-               return false;
-       }
-#endif
-       return true;
+
+FileName::FileName(FileName const & rhs) : d(new Private)
+{
+       d->fi = rhs.d->fi;
 }
 
 
-namespace lyx {
-namespace support {
+FileName & FileName::operator=(FileName const & rhs)
+{
+       d->fi = rhs.d->fi;
+       return *this;
+}
 
 
-/////////////////////////////////////////////////////////////////////
-//
-// FileName
-//
-/////////////////////////////////////////////////////////////////////
+bool FileName::empty() const
+{
+       lyxerr << "FileName::empty() '" << absFilename() << "'" << endl;
+       return d->fi.absoluteFilePath().isEmpty();
+}
 
 
-FileName::FileName(string const & abs_filename)
-       : name_(abs_filename)
+string FileName::absFilename() const
 {
-       BOOST_ASSERT(empty() || absolutePath(name_));
-#if defined(_WIN32)
-       BOOST_ASSERT(!contains(name_, '\\'));
-#endif
+       return fromqstr(d->fi.absoluteFilePath());
 }
 
 
 void FileName::set(string const & name)
 {
-       name_ = name;
-       BOOST_ASSERT(absolutePath(name_));
+       d->fi.setFile(toqstr(name));
+       BOOST_ASSERT(d->fi.isAbsolute());
 #if defined(_WIN32)
-       BOOST_ASSERT(!contains(name_, '\\'));
+       BOOST_ASSERT(!contains(name, '\\'));
 #endif
 }
 
 
 void FileName::erase()
 {
-       name_.erase();
+       d->fi = QFileInfo();
 }
 
 
-bool FileName::copyTo(FileName const & name, bool noclobber) const
+bool FileName::copyTo(FileName const & name) const
 {
-       try {
-               copy_file(toFilesystemEncoding(), name.toFilesystemEncoding(), noclobber);
-               return true;
-       }
-       catch (...) {
-       }
-       return false;
+       return QFile::copy(d->fi.absoluteFilePath(), name.d->fi.absoluteFilePath());
 }
 
 
 string FileName::toFilesystemEncoding() const
 {
-       QByteArray const encoded = QFile::encodeName(toqstr(name_));
+       QByteArray const encoded = QFile::encodeName(d->fi.absoluteFilePath());
        return string(encoded.begin(), encoded.end());
 }
 
@@ -199,39 +152,37 @@ FileName FileName::fromFilesystemEncoding(string const & name)
 
 bool FileName::exists() const
 {
-       return QFileInfo(toqstr(name_)).exists();
+       return d->fi.exists();
 }
 
 
 bool FileName::isSymLink() const
 {
-       return QFileInfo(toqstr(name_)).isSymLink();
+       return d->fi.isSymLink();
 }
 
 
 bool FileName::isFileEmpty() const
 {
-       return QFileInfo(toqstr(name_)).size() == 0;
+       return d->fi.size() == 0;
 }
 
 
 bool FileName::isDirectory() const
 {
-       return QFileInfo(toqstr(name_)).isDir();
+       return d->fi.isDir();
 }
 
 
 bool FileName::isReadOnly() const
 {
-       QFileInfo const fi(toqstr(name_));
-       return fi.isReadable() && !fi.isWritable();
+       return d->fi.isReadable() && !d->fi.isWritable();
 }
 
 
 bool FileName::isReadableDirectory() const
 {
-       QFileInfo const fi(toqstr(name_));
-       return fi.isDir() && fi.isReadable();
+       return d->fi.isDir() && d->fi.isReadable();
 }
 
 
@@ -249,15 +200,13 @@ FileName FileName::onlyPath() const
 
 bool FileName::isReadableFile() const
 {
-       QFileInfo const fi(toqstr(name_));
-       return fi.isFile() && fi.isReadable();
+       return d->fi.isFile() && d->fi.isReadable();
 }
 
 
 bool FileName::isWritable() const
 {
-       QFileInfo const fi(toqstr(name_));
-       return fi.isWritable();
+       return d->fi.isWritable();
 }
 
 
@@ -283,7 +232,7 @@ FileName FileName::tempName(FileName const & dir, std::string const & mask)
 
 std::time_t FileName::lastModified() const
 {
-       return QFileInfo(toqstr(name_)).lastModified().toTime_t();
+       return d->fi.lastModified().toTime_t();
 }
 
 
@@ -315,7 +264,7 @@ static bool rmdir(QFileInfo const & fi)
 
 bool FileName::destroyDirectory() const
 {
-       bool const success = rmdir(QFileInfo(toqstr(name_)));
+       bool const success = rmdir(d->fi);
        if (!success)
                lyxerr << "Could not delete " << *this << "." << endl;
 
@@ -590,14 +539,14 @@ DocFileName::DocFileName(FileName const & abs_filename, bool save_abs)
 void DocFileName::set(string const & name, string const & buffer_path)
 {
        save_abs_path_ = absolutePath(name);
-       name_ = save_abs_path_ ? name : makeAbsPath(name, buffer_path).absFilename();
+       FileName::set(save_abs_path_ ? name : makeAbsPath(name, buffer_path).absFilename());
        zipped_valid_ = false;
 }
 
 
 void DocFileName::erase()
 {
-       name_.erase();
+       FileName::erase();
        zipped_valid_ = false;
 }
 
@@ -605,14 +554,13 @@ void DocFileName::erase()
 string const DocFileName::relFilename(string const & path) const
 {
        // FIXME UNICODE
-       return to_utf8(makeRelPath(from_utf8(name_), from_utf8(path)));
+       return to_utf8(makeRelPath(qstring_to_ucs4(d->fi.absoluteFilePath()), from_utf8(path)));
 }
 
 
 string const DocFileName::outputFilename(string const & path) const
 {
-       // FIXME UNICODE
-       return save_abs_path_ ? name_ : to_utf8(makeRelPath(from_utf8(name_), from_utf8(path)));
+       return save_abs_path_ ? absFilename() : relFilename(path);
 }
 
 
@@ -622,14 +570,15 @@ string const DocFileName::mangledFilename(std::string const & dir) const
        // filename returns the same mangled name.
        typedef map<string, string> MangledMap;
        static MangledMap mangledNames;
-       MangledMap::const_iterator const it = mangledNames.find(name_);
+       MangledMap::const_iterator const it = mangledNames.find(absFilename());
        if (it != mangledNames.end())
                return (*it).second;
 
+       string const name = absFilename();
        // Now the real work
-       string mname = os::internal_path(name_);
+       string mname = os::internal_path(name);
        // Remove the extension.
-       mname = changeExtension(name_, string());
+       mname = changeExtension(name, string());
        // The mangled name must be a valid LaTeX name.
        // The list of characters to keep is probably over-restrictive,
        // but it is not really a problem.
@@ -644,7 +593,7 @@ string const DocFileName::mangledFilename(std::string const & dir) const
        while ((pos = mname.find_first_not_of(keep, pos)) != string::npos)
                mname[pos++] = '_';
        // Add the extension back on
-       mname = changeExtension(mname, getExtension(name_));
+       mname = changeExtension(mname, getExtension(name));
 
        // Prepend a counter to the filename. This is necessary to make
        // the mangled name unique.
@@ -672,7 +621,7 @@ string const DocFileName::mangledFilename(std::string const & dir) const
                }
        }
 
-       mangledNames[name_] = mname;
+       mangledNames[absFilename()] = mname;
        return mname;
 }
 
@@ -689,7 +638,7 @@ bool DocFileName::isZipped() const
 
 string const DocFileName::unzippedFilename() const
 {
-       return unzippedFileName(name_);
+       return unzippedFileName(absFilename());
 }
 
 
@@ -707,6 +656,3 @@ bool operator!=(DocFileName const & lhs, DocFileName const & rhs)
 
 } // namespace support
 } // namespace lyx
-
-
-
index 7ff353ee7677207cbbf0cc8c8c7fae6600205fe4..dfda706f2397461abf684f739652d3ec5bc165ee 100644 (file)
@@ -30,7 +30,7 @@ namespace support {
 class FileName {
 public:
        /// Constructor for empty filenames
-       FileName() {}
+       FileName();
        /** Constructor for nonempty filenames.
         * explicit because we don't want implicit conversion of relative
         * paths in function arguments (e.g. of unlink).
@@ -38,6 +38,13 @@ public:
         * Encoding is always UTF-8.
         */
        explicit FileName(std::string const & abs_filename);
+
+       /// copy constructor.
+       FileName(FileName const &);
+
+       ///
+       FileName & operator=(FileName const &);
+
        virtual ~FileName() {}
        /** Set a new filename.
         * \param filename the file in question. Must have an absolute path.
@@ -46,9 +53,9 @@ public:
        virtual void set(std::string const & filename);
        virtual void erase();
        /// Is this filename empty?
-       bool empty() const { return name_.empty(); }
+       bool empty() const;
        /// get the absolute file name in UTF-8 encoding
-       std::string const absFilename() const { return name_; }
+       std::string absFilename() const;
        /**
         * Get the file name in the encoding used by the file system.
         * Only use this for accessing the file, e.g. with an fstream.
@@ -77,7 +84,7 @@ public:
        bool isDirWritable() const;
        
        /// return true when file/directory is writable (write test file)
-       bool copyTo(FileName const & target, bool noclobber) const;
+       bool copyTo(FileName const & target) const;
 
        /// remove directory and all contents, returns true on success
        bool destroyDirectory() const;
@@ -115,10 +122,11 @@ public:
        /// used for display in the Gui
        docstring displayName(int threshold = 1000) const;
 
-
-protected:
-       /// The absolute file name in UTF-8 encoding.
-       std::string name_;
+private:
+       friend class DocFileName;
+       ///
+       struct Private;
+       Private * const d;
 };