Changeset View
Changeset View
Standalone View
Standalone View
src/options/ksaneoptcombo.cpp
| Context not available. | |||||
| 328 | if (state() == STATE_HIDDEN) { | 328 | if (state() == STATE_HIDDEN) { | ||
|---|---|---|---|---|---|
| 329 | return false; | 329 | return false; | ||
| 330 | } | 330 | } | ||
| 331 | if (val == m_currentText) { | | |||
| 332 | return true; | | |||
| 333 | } | | |||
| 334 | 331 | | |||
| 335 | unsigned char data[4]; | 332 | unsigned char data[4]; | ||
| 336 | void* data_ptr = nullptr; | 333 | void* data_ptr = nullptr; | ||
| Context not available. | |||||
sars: maybe just remove the whole commented code and the comment. | |||||
This fix could have been a separate commit and review. (it has nothing to do with the reorganization of the restoration of the options) sars: This fix could have been a separate commit and review. (it has nothing to do with the… | |||||
Still holds (could be in the bug-fix commit), but we can let it in here anyways :D sars: Still holds (could be in the bug-fix commit), but we can let it in here anyways :D | |||||
The initial code that your comment referred to has already been moved to https://phabricator.kde.org/D25588 antonarnold: The initial code that your comment referred to has already been moved to https://phabricator. | |||||
maybe just remove the whole commented code and the comment.