By doing this DatabaseSanitizer::getDocuments() can be removed from the public API and also q-pointer is no longer necessary
Details
Details
- Reviewers
kossebau - Group Reviewers
Baloo Frameworks - Maniphest Tasks
- T8250: Sanitize the database
- Commits
- R293:5e471c3d39b3: sanitizer: Move implementation into Baloo namespace
Diff Detail
Diff Detail
- Repository
- R293 Baloo
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Only looked at the diff, but nothing suspicious seen. If it builds and works, should be fine :)
Perhaps mention in the commit message also that this moving into the namespace allows to remove DatabaseSanitizer::getDocuments() from the public API, so it gets more clear why this change was done and why there are also those changes in the patch. The reader of this commit in 2 years might be abit thankful :)