Details
manual testing on the context menu
Diff Detail
- Repository
- R241 KIO
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
| src/filewidgets/kfileplacesview.cpp | ||
|---|---|---|
| 811–831 | This whole change on the if structure there is unrelated to the rest of the patch so please remove it. Beside this seems to change the semantic, we never tested for result being nullptr or not before, we tested for the various actions to be there, you changed that logic completely that smells dangerous to me. | |
| src/filewidgets/kfileplacesview.h | ||
| 111–112 ↗ | (On Diff #23026) | They are never used as slots AFAICT so this should go away I think. |
| src/filewidgets/kfileplacesview.cpp | ||
|---|---|---|
| 811–831 | Indeed the structural change could be in an other patch. Not sure to see where it's dangerous tho, previous code testing the nullptr where doing it on a variable eg: empty trash, and only if result variable was equal to empty trash, which had to be differnt from nullptr it would go on, I think this endup being the same with the new change. If result is different from nullptr AND equal to empty trash is equivalent to result being equal to emptyTrash and emptyTrash being different to nulltpr. The code is not doing anything for result being equal to nullptr alone, so no new behavior AFAIK. Removing it, maybe in an other commit :-) | |