namespace lyx {
namespace support {
-bool isLyXFilename(string const & filename)
+bool isLyXFileName(string const & filename)
{
return suffixIs(ascii_lowercase(filename), ".lyx");
}
-bool isSGMLFilename(string const & filename)
+bool isSGMLFileName(string const & filename)
{
return suffixIs(ascii_lowercase(filename), ".sgml");
}
-bool isValidLaTeXFilename(string const & filename)
+bool isValidLaTeXFileName(string const & filename)
{
string const invalid_chars("#$%{}()[]\"^");
return filename.find_first_of(invalid_chars) == string::npos;
FileName const makeLatexName(FileName const & file)
{
string name = file.onlyFileName();
- string const path = file.onlyPath().absFilename() + "/";
+ string const path = file.onlyPath().absFileName() + "/";
// ok so we scan through the string twice, but who cares.
// FIXME: in Unicode time this will break for sure! There is
{
switch(style) {
case quote_shell:
- // This does not work for filenames containing " (windows)
- // or ' (all other OSes). This can't be changed easily, since
- // we would need to adapt the command line parser in
- // Forkedcall::generateChild. Therefore we don't pass user
- // filenames to child processes if possible. We store them in
- // a python script instead, where we don't have these
- // limitations.
+ // This does not work on native Windows for filenames
+ // containing the following characters < > : " / \ | ? *
+ // Moreover, it can't be made to work, as, according to
+ // http://msdn.microsoft.com/en-us/library/aa365247(VS.85).aspx
+ // those are reserved characters, and thus are forbidden.
+ // Please, also note that the command-line parser in
+ // ForkedCall::generateChild cannot deal with filenames
+ // containing " or ', therefore we don't pass user filenames
+ // to child processes if possible. We store them in a python
+ // script instead, where we don't have these limitations.
#ifndef USE_QPROCESS
return (os::shell() == os::UNIX) ?
- '\'' + name + '\'':
+ '\'' + subst(name, "'", "\'\\\'\'") + '\'' :
'"' + name + '"';
#else
- return '"' + name + '"';
+ // According to the QProcess parser, a single double
+ // quote is represented by three consecutive ones.
+ // Here we simply escape the double quote and let our
+ // simple parser in Systemcall.cpp do the substitution.
+ return '"' + subst(name, "\"", "\\\"") + '"';
#endif
case quote_python:
return "\"" + subst(subst(name, "\\", "\\\\"), "\"", "\\\"")
if (!suffixIs(path_element, '/'))
path_element += '/';
path_element = subst(path_element, "$$LyX",
- package().system_support().absFilename());
+ package().system_support().absFileName());
path_element = subst(path_element, "$$User",
- package().user_support().absFilename());
+ package().user_support().absFileName());
real_file = fileSearch(path_element, name, ext);
return mode == may_not_exist ? fullname : FileName();
// Only add the extension if it is not already the extension of
// fullname.
- if (getExtension(fullname.absFilename()) != ext)
- fullname = FileName(addExtension(fullname.absFilename(), ext));
+ if (getExtension(fullname.absFileName()) != ext)
+ fullname = FileName(addExtension(fullname.absFileName(), ext));
if (fullname.isReadableFile() || mode == may_not_exist)
return fullname;
return FileName();
FileName const libFileSearch(string const & dir, string const & name,
string const & ext)
{
- FileName fullname = fileSearch(addPath(package().user_support().absFilename(), dir),
+ FileName fullname = fileSearch(addPath(package().user_support().absFileName(), dir),
name, ext);
if (!fullname.empty())
return fullname;
if (!package().build_support().empty())
- fullname = fileSearch(addPath(package().build_support().absFilename(), dir),
+ fullname = fileSearch(addPath(package().build_support().absFileName(), dir),
name, ext);
if (!fullname.empty())
return fullname;
- return fileSearch(addPath(package().system_support().absFilename(), dir), name, ext);
+ return fileSearch(addPath(package().system_support().absFileName(), dir), name, ext);
}
// Does this script file exist?
string const script =
- libFileSearch(".", command.substr(start_script, size_script)).absFilename();
+ libFileSearch(".", command.substr(start_script, size_script)).absFileName();
if (script.empty()) {
// Replace "$$s/" with ""
if (FileName::isAbsolute(basePath))
tempBase = basePath;
else
- tempBase = addPath(FileName::getcwd().absFilename(), basePath);
+ tempBase = addPath(FileName::getcwd().absFileName(), basePath);
// Handle /./ at the end of the path
while (suffixIs(tempBase, "/./"))
// Split by first /
rTemp = split(rTemp, temp, '/');
if (temp == "~") {
- tempBase = package().home_dir().absFilename();
+ tempBase = package().home_dir().absFileName();
tempRel = rTemp;
}
// Chops any path of filename.
string const addName(string const & path, string const & fname)
{
- string const basename = onlyFilename(fname);
+ string const basename = onlyFileName(fname);
string buf;
if (path != "." && path != "./" && !path.empty()) {
// Strips path from filename
-string const onlyFilename(string const & fname)
+string const onlyFileName(string const & fname)
{
if (fname.empty())
return fname;
rTemp = split(rTemp, temp, '/');
if (temp == ".")
- return FileName::getcwd().absFilename() + '/' + rTemp;
+ return FileName::getcwd().absFileName() + '/' + rTemp;
if (temp == "~")
- return package().home_dir().absFilename() + '/' + rTemp;
+ return package().home_dir().absFileName() + '/' + rTemp;
if (temp == "..")
- return makeAbsPath(copy).absFilename();
+ return makeAbsPath(copy).absFileName();
// Don't know how to handle this
return copy;
string const ext = getExtension(zipped_file);
if (ext == "gz" || ext == "z" || ext == "Z")
return changeExtension(zipped_file, string());
- return onlyPath(zipped_file) + "unzipped_" + onlyFilename(zipped_file);
+ return onlyPath(zipped_file) + "unzipped_" + onlyFileName(zipped_file);
}
string str = path;
// If file is from LyXDir, display it as if it were relative.
- string const system = package().system_support().absFilename();
+ string const system = package().system_support().absFileName();
if (prefixIs(str, system) && str != system)
return from_utf8("[" + str.erase(0, system.length()) + "]");
// replace /home/blah with ~/
- string const home = package().home_dir().absFilename();
+ string const home = package().home_dir().absFileName();
if (!home.empty() && prefixIs(str, home))
str = subst(str, home, "~");
if (str.empty()) {
// Yes, filename itself is too long.
// Pick the start and the end of the filename.
- str = onlyFilename(path);
+ str = onlyFileName(path);
string const head = str.substr(0, threshold / 2 - 3);
string::size_type len = str.length();
bool readLink(FileName const & file, FileName & link)
{
#ifdef HAVE_READLINK
- char linkbuffer[512];
- // Should be PATH_MAX but that needs autconf support
+ char linkbuffer[PATH_MAX + 1];
string const encoded = file.toFilesystemEncoding();
int const nRead = ::readlink(encoded.c_str(),
linkbuffer, sizeof(linkbuffer) - 1);
if (nRead <= 0)
return false;
linkbuffer[nRead] = '\0'; // terminator
- link = makeAbsPath(linkbuffer, onlyPath(file.absFilename()));
+ link = makeAbsPath(linkbuffer, onlyPath(file.absFileName()));
return true;
#else
return false;