[PATCH] Make a map ctrl-click toggle the selection state
Dirk Hohndel
dirk at hohndel.org
Fri Jun 7 21:39:34 PDT 2013
On Fri, 2013-06-07 at 19:28 -0700, Thiago Macieira wrote:
> On sexta-feira, 7 de junho de 2013 18.29.37, Linus Torvalds wrote:
> > + QFlags<QItemSelectionModel::SelectionFlag> flags;
>
> There's a typedef for this. We prefer QItemSelectionModel::SelectionFlags
> (note the plural).
>
Let's take a step back please. With 'SelectionFlags' this doesn't even
compile:
qt-ui/divelistview.cpp: In member function ‘void
DiveListView::selectDive(dive*, bool, bool)’:
qt-ui/divelistview.cpp:60:70: error: no match for ‘operator=’ in ‘flags
= (toggle ? (QItemSelectionModel::SelectionFlag)8u :
(QItemSelectionModel::SelectionFlag)2u)’
So I'm taking the patch as Linus suggested and I'll wait for Thiago to
send in a fix to make it work (tm).
/D
More information about the subsurface
mailing list