aboutsummaryrefslogtreecommitdiffstats
path: root/src/plugins/projectexplorer/abstractprocessstep.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Utils: Extract a CommandLine structure from a QtcProcesshjk2019-05-291-3/+5
| | | | | | | | | | | | We regularly pass around strings or filenames or pairs of strings or filenames and stringlist etc the in the end will be used as a kind of "command line", with quite a bit of ad-hoc user code and QtcProcess::addArg etc to set them up and manipulate them. Let's have a class for that concept. Change-Id: I288ab939d853b32c717135a65242c584c2beab50 Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* Utils: Rename FileName to FilePathhjk2019-05-281-1/+1
| | | | | | | | More in line with QFileInfo terminonlogy which appears to be best-of-breed within Qt. Change-Id: I1d051ff1c8363ebd4ee56376451df45216c4c9ab Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* ProjectExplorer: Use Utils::FileName in ProcessParametershjk2019-05-151-5/+5
| | | | | | | For the command and the working directory. Change-Id: Ia69dc7100aeb57bb6e1b35f4dd4f3cf3763d8cda Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* AbstractProcessStep: Re-use FileInProjectFinderChristian Kandeler2019-03-191-72/+15
| | | | | | Change-Id: Ifd5d24b6cac707372a95b2b31e270722a6202fc0 Reviewed-by: Antonio Di Monaco <tony@becrux.com> Reviewed-by: hjk <hjk@qt.io>
* Replace static_casts by QOverload where possiblehjk2019-02-261-1/+1
| | | | | | | | | Mainly to get rid of the QProcess::finished deprecation warning. Also adjust coding style in the surrounding connects when needed. Change-Id: I12f9b248c7974b892c4a069356e578e80f8c59e9 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* ProjectExplorer: Rework the build step run interfaceChristian Kandeler2019-01-311-12/+8
| | | | | | | | | | | | | | | | | | | | | Originally, the build manager used to run all build steps in a dedicated thread. Communication between the step and the manager happened via a QFutureInterface that was passed into the step's run() function. Later, new steps were added that operated asynchronously, so the build manager had to differentiate between the different kinds of steps for starting and stopping. These days, almost all build and deploy steps work asynchronously, which made the QFuture-based interface look increasingly odd. With this patch, all build steps are expected to work asynchronously, so the build manager no longer needs to differentiate. Steps are started and requested to stop via the run() and cancel() functions, respectively, and emit the finished() signal when they are done. Build step implementors no longer have to deal with a QFutureInterface. For steps whose implementation is inherently synchronous, the BuildStep base class offers a runInThread() function. Change-Id: If905c68b234c5a669f6e19f43142eaa57d594803 Reviewed-by: hjk <hjk@qt.io>
* ProjectExplorer: Sanitize AbstractProcessStep cancellationChristian Kandeler2019-01-281-16/+5
| | | | | | | | This build step runs in the main thread and thus can and should use the cancel() function, rather than employing a polling approach. Change-Id: Iba013f474db79820632e18224ad485bc1c4ec9b5 Reviewed-by: hjk <hjk@qt.io>
* ProjectExplorer: Simplify BuildStep::init() signaturehjk2019-01-181-3/+1
| | | | | | | | The extra parameter was always computed but used only in one place, and that use got removed lately. Change-Id: Ie10c0107ca70ee97ce03f83294992aab8d1a3ffe Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* OutputFormatter/AbstractProcessStep: Handle \r\n as newlineOrgad Shaneh2018-12-181-0/+2
| | | | | | | \r\n means newline. It shouldn't erase the previous line. Change-Id: I22d9919c9c0277e4c0932eba535dbfb84a1d5e71 Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* AbstractProcessStep: Support \r in outputOrgad Shaneh2018-11-191-9/+39
| | | | | | | Treat \r like \n. Pass it to the output formatter for processing. Change-Id: Icf27a9a555c6eabdd610bca4bb874bfcde4037b3 Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* ProjectExplorer: Pimpl AbstractProcessStepOrgad Shaneh2018-11-191-77/+112
| | | | | Change-Id: I0f77ff2e88b29674c306b394093deb2060db70c8 Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* ProjectExplorer: Use a data member for BuildStep::runInGuiThreadhjk2018-10-231-0/+1
| | | | | Change-Id: Ia219fcf595c05c6f1b82f420454bd906c6870ee7 Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
* Fix crash of process steps in deploy configurationEike Ziller2018-06-191-1/+5
| | | | | | | | | | | | | | | If the step is in a deploy configuration, it doesn't have a direct parent build configuration, so we need to get that from the active target. This is generally improved for BuildStep in master, but here we still have to do that by hand. Fix-up of 75afda1798ebea05be9be8c4c415dd910a8966c9 Task-number: QTCREATORBUG-20631 Change-Id: I22fa16d424ee5928c920663e9d01c791b964e873 Reviewed-by: Tobias Hunger <tobias.hunger@qt.io> Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
* ProjectExplorer: Fix switching build console to Utf8 for MSVCIvan Donchevskii2018-05-241-1/+5
| | | | | | | | | | | Wrap make command into the script to switch console code page to Utf8 before make (when the Kit check is on). Task-number: QTCREATORBUG-20327 Change-Id: Ie3e372e52a09b93a41c5ac7ad63b7b14384655fb Reviewed-by: André Hartmann <aha_1980@gmx.de> Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io> Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* Fix UI freeze while building with multiple cores / distributed buildAntonio Di Monaco2018-04-131-14/+51
| | | | | | | | | | | | | | | When parallelizing the build, and a verbose compiler error/warning is printed (i.e. a template overload error), the IDE wastes a lot of time parsing always the same file patterns, and that freezes completely the UI. The only workaround is to kill the build process using a terminal. Implement a file cache, thus mitigating the freeze issue. Change-Id: Ibcbdb6e6161af7cef424e90f7cfdc2fc34f6d7c1 Reviewed-by: Tobias Hunger <tobias.hunger@qt.io> Reviewed-by: Antonio Di Monaco <tony@becrux.com> Reviewed-by: Orgad Shaneh <orgads@gmail.com>
* ProjectExplorer/all: Re-organize BuildSteps/{Deploy,Build}Config setuphjk2017-12-081-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This follow the rough pattern of recent *RunConfigurationFactory changes for build and deploy configurations. - Collapse the two lines of constructors similar to what 890c1906e6fb2ec did for RunConfigurations * Deploy* was purely mechanical * Build* ctors are split in connects() in the ctor body to create "empty shell for clone" etc and build step additions in initialize() functions which are only used in the create() case. -- Allows to collapse the shared 'ctor()' functions, too. - Move FooBuildConfigurationFactory::create() implementations to FooBuildConfiguration() constructor. That was a strange and unneeded ping-pong between factories and objects, and furthermore allows one level less of indirection (and for a later, left out here, some reduction of the FooBuildConfiguration interfaces that were only used to accommodate the *Factory::create() functions. - Most {Build,Deploy}Configuration{,Factory} classes had a canHandle(), but there wasn't one in the base classses. Have one there. - Most canHandle() functions were checking simple restrictions on e.g. project or target types, specify those by setters in the constructors instead and check them in the base canHandle() - clone() is generally replaced by a creation of a "shell object" and a fromMap(source->toMap()), implemented in the base, there are two cases left for Android and Qbs that needed(?) some extra polish - generally use canHandle() in base implementation, instead of doing that in all Derived::canFoo() - as a result, canCreate/create/canClone/clone reimplementations are not needed anymore, keep the base implementation for now (could be inlined into their only users later), but de-virtualize them. - Combine Ios{Preset,DSym}BuildStepFactory. There was only one 'dsym' build step they could create. - Split the 'mangled' id into the ProjectConfiguration subtype specific constant identifier, and a QString extraId() bit. Only maintain the mangled id in saved settings. - Make ProjectConfiguration::m_id a constant member, adapt all constructors of derived classe. Not done in this patch: - Finish possible cosmetic changes on top - Add a way to specify restrictions to supported Qt versions (used in Android/Ios), as the base implementation does not depend on the qtsupport plugin - Combine the QList<X> availableFoo() + createFoo(X) function pairs to somthing like a direct QList<struct { X; std::function<X()>; }> fooCreators() to avoid e.g. the baseId.withSuffix() <-> id.suffixAfter(base) pingpong - Remove the *Factories from the global object pool - Do something about priority(). Falling back to plain qmake in android+qmake setup is not helpful. Change-Id: I2be7d88d554c5aa8b7db8edf5b93278e1ae0112a Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* Project: Make Project::files return a FileNameListTobias Hunger2017-12-081-2/+2
| | | | | Change-Id: I75ceb22ac65b8288d824f229d44089cba6fc8ea3 Reviewed-by: Eike Ziller <eike.ziller@qt.io>
* ProcessExplorer: Remove QLatin1String from AbstractProcessStepTobias Hunger2017-06-131-1/+1
| | | | | Change-Id: I1eed445b2e0400272ded021123420eee279ea1df Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
* ProjectExplorer: Do not try to resolve file name for internalsOrgad Shaneh2017-05-171-1/+1
| | | | | | | like <command line> or <built-in>. Change-Id: I7d0dc63b2ce54f8d92cba248d504d8124a3689a7 Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* AbstractProcessStep: Free output parser chain in error caseTobias Hunger2017-04-251-0/+1
| | | | | Change-Id: Ie0dc094068bfa56f728d84ee13e70efbc980f8b3 Reviewed-by: hjk <hjk@qt.io>
* AbstractProcessStep: Make init fail if a process is still runningTobias Hunger2017-04-251-1/+1
| | | | | | | | This should not happen, but better save than sorry. Task-number: QTCREATORBUG-17984 Change-Id: I3efa8fb38a915ae29366f3ab715077b762d53828 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
* ProjectExplorer: fix crash while canceling a processTim Jenssen2017-02-101-1/+1
| | | | | | | | m_process was nullptr at this moment Change-Id: If990eb7034c3bacc1ace7fd607d0247aec8aeb68 Task-number: QTCREATORBUG-17698 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* ProjectExplorer: Polish AbstractProcessStepTobias Hunger2017-01-201-2/+1
| | | | | | | Make a bool const. Change-Id: Icfddb851ce11e1d2149349e239dcca4b3fcbea0b Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
* Give the values of BuildStep::OutputFormat better namesChristian Kandeler2017-01-191-9/+9
| | | | | | | | The old ones did not convey their meaning very well. In particular, NormalOutput and MessageOutput were easily confused. Change-Id: Ia0a8c1b1c366ab3f5c59f751b37b8b1f68f6831d Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
* AbstractProcessStep: Do not crash when cancelling a buildTobias Hunger2017-01-161-10/+20
| | | | | | Task-number: QTCREATORBUG-17591 Change-Id: I3d03b46ba1abbf11ba681d690647b3114fbb28a5 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
* AbstractProcessStep: Fix double-free of QFutureInterfaceTobias Hunger2016-12-081-4/+3
| | | | | | | Partial revert of 732b83297302cc9ca4afa7c6632adf8112b1aa19 Change-Id: I22e1af3071b7345eae6459cfdfe5e3699f4ee804 Reviewed-by: Christian Stenger <christian.stenger@qt.io>
* AbstractProcessStep: Clean up codeTobias Hunger2016-12-071-64/+36
| | | | | Change-Id: Iae3d1e7cbd02d34da5cebbf0291948dbdd2e95df Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
* ProjectExplorer: Simplify code in AbstractProcessStepTobias Hunger2016-12-071-6/+2
| | | | | Change-Id: Ief747c8f0332041f964106d3bc71919b9924efd1 Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
* BuildStep: Remove finished() signal and use FutureInterface to reportTobias Hunger2016-04-251-10/+8
| | | | | | | | | | Remove the finished() signal that is (sometimes) used to report that a buildstep is done and use the FutureInterface for that purpose consistently. Change-Id: Ibe5520b562b91f1a7f4fc73ee898b33b930029ec Reviewed-by: Tim Jenssen <tim.jenssen@theqtcompany.com> Reviewed-by: Christian Kandeler <christian.kandeler@theqtcompany.com>
* Projectexplorer: Modernize codebaseTobias Hunger2016-04-151-15/+9
| | | | | | Change-Id: I4793b58aa77bcd46af99f8b843c0f7439cdc9269 Reviewed-by: Tim Jenssen <tim.jenssen@theqtcompany.com> Reviewed-by: Vikas Pachdha <vikas.pachdha@theqtcompany.com>
* ProjectExplorer: Use Qt5-style connectsOrgad Shaneh2016-02-011-7/+7
| | | | | | | The heavy lifting was done by clazy. Change-Id: I619db09a79760186b72e7662490ed1205155c1a7 Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>
* Update License according to agreement with Free Qt FoundationTobias Hunger2016-01-191-17/+12
| | | | | | | * Update files in src/plugins Change-Id: Ia5d77fad7d19d4bb3498e78661982f68729adb22 Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com>
* BuildManager: Provide context of earlier build steps to initTobias Hunger2015-11-161-1/+2
| | | | | | | | This information can be used to get information from earlier steps, which e.g. queried for android devices to deploy to. Change-Id: Iefe1c9443915cb6211f86f98ff7aaf3cb75145ba Reviewed-by: Niels Weber <niels.weber@theqtcompany.com>
* Tasks: Make the linking of compile output to Tasks more robustDaniel Teske2015-04-211-2/+2
| | | | | | | | | | | | | | | | | | | | Clicking on error messages is supposed to jump to the editor. And "Show Output" on the task is supposed to select the error in the output. The old code just registered the task for the last line of output. This broke for every parser that allowed for error messages that spanned multiple lines. And was obviously also incorrect for tasks that weren't generated due to compile output. Fix both of those issues by giving the IOutputParsers more control on which lines are linked to a task. Task-number: QTCREATORBUG-14136 Change-Id: I095922c9875620dabfb7d406f6b152c8a9b25b62 Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com> Reviewed-by: Daniel Teske <daniel.teske@theqtcompany.com>
* Merge commit '3c85058694ee2e41658d17f524fb48f0b187d2fe'Eike Ziller2015-02-121-6/+6
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/libs/utils/tooltip/tipcontents.cpp src/libs/utils/tooltip/tipcontents.h src/plugins/android/androiddeployqtstep.cpp src/plugins/baremetal/baremetalconstants.h src/plugins/baremetal/baremetaldevice.cpp src/plugins/baremetal/baremetaldevice.h src/plugins/baremetal/baremetaldeviceconfigurationwidget.cpp src/plugins/baremetal/baremetaldeviceconfigurationwidget.h src/plugins/baremetal/baremetaldeviceconfigurationwizard.cpp src/plugins/baremetal/baremetaldeviceconfigurationwizardpages.cpp src/plugins/baremetal/baremetaldeviceconfigurationwizardpages.h src/plugins/baremetal/baremetalplugin.cpp src/plugins/baremetal/baremetalplugin.h src/plugins/baremetal/baremetalruncontrolfactory.cpp src/plugins/baremetal/baremetalruncontrolfactory.h src/plugins/cppeditor/cppcodemodelinspectordialog.cpp src/plugins/cppeditor/cppdoxygen_test.cpp src/plugins/cppeditor/cppdoxygen_test.h src/plugins/debugger/breakpointmarker.cpp src/plugins/debugger/debuggeritemmodel.cpp src/plugins/debugger/debuggeritemmodel.h src/plugins/debugger/loadcoredialog.cpp src/plugins/genericprojectmanager/cppmodelmanagerhelper.cpp src/plugins/projectexplorer/addnewmodel.cpp src/plugins/projectexplorer/addnewmodel.h src/plugins/projectexplorer/jsonwizard/jsonfieldpage.cpp src/plugins/qmlprofiler/abstracttimelinemodel.cpp src/plugins/qmlprofiler/abstracttimelinemodel.h src/plugins/qmlprofiler/notesmodel.cpp src/plugins/qmlprofiler/qml/CategoryLabel.qml src/plugins/qmlprofiler/qml/MainView.qml src/plugins/qmlprofiler/qml/Overview.js src/plugins/qmlprofiler/qml/Overview.qml src/plugins/qmlprofiler/qml/TimeDisplay.qml src/plugins/qmlprofiler/qml/TimeMarks.qml src/plugins/qmlprofiler/qmlprofilertimelinemodelproxy.cpp src/plugins/qmlprofiler/sortedtimelinemodel.cpp src/plugins/qmlprofiler/sortedtimelinemodel.h src/plugins/qmlprofiler/timelinemodelaggregator.cpp src/plugins/qmlprofiler/timelinemodelaggregator.h src/plugins/qmlprofiler/timelinerenderer.cpp src/plugins/qmlprofiler/timelinerenderer.h src/plugins/qmlprojectmanager/QmlProjectManager.json.in src/plugins/texteditor/findinfiles.cpp src/plugins/vcsbase/vcsconfigurationpage.cpp src/shared/qbs src/shared/scriptwrapper/interface_wrap_helpers.h src/shared/scriptwrapper/wrap_helpers.h tests/auto/qmlprofiler/abstracttimelinemodel/tst_abstracttimelinemodel.cpp tests/system/suite_debugger/tst_debug_empty_main/test.py tests/system/suite_debugger/tst_qml_js_console/test.py tests/system/suite_debugger/tst_qml_locals/test.py Change-Id: I67540b648f8b162496f4aa606b04d50c7c9125c6
| * Update LicenseEike Ziller2015-01-161-6/+6
| | | | | | | | | | Change-Id: I711d5fb475ef814a1dc9d2822740e827f3f67125 Reviewed-by: Alessandro Portale <alessandro.portale@digia.com>
* | Replace QFileInfo::fileName() with FileName::fileName()Orgad Shaneh2015-01-291-1/+1
|/ | | | | Change-Id: I4852ff215abf25649fc5eac1e922ae901839ca3d Reviewed-by: hjk <hjk@theqtcompany.com>
* Use QFileInfo::exist(f) instead of QFileInfo(f).exists() if possiblehjk2014-11-031-3/+1
| | | | | | | | Faster. Change-Id: I91aa67462e11ff3258600d7f158de79942d0dc81 Reviewed-by: Marc Reilly <marc.reilly@gmail.com> Reviewed-by: Christian Stenger <christian.stenger@digia.com>
* AbastractProcessStep; Check that creating the build directory workedDaniel Teske2014-10-221-2/+12
| | | | | | | Task-number: QTCREATORBUG-13217 Change-Id: I26ac5db79c0e1b6556d085648f9ebc71efae015d Reviewed-by: Tobias Hunger <tobias.hunger@theqtcompany.com> Reviewed-by: Christian Stenger <christian.stenger@digia.com>
* Merge "Merge remote-tracking branch 'origin/3.2'"Eike Ziller2014-10-141-7/+8
|\
| * License updateEike Ziller2014-10-091-7/+8
| | | | | | | | | | Change-Id: I3c22ef2685d7aa589f5d0ab74d693653a4c32082 Reviewed-by: Alessandro Portale <alessandro.portale@digia.com>
* | ProjectExplorer: Remove redundant namespace qualificationsOrgad Shaneh2014-10-141-10/+10
|/ | | | | Change-Id: Id9034e31f3f61c38894111951534dd0d59b746dc Reviewed-by: hjk <hjk121@nokiamail.com>
* Always pass Core::Id by value.Christian Kandeler2014-07-011-1/+1
| | | | | | | | | | | Currently we pass in some places by value, elsewhere by const ref and for some weird reason also by const value in a lot of places. The latter is particularly annoying, as it is also used in interfaces and therefore forces all implementors to do the same, since leaving the "const" off is causing compiler warnings with MSVC. Change-Id: I65b87dc3cce0986b8a55ff6119cb752361027803 Reviewed-by: hjk <hjk121@nokiamail.com>
* Unify "missing compiler" and "missing build configuration" tasksTobias Hunger2014-06-231-0/+6
| | | | | | | | | | Everybody used to do their own thing... Also unify the "Configuration is faulty" message we write into the Application output window. Change-Id: I0e5c4ec68155d66aa1d0ea53134b98917869e5c6 Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>
* Avoid more platform ifdefsEike Ziller2014-04-081-3/+2
| | | | | | Change-Id: I184a2dc4c20ffa17e258d049a6183f0ef1fbd8b6 Reviewed-by: hjk <hjk121@nokiamail.com> Reviewed-by: Daniel Teske <daniel.teske@digia.com>
* AbstractProcessStep: Check for existence of executableDaniel Teske2014-01-141-1/+9
| | | | | | Task-number: QTCREATORBUG-11250 Change-Id: Idba517091a633f8ff57df5a6f42fe0f03a35142d Reviewed-by: Robert Loehning <robert.loehning@digia.com>
* Incremented year in copyright infoRobert Loehning2014-01-081-1/+1
| | | | | | Change-Id: Ib5423fdd064e4546f848c0b640b0ed0514c26d3a Reviewed-by: Leena Miettinen <riitta-leena.miettinen@digia.com> Reviewed-by: Kai Koehne <kai.koehne@digia.com>
* Rename "[Mm]ethod(s)" to "[Ff]unction(s)"Nikolai Kosjar2013-10-101-1/+1
| | | | | | | | | | | Only methods as programming functions are affected. Besides renaming some actions like "Switch Between Function Declaration/Definition" this mostly touches (api) code comments. This is a follow-up patch to commit 872bfb7. Change-Id: Icb65e8d73b59a022f8885b14df497169543a3b92 Reviewed-by: hjk <hjk121@nokiamail.com>
* Merge remote-tracking branch 'origin/2.8'Eike Ziller2013-09-271-20/+22
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: src/libs/extensionsystem/pluginmanager.cpp src/libs/extensionsystem/pluginspec.cpp src/plugins/coreplugin/basefilewizard.cpp src/plugins/coreplugin/coreplugin.pro src/plugins/coreplugin/coreplugin.qbs src/plugins/coreplugin/editormanager/ieditor.cpp src/plugins/coreplugin/externaltool.cpp src/plugins/coreplugin/icore.cpp src/plugins/cpptools/cppmodelmanager.cpp src/plugins/fakevim/fakevimhandler.cpp src/plugins/find/ifindfilter.cpp src/plugins/projectexplorer/buildstep.cpp src/plugins/projectexplorer/devicesupport/idevice.cpp src/plugins/projectexplorer/runconfiguration.cpp src/plugins/vcsbase/vcsbaseeditor.cpp tests/system/suite_debugger/tst_simple_analyze/test.py Change-Id: I11dc9e60bfc14bad4f8af747d041fc7678a07c17
| * Doc: edit projectexplorer API docsLeena Miettinen2013-09-111-20/+22
| | | | | | | | | | | | | | | | | | | | Fix grammar, punctuation, and style issues. Use standard wording for QDoc commands. Remove \brief commands from function descriptions. Move some function descriptions directly above the functions, so that the \fn command can be removed. Change-Id: Iedf4f0041af24541a982241f99bd4906e86af916 Reviewed-by: Daniel Teske <daniel.teske@digia.com>