remove last traces of js support
authorOswald Buddenhagen <oswald.buddenhagen@nokia.com>
Wed, 18 Apr 2012 10:09:48 +0000 (12:09 +0200)
committerOswald Buddenhagen <oswald.buddenhagen@nokia.com>
Tue, 19 Jun 2012 14:40:01 +0000 (16:40 +0200)
don't even complain about trying to use js-based feature files. we have
been doing that for long enough.

Change-Id: Ib58ec204322442c488e8d780989f26b1e32595ed
Reviewed-by: Joerg Bornemann <joerg.bornemann@nokia.com>
qmake/option.cpp
qmake/option.h
qmake/project.cpp

index a22d184..42bc7ae 100644 (file)
@@ -54,7 +54,6 @@ QT_BEGIN_NAMESPACE
 //convenience
 const char *Option::application_argv0 = 0;
 QString Option::prf_ext;
-QString Option::js_ext;
 QString Option::prl_ext;
 QString Option::libtool_ext;
 QString Option::pkgcfg_ext;
@@ -422,7 +421,6 @@ Option::init(int argc, char **argv)
     Option::libtool_ext = ".la";
     Option::pkgcfg_ext = ".pc";
     Option::prf_ext = ".prf";
-    Option::js_ext = ".js";
     Option::ui_ext = ".ui";
     Option::h_ext << ".h" << ".hpp" << ".hh" << ".hxx";
     Option::c_ext << ".c";
index 3aa7d6d..8c273e6 100644 (file)
@@ -72,7 +72,6 @@ void warn_msg(QMakeWarn t, const char *fmt, ...);
 struct Option
 {
     //simply global convenience
-    static QString js_ext;
     static QString libtool_ext;
     static QString pkgcfg_ext;
     static QString prf_ext;
index 768a1c2..b6d2143 100644 (file)
@@ -1690,7 +1690,6 @@ QMakeProject::doProjectTest(QString func, const QString &params,
 QMakeProject::IncludeStatus
 QMakeProject::doProjectInclude(QString file, uchar flags, QHash<QString, QStringList> &place)
 {
-    enum { UnknownFormat, ProFormat, JSFormat } format = UnknownFormat;
     if(flags & IncludeFlagFeature) {
         if(!file.endsWith(Option::prf_ext))
             file += Option::prf_ext;
@@ -1722,18 +1721,13 @@ QMakeProject::doProjectInclude(QString file, uchar flags, QHash<QString, QString
             }
             for(int root = start_root; root < feature_roots->size(); ++root) {
                 QString prf(feature_roots->at(root) + QLatin1Char('/') + file);
-                if(QFile::exists(prf + Option::js_ext)) {
-                    format = JSFormat;
-                    file = prf + Option::js_ext;
-                    break;
-                } else if(QFile::exists(prf)) {
-                    format = ProFormat;
+                if (QFile::exists(prf)) {
                     file = prf;
-                    break;
+                    goto foundf;
                 }
             }
-            if(format == UnknownFormat)
-                return IncludeNoExist;
+            return IncludeNoExist;
+          foundf: ;
         }
         if(place["QMAKE_INTERNAL_INCLUDED_FEATURES"].indexOf(file) != -1)
             return IncludeFeatureAlreadyLoaded;
@@ -1751,19 +1745,13 @@ QMakeProject::doProjectInclude(QString file, uchar flags, QHash<QString, QString
             testName += file;
             if(QFile::exists(testName)) {
                 file = testName;
-                break;
+                goto foundi;
             }
         }
-    }
-    if(format == UnknownFormat) {
-        if(QFile::exists(file)) {
-            if(file.endsWith(Option::js_ext))
-                format = JSFormat;
-            else
-                format = ProFormat;
-        } else {
-            return IncludeNoExist;
-        }
+        return IncludeNoExist;
+      foundi: ;
+    } else if (!QFile::exists(file)) {
+        return IncludeNoExist;
     }
     if(Option::mkfile::do_preprocess) //nice to see this first..
         fprintf(stderr, "#switching file %s(%s) - %s:%d\n", (flags & IncludeFlagFeature) ? "load" : "include",
@@ -1783,10 +1771,7 @@ QMakeProject::doProjectInclude(QString file, uchar flags, QHash<QString, QString
     }
     bool parsed = false;
     parser_info pi = parser;
-    if(format == JSFormat) {
-        warn_msg(WarnParser, "%s:%d: QtScript support disabled for %s.",
-                 pi.file.toLatin1().constData(), pi.line_no, orig_file.toLatin1().constData());
-    } else {
+    {
         if(flags & (IncludeFlagNewProject|IncludeFlagNewParser)) {
             // The "project's variables" are used in other places (eg. export()) so it's not
             // possible to use "place" everywhere. Instead just set variables and grab them later