Make the signal connection and disconnection more readable
Details
Details
Diff Detail
Diff Detail
- Repository
- R261 KMyMoney
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Beside trhee comments, it looks good to me.
kmymoney/kmymoney.cpp | ||
---|---|---|
465–502 | Do you need to see that "file" is of MyMoneyFile kind? const auto file = MyMoneyFile::instance(); | |
467 | Why not? const auto accountsModel = file->accountsModel(); | |
508 | Same here, i.e q->disconnect(file, nullptr, file->accountsModel(), nullptr); |
Comment Actions
I will make the requested changes and commit. I still have to get used to auto :-)
kmymoney/kmymoney.cpp | ||
---|---|---|
508 | Nope, that won't work. MyMoneyFile::instance() != Models::instance() |