Fix line lengths

Alex
This commit is contained in:
Alex Neundorf 2011-08-10 20:45:22 +02:00
parent 83b730cd1a
commit 735a5bb321
1 changed files with 60 additions and 49 deletions

View File

@ -7,8 +7,6 @@
#include "cmQtAutomoc.h" #include "cmQtAutomoc.h"
#define TRACE_LINE() printf(" %s %d\n", __PRETTY_FUNCTION__, __LINE__)
cmQtAutomoc::cmQtAutomoc() cmQtAutomoc::cmQtAutomoc()
:Verbose(true) :Verbose(true)
,RunMocFailed(false) ,RunMocFailed(false)
@ -79,7 +77,8 @@ void cmQtAutomoc::SetupAutomocTarget(cmMakefile* makefile,
if ((skip==false) && (generated == false)) if ((skip==false) && (generated == false))
{ {
std::string ext = cmSystemTools::GetFilenameExtension(fileIt->c_str()); std::string ext = cmSystemTools::GetFilenameExtension(fileIt->c_str());
cmSystemTools::FileFormat fileType = cmSystemTools::GetFileFormat(ext.c_str()); cmSystemTools::FileFormat fileType = cmSystemTools::GetFileFormat(
ext.c_str());
if (fileType == cmSystemTools::CXX_FILE_FORMAT) if (fileType == cmSystemTools::CXX_FILE_FORMAT)
{ {
_moc_files += sepFiles; _moc_files += sepFiles;
@ -200,11 +199,13 @@ bool cmQtAutomoc::ReadAutomocInfoFile(cmMakefile* makefile,
this->QtMajorVersion = makefile->GetSafeDefinition("AM_QT_VERSION_MAJOR"); this->QtMajorVersion = makefile->GetSafeDefinition("AM_QT_VERSION_MAJOR");
this->Sources = makefile->GetSafeDefinition("AM_SOURCES"); this->Sources = makefile->GetSafeDefinition("AM_SOURCES");
this->Headers = makefile->GetSafeDefinition("AM_HEADERS"); this->Headers = makefile->GetSafeDefinition("AM_HEADERS");
this->IncludeProjectDirsBefore = makefile->IsOn("AM_CMAKE_INCLUDE_DIRECTORIES_PROJECT_BEFORE"); this->IncludeProjectDirsBefore = makefile->IsOn(
"AM_CMAKE_INCLUDE_DIRECTORIES_PROJECT_BEFORE");
this->Srcdir = makefile->GetSafeDefinition("AM_CMAKE_CURRENT_SOURCE_DIR"); this->Srcdir = makefile->GetSafeDefinition("AM_CMAKE_CURRENT_SOURCE_DIR");
this->Builddir = makefile->GetSafeDefinition("AM_CMAKE_BINARY_DIR"); this->Builddir = makefile->GetSafeDefinition("AM_CMAKE_BINARY_DIR");
this->MocExecutable = makefile->GetSafeDefinition("AM_QT_MOC_EXECUTABLE"); this->MocExecutable = makefile->GetSafeDefinition("AM_QT_MOC_EXECUTABLE");
this->MocCompileDefinitionsStr = makefile->GetSafeDefinition("AM_MOC_COMPILE_DEFINITIONS"); this->MocCompileDefinitionsStr = makefile->GetSafeDefinition(
"AM_MOC_COMPILE_DEFINITIONS");
this->MocDefinitionsStr = makefile->GetSafeDefinition("AM_MOC_DEFINITIONS"); this->MocDefinitionsStr = makefile->GetSafeDefinition("AM_MOC_DEFINITIONS");
this->MocIncludesStr = makefile->GetSafeDefinition("AM_MOC_INCLUDES"); this->MocIncludesStr = makefile->GetSafeDefinition("AM_MOC_INCLUDES");
this->ProjectBinaryDir = makefile->GetSafeDefinition("AM_CMAKE_BINARY_DIR"); this->ProjectBinaryDir = makefile->GetSafeDefinition("AM_CMAKE_BINARY_DIR");
@ -361,13 +362,18 @@ bool cmQtAutomoc::RunAutomocQt4()
this->GenerateAll = true; this->GenerateAll = true;
} }
// the program goes through all .cpp files to see which moc files are included. It is not really // the program goes through all .cpp files to see which moc files are
// interesting how the moc file is named, but what file the moc is created from. Once a moc is // included. It is not really interesting how the moc file is named, but
// included the same moc may not be included in the _automoc.cpp file anymore. OTOH if there's a // what file the moc is created from. Once a moc is included the same moc
// header containing Q_OBJECT where no corresponding moc file is included anywhere a // may not be included in the _automoc.cpp file anymore. OTOH if there's a
// moc_<filename>.cpp file is created and included in the _automoc.cpp file. // header containing Q_OBJECT where no corresponding moc file is included
std::map<std::string, std::string> includedMocs; // key = moc source filepath, value = moc output filepath // anywhere a moc_<filename>.cpp file is created and included in
std::map<std::string, std::string> notIncludedMocs; // key = moc source filepath, value = moc output filename // the _automoc.cpp file.
// key = moc source filepath, value = moc output filepath
std::map<std::string, std::string> includedMocs;
// key = moc source filepath, value = moc output filename
std::map<std::string, std::string> notIncludedMocs;
cmsys::RegularExpression mocIncludeRegExp( cmsys::RegularExpression mocIncludeRegExp(
"[\n][ \t]*#[ \t]*include[ \t]+" "[\n][ \t]*#[ \t]*include[ \t]+"
@ -422,8 +428,8 @@ bool cmQtAutomoc::RunAutomocQt4()
int matchOffset = 0; int matchOffset = 0;
if (!mocIncludeRegExp.find(contentsString.c_str())) if (!mocIncludeRegExp.find(contentsString.c_str()))
{ {
// no moc #include, look whether we need to create a moc from the .h nevertheless // no moc #include, look whether we need to create a moc from
//std::cout << "no moc #include in the .cpp file"; // the .h nevertheless
const std::string basename = const std::string basename =
cmsys::SystemTools::GetFilenameWithoutLastExtension(absFilename); cmsys::SystemTools::GetFilenameWithoutLastExtension(absFilename);
for(std::list<std::string>::const_iterator ext = for(std::list<std::string>::const_iterator ext =
@ -479,20 +485,19 @@ bool cmQtAutomoc::RunAutomocQt4()
GetFilenameWithoutLastExtension(currentMoc); GetFilenameWithoutLastExtension(currentMoc);
const bool moc_style = this->StartsWith(basename, "moc_"); const bool moc_style = this->StartsWith(basename, "moc_");
// If the moc include is of the moc_foo.cpp style we expect the Q_OBJECT class // If the moc include is of the moc_foo.cpp style we expect
// declaration in a header file. // the Q_OBJECT class declaration in a header file.
// If the moc include is of the foo.moc style we need to look for a Q_OBJECT // If the moc include is of the foo.moc style we need to look for
// macro in the current source file, if it contains the macro we generate the // a Q_OBJECT macro in the current source file, if it contains the
// moc file from the source file, else from the header. // macro we generate the moc file from the source file, else from the
// // header.
// TODO: currently any .moc file name will be used if the source contains
// Q_OBJECT // Q_OBJECT
if (moc_style || !qObjectRegExp.find(contentsString)) if (moc_style || !qObjectRegExp.find(contentsString))
{ {
if (moc_style) if (moc_style)
{ {
// basename should be the part of the moc filename used for finding the // basename should be the part of the moc filename used for
// correct header, so we need to remove the moc_ part // finding the correct header, so we need to remove the moc_ part
basename = basename.substr(4); basename = basename.substr(4);
} }
@ -513,7 +518,8 @@ bool cmQtAutomoc::RunAutomocQt4()
} }
if (!headerFound) if (!headerFound)
{ {
// the moc file is in a subdir => look for the header in the same subdir // the moc file is in a subdir => look for the header in the
// same subdir
if (currentMoc.find_first_of('/') != std::string::npos) if (currentMoc.find_first_of('/') != std::string::npos)
{ {
const std::string &filepath = absPath const std::string &filepath = absPath
@ -536,20 +542,23 @@ bool cmQtAutomoc::RunAutomocQt4()
} }
if (!headerFound) if (!headerFound)
{ {
std::cerr << "automoc4: The file \"" << absFilename << std::cerr << "automoc4: The file \"" << absFilename
"\" includes the moc file \"" << currentMoc << "\", but neither \"" << << "\" includes the moc file \"" << currentMoc
absPath + basename + '{' + this->Join(headerExtensions, ',') + "}\" nor \"" << << "\", but neither \"" << absPath << basename
filepath + '{' + this->Join(headerExtensions, ',') + '}' << << '{' << this->Join(headerExtensions, ',')
"\" exist." << std::endl; << "}\" nor \"" << filepath << '{'
<< this->Join(headerExtensions, ',') << '}'
<< "\" exist." << std::endl;
::exit(EXIT_FAILURE); ::exit(EXIT_FAILURE);
} }
} }
else else
{ {
std::cerr << "automoc4: The file \"" << absFilename << std::cerr << "automoc4: The file \"" << absFilename
"\" includes the moc file \"" << currentMoc << "\", but \"" << << "\" includes the moc file \"" << currentMoc
absPath + basename + '{' + this->Join(headerExtensions, ',') + '}' << << "\", but \"" << absPath << basename << '{'
"\" does not exist." << std::endl; << this->Join(headerExtensions, ',') << '}'
<< "\" does not exist." << std::endl;
::exit(EXIT_FAILURE); ::exit(EXIT_FAILURE);
} }
} }
@ -578,17 +587,20 @@ bool cmQtAutomoc::RunAutomocQt4()
if (includedMocs.find(absFilename) == includedMocs.end() if (includedMocs.find(absFilename) == includedMocs.end()
&& notIncludedMocs.find(absFilename) == notIncludedMocs.end()) && notIncludedMocs.find(absFilename) == notIncludedMocs.end())
{ {
// if this header is not getting processed yet and is explicitly mentioned for the // if this header is not getting processed yet and is explicitly
// automoc the moc is run unconditionally on the header and the resulting file is // mentioned for the automoc the moc is run unconditionally on the
// included in the _automoc.cpp file (unless there's a .cpp file later on that // header and the resulting file is included in the _automoc.cpp file
// includes the moc from this header) // (unless there's a .cpp file later on that includes the moc from
const std::string currentMoc = "moc_" + cmsys::SystemTools::GetFilenameWithoutLastExtension(absFilename) + ".cpp"; // this header)
const std::string currentMoc = "moc_" + cmsys::SystemTools::
GetFilenameWithoutLastExtension(absFilename) + ".cpp";
notIncludedMocs[absFilename] = currentMoc; notIncludedMocs[absFilename] = currentMoc;
} }
} }
// run moc on all the moc's that are #included in source files // run moc on all the moc's that are #included in source files
for(std::map<std::string, std::string>::const_iterator it = includedMocs.begin(); for(std::map<std::string, std::string>::const_iterator
it = includedMocs.begin();
it != includedMocs.end(); it != includedMocs.end();
++it) ++it)
{ {
@ -605,8 +617,10 @@ bool cmQtAutomoc::RunAutomocQt4()
} }
else else
{ {
// run moc on the remaining headers and include them in the _automoc.cpp file // run moc on the remaining headers and include them in
for(std::map<std::string, std::string>::const_iterator it = notIncludedMocs.begin(); // the _automoc.cpp file
for(std::map<std::string, std::string>::const_iterator
it = notIncludedMocs.begin();
it != notIncludedMocs.end(); it != notIncludedMocs.end();
++it) ++it)
{ {
@ -621,8 +635,6 @@ bool cmQtAutomoc::RunAutomocQt4()
if (this->RunMocFailed) if (this->RunMocFailed)
{ {
// if any moc process failed we don't want to touch the _automoc.cpp file so that
// automoc4 is rerun until the issue is fixed
std::cerr << "returning failed.."<< std::endl; std::cerr << "returning failed.."<< std::endl;
return false; return false;
} }
@ -638,7 +650,6 @@ bool cmQtAutomoc::RunAutomocQt4()
return true; return true;
} }
} }
// either the contents of the _automoc.cpp file or one of the mocs included by it have changed
// source file that includes all remaining moc files (_automoc.cpp file) // source file that includes all remaining moc files (_automoc.cpp file)
std::fstream outfile; std::fstream outfile;
@ -654,7 +665,6 @@ bool cmQtAutomoc::RunAutomocQt4()
bool cmQtAutomoc::GenerateMoc(const std::string& sourceFile, bool cmQtAutomoc::GenerateMoc(const std::string& sourceFile,
const std::string& mocFileName) const std::string& mocFileName)
{ {
//std::cout << "AutoMoc::generateMoc" << sourceFile << mocFileName << std::endl;
const std::string mocFilePath = this->Builddir + mocFileName; const std::string mocFilePath = this->Builddir + mocFileName;
int sourceNewerThanMoc = 0; int sourceNewerThanMoc = 0;
bool success = cmsys::SystemTools::FileTimeCompare(sourceFile.c_str(), bool success = cmsys::SystemTools::FileTimeCompare(sourceFile.c_str(),
@ -710,10 +720,11 @@ bool cmQtAutomoc::GenerateMoc(const std::string& sourceFile,
std::string output; std::string output;
int retVal = 0; int retVal = 0;
const bool result = cmSystemTools::RunSingleCommand(command, &output, &retVal); bool result = cmSystemTools::RunSingleCommand(command, &output, &retVal);
if (!result || retVal) if (!result || retVal)
{ {
std::cerr << "automoc4: process for " << mocFilePath << " failed:\n" << output << std::endl; std::cerr << "automoc4: process for " << mocFilePath << " failed:\n"
<< output << std::endl;
this->RunMocFailed = true; this->RunMocFailed = true;
cmSystemTools::RemoveFile(mocFilePath.c_str()); cmSystemTools::RemoveFile(mocFilePath.c_str());
} }