| apol | |
| kfunk |
| KDevelop |
The patch reorganize "Run" menu to more consistent view. In the revision D4686 Aleix Pol suggests to split source code and run-time analyzers. This patch makes first step to implement this idea - runtime-checkers (Valgrind and Heapcheck) now are moved to the "Run" menu (this fixes will be pushed to upstream after the revision acceptance).
Tested with master branch and valgrind/heaptrack plugins.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
lgtm in principle, but I find it odd that to is lower-cased, but With is uppercased
| debuggers/common/midebuggerplugin.cpp | ||
|---|---|---|
| 87 | hm should the To also be upper-cased? can you ask the HIG? | |
For actual readability, it would be much nicer to have both lowercased than uppercased.
The HIG agrees:
Do not capitalize: Articles: a, an, the Coordinating Conjunctions: and, but, or, for, nor, etc. Prepositions (fewer than five letters): on, at, to, from, by, etc.