Skip to content

Commit

Permalink
fix strings from #included sources not being collected
Browse files Browse the repository at this point in the history
Task-number: QTBUG-27974
Change-Id: I30edae2db5449605984a118458b1ccce7ad67eb1
Reviewed-by: hjk <[email protected]>
Reviewed-by: Oswald Buddenhagen <[email protected]>
(cherry picked from qttools/bcaa8575352ad16e1f6020436e1a3f450eefd53e)
  • Loading branch information
ossilator authored and The Qt Project committed Feb 5, 2014
1 parent cfdeca8 commit 31c6cbb
Show file tree
Hide file tree
Showing 4 changed files with 53 additions and 0 deletions.
42 changes: 42 additions & 0 deletions tests/auto/linguist/lupdate/testdata/good/parsecpp/included.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
/****************************************************************************
**
** Copyright (C) 2013 Digia Plc and/or its subsidiary(-ies).
** Contact: http://www.qt-project.org/legal
**
** This file is part of the test suite of the Qt Toolkit.
**
** $QT_BEGIN_LICENSE:LGPL$
** Commercial License Usage
** Licensees holding valid commercial Qt licenses may use this file in
** accordance with the commercial license agreement provided with the
** Software or, alternatively, in accordance with the terms contained in
** a written agreement between you and Digia. For licensing terms and
** conditions see http://qt.digia.com/licensing. For further information
** use the contact form at http://qt.digia.com/contact-us.
**
** GNU Lesser General Public License Usage
** Alternatively, this file may be used under the terms of the GNU Lesser
** General Public License version 2.1 as published by the Free Software
** Foundation and appearing in the file LICENSE.LGPL included in the
** packaging of this file. Please review the following information to
** ensure the GNU Lesser General Public License version 2.1 requirements
** will be met: http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html.
**
** In addition, as a special exception, Digia gives you certain additional
** rights. These rights are described in the Digia Qt LGPL Exception
** version 1.1, included in the file LGPL_EXCEPTION.txt in this package.
**
** GNU General Public License Usage
** Alternatively, this file may be used under the terms of the GNU
** General Public License version 3.0 as published by the Free Software
** Foundation and appearing in the file LICENSE.GPL included in the
** packaging of this file. Please review the following information to
** ensure the GNU General Public License version 3.0 requirements will be
** met: http://www.gnu.org/copyleft/gpl.html.
**
**
** $QT_END_LICENSE$
**
****************************************************************************/

QObject::tr("message from #included .cpp file");
5 changes: 5 additions & 0 deletions tests/auto/linguist/lupdate/testdata/good/parsecpp/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -343,3 +343,8 @@ void YetMoreFun::funStuff()
}

}



// QTBUG-27974: strings from included sources are not collected
#include "included.cpp"
Original file line number Diff line number Diff line change
Expand Up @@ -228,6 +228,11 @@ backslashed \ stuff.</source>
<source>just QObject</source>
<translation type="unfinished"></translation>
</message>
<message>
<location filename="included.cpp" line="42"/>
<source>message from #included .cpp file</source>
<translation type="unfinished"></translation>
</message>
</context>
<context>
<name>QTranslator</name>
Expand Down
1 change: 1 addition & 0 deletions tools/linguist/lupdate/cpp.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1411,6 +1411,7 @@ void CppParser::processInclude(const QString &file, ConversionData &cd, const QS
parser.functionContextUnresolved = functionContextUnresolved;
parser.pendingContext = pendingContext;
parser.setInput(ts, cleanFile);
parser.setTranslator(tor);
QStringList stack = includeStack;
stack << cleanFile;
parser.parseInternal(cd, stack, inclusions);
Expand Down

0 comments on commit 31c6cbb

Please sign in to comment.