Instead of negating the logic values in several places, just invert
the logic. Also move it out of class, as it does not access any
class members.
Details
Details
- Reviewers
ngraham - Group Reviewers
Baloo - Commits
- R119:31e823b6a67a: Invert logic of shouldShowMountPoint -> ignoredMountPoint
Diff Detail
Diff Detail
- Repository
- R119 Plasma Desktop
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
2 Question :
1- isnt that method should be const'ified ?
2- also its name will be much better isIgnoreableMountPoint (or something can be more understandable
that it will just return true or false ) becase right now it looks like
"ignore that mount point"