The header: vcl/impdel.hxx - which is included in a ton of files looks strongly to me as if it should (instead) be internal to VCL: git grep DeletionListener git grep DeletionNotifier Appears to show no hits outside vcl/; so something like: git move include/vcl/impdel.hxx vcl/inc And then fixing all the internal vcl include paths to be just #include "impdel.hxx" would be great. Then dunging all the redundant include statements out of other modules would be great ! =) Thanks !
I'm happy to take this one. I'll try to give it to you in one patch :-) No more little patches I promise Michael!
On second thoughts, I might welsh on this one :-) Some other lucky person can have the pleasure! Sorry all.
Heh - my hope was that Nurhak might do this one =)
Now, I'm on that bug thanks for orientation =)
JanI is default CC for Easy Hacks (Add Jan; remove LibreOffice Dev List from CC) [NinjaEdit]
Jochen Nitschke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=ef9757605619c6bcbdde61c93c834a0467453d67 tdf#97525 vcl: make DeletionListener internal It will be available in 5.2.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
closing header is moved and includes are adjusted