delta/qt-creator.git/src/plugins/cppeditor/cppquickfixes.cpp, branch wip/clang code.qt.io: qt-creator/qt-creator.git CppEditor: Cleanup quickfixes 2013-12-03T18:20:24+00:00 Orgad Shaneh orgad.shaneh@audiocodes.com 2013-11-11T20:11:14+00:00 a06c5ef1515822b5693e2be24060a77a049cf465 Change-Id: I19fb785372291f66b756cf5be1fc941870c304c3 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com> 
Change-Id: I19fb785372291f66b756cf5be1fc941870c304c3
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com>
CppEditor: Fix stack overflow in quick fixes 2013-11-27T16:08:34+00:00 Nikolai Kosjar nikolai.kosjar@digia.com 2013-11-27T14:53:03+00:00 323be40b3e14a77d39856674a13bd1ecadacd04b The MoveDeclarationOutOfIf quick fix modified a IfStatementAST in case there were several such ASTs in interface->path(). The resulting AST was invalid (else_statement was pointing to 'this' afterwards), thus the afterwards executed PointerDeclarationFormatter (an ASTVisitor) was not able to finish his visit. The actual problem was that op->pattern in the match() calls was not reset. Task-number: QTCREATORBUG-10919 Change-Id: Ifbcac73e75690321ef45b6d8c2dc32fcea236efa Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com> 
The MoveDeclarationOutOfIf quick fix modified a IfStatementAST in case
there were several such ASTs in interface->path(). The resulting AST was
invalid (else_statement was pointing to 'this' afterwards), thus the
afterwards executed PointerDeclarationFormatter (an ASTVisitor) was not
able to finish his visit.

The actual problem was that op->pattern in the match() calls was not
reset.

Task-number: QTCREATORBUG-10919
Change-Id: Ifbcac73e75690321ef45b6d8c2dc32fcea236efa
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>
CppEditor: Prioritize inserting function def. in source file 2013-11-12T15:42:06+00:00 Nikolai Kosjar nikolai.kosjar@digia.com 2013-11-07T13:11:17+00:00 03270ad3012d8136c8fb703f29c692fd2d024359 For the quick fix InsertDefOperation ("Add Definition ...") the order of the offered operations is reversed to better support the typical use case of inserting the definition in the corresponding source file. Change-Id: I0cc502663c43fdeb4773a99d2cd5ae8ddf62f9f0 Reviewed-by: Christian Kandeler <christian.kandeler@digia.com> Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com> 
For the quick fix InsertDefOperation ("Add Definition ...") the order of
the offered operations is reversed to better support the typical use
case of inserting the definition in the corresponding source file.

Change-Id: I0cc502663c43fdeb4773a99d2cd5ae8ddf62f9f0
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>
CppEditor: Fix crash in CompleteSwitchCaseStatement 2013-10-17T10:04:42+00:00 Nikolai Kosjar nikolai.kosjar@digia.com 2013-10-14T13:47:07+00:00 d808ebed888891d41173876122f2178023880580 Task-number: QTCREATORBUG-10366 Change-Id: I6d5af5e7a59f3867141c8d7f098128d3db532ee5 Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com> 
Task-number: QTCREATORBUG-10366

Change-Id: I6d5af5e7a59f3867141c8d7f098128d3db532ee5
Reviewed-by: Erik Verbruggen <erik.verbruggen@digia.com>
Rename "[Mm]ethod(s)" to "[Ff]unction(s)" 2013-10-10T13:56:12+00:00 Nikolai Kosjar nikolai.kosjar@digia.com 2013-10-07T11:34:40+00:00 b8dbac0b9c4c45ee0d17d16eeae9086a14e9656a 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> 
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>
CppEditor: Fix insert position in MoveFuncDefToDecl 2013-10-07T09:23:41+00:00 Lorenz Haas lykurg@gmail.com 2013-10-05T19:15:43+00:00 e3d95168a08c36a8cca4f08d906d3a6162eff9f5 When a class was directly assigned to a variable the definition was misplaced right after the variable. Task-number: QTCREATORBUG-10303 Change-Id: I2cdfee784b085d856d7ff5ebe62bf791b9a6754e Reviewed-by: Orgad Shaneh <orgads@gmail.com> Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com> 
When a class was directly assigned to a variable the definition was
misplaced right after the variable.

Task-number: QTCREATORBUG-10303
Change-Id: I2cdfee784b085d856d7ff5ebe62bf791b9a6754e
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com>
CppEditor: implement ExtractLiteralAsParameter quickfix 2013-10-01T13:12:11+00:00 Joerg Bornemann joerg.bornemann@digia.com 2013-09-11T10:00:07+00:00 a6c8158484931c41e026b265297a1f2ae8a6e2c7 Task-number: QTCREATORBUG-9617 Change-Id: I6c6313746b837775bab665bb7019a2adf0b0f286 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com> 
Task-number: QTCREATORBUG-9617

Change-Id: I6c6313746b837775bab665bb7019a2adf0b0f286
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com>
CppEditor: New quick fix "Optimize For Loop" 2013-09-30T16:18:34+00:00 Lorenz Haas lykurg@gmail.com 2013-05-23T18:59:02+00:00 02825b589443836eff985e61a03af8be28a2f17d Rewrites post increment/decrement operators (++ and --) as pre increment/decrement operators and moves non string/numeric literals and non id expressions from loops condition to loops initializer. Change-Id: Id95334b6df6fcaa9af436cc1d2d0982d38bf8fe2 Reviewed-by: Leena Miettinen <riitta-leena.miettinen@digia.com> Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com> 
Rewrites post increment/decrement operators (++ and --) as pre
increment/decrement operators and moves non string/numeric literals and
non id expressions from loops condition to loops initializer.

Change-Id: Id95334b6df6fcaa9af436cc1d2d0982d38bf8fe2
Reviewed-by: Leena Miettinen <riitta-leena.miettinen@digia.com>
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com>
Apply static pattern to TextEditorSetting 2013-09-20T08:28:22+00:00 hjk hjk121@nokiamail.com 2013-09-19T15:59:27+00:00 ca15d0aa95fec76ce41c72ed747894969c978986 Change-Id: I4e6f573d893c0aa2bb4ca9812fc8db2961dac172 Reviewed-by: Tobias Hunger <tobias.hunger@digia.com> 
Change-Id: I4e6f573d893c0aa2bb4ca9812fc8db2961dac172
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>
CppEditor: make isNamespaceFunction available for all C++ quickfixes 2013-09-12T14:42:31+00:00 Joerg Bornemann joerg.bornemann@digia.com 2013-09-12T11:40:17+00:00 fa491c882c8779b546693d0e4d620f8a898217ba This function is generally useful when dealing with free functions. Change-Id: I52c0057b587d81c4b0eddac24e4d7659f80cf840 Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com> 
This function is generally useful when dealing with free functions.

Change-Id: I52c0057b587d81c4b0eddac24e4d7659f80cf840
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com>