Commit d40d5933 authored by Boudewijn Rempt's avatar Boudewijn Rempt

update hacking file to note the deprecated QSignalMapper. Sigh.

parent b180f3c7
......@@ -17,12 +17,13 @@ Qt vs STD vs Boost:
C++11 and C++14
Yes, but. Avoid lambdas. Avoid the new sig/slot connection syntax _unless_
you are porting all of Krita to the new syntax. Sure, it has some advantages,
but having two different ways of doing the same thing is begging for trouble
and comprehension problems. For now, keep using Q_FOREACH, we're using it all
over the place. auto is fine, when using in for loops. Don't go overboard
using it in other places.
Yes, but. Avoid lambdas (except when replacing the use of
QSignalMapper. Sigh.). Avoid the new sig/slot connection syntax
_unless_ you are porting all of Krita to the new syntax. Sure, it
has some advantages, but having two different ways of doing the same
thing is begging for trouble and comprehension problems. For now,
keep using Q_FOREACH, we're using it all over the place. auto is fine,
when using in for loops. Don't go use it in other places.
Before using other new features, discuss on #krita so we can expand this list.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment