| commit | fa4662725cab8713168eaf48e1dcb6f2a63e07b8 | |
| author | Volker Krause <vkrause@kde.org> | |
| Sat, 11 Jun 2011 16:39:28 +0000 (11 18:39 +0200) | ||
| committer | Volker Krause <vkrause@kde.org> | |
| Sat, 11 Jun 2011 16:39:28 +0000 (11 18:39 +0200) | ||
| tree | 785b04c557f51e45a491b1c4f35951f47f023c44 | treesnapshot (tar.gz zip) |
| parent | 35fa2b529aad1baafc7b68c8f53f73e1ef76d234 | commitdiff |
Don't store the sender/receiver twice.
Not as bad as it looks, due to implicit sharing of QString's, but still
unnecessary. Instead use a bool to decide between sender and receiver. This
is of course global and not something that needs to be stored per item, but
it's much easier this way (and less risky to change for now). Since we
still have a few bits left in the ItemPrivate memory layout it doesn't cost
us anything.
Not as bad as it looks, due to implicit sharing of QString's, but still
unnecessary. Instead use a bool to decide between sender and receiver. This
is of course global and not something that needs to be stored per item, but
it's much easier this way (and less risky to change for now). Since we
still have a few bits left in the ItemPrivate memory layout it doesn't cost
us anything.