There is a wrapper file lotuswordpro/source/filter/lwpunoheader.hxx. All files which include lwpunoheader.hxx should directly include the corresponding header files.
1. git grep 'lwpunoheader.hxx'
2. Fix all files
3. Remove lwpunoheader.hxx
Please leave this EasyHack for real beginners!!! If you are on, feel free to catch it.
Created attachment 103989 [details]
Here's a patch that should fix this easyhack.
I'm a real beginner to open source contributing. I did what was said in the bug specification and my patch should fix this easyhack.
Hi Cyrille, this is a great start - thanks for the patch :-)
Worth cleaning up a few things before we commit this though, I think:
It is good to have sal/config.h as the first include if possible.
Please remove the header you made redundant =)
Please filter any interface headers that are not used; just experiment with removing them in each module and seeing if cd lotuswordpro ; make - continues to work.
When you've done that, doing a:
git commit -a
and attaching the result of git format-patch -1 as an here would make it much easier to merge (and includes the removal :-)
Many thanks for tackling this & looking forward to the next iteration.
All the best !
Created attachment 104675 [details]
New attachment that follows Michael's recommandations.
Sorry, I made a mistake with my post, so I needed to write a double post. I followed your recommandations Michael and I removed the redundant header.
I didn't find any unused interface headers.
Thank you for the advices, Michael.
thanks for your work! Please could you create the patch with:
git format-patch -1
I can't read the current format. Thanks!
Created attachment 107853 [details]
Remove th lwpunoheader.hxx wrapper.
This attachment should be compatible with libreoffice.
Hi Thomas, this latest attachment follows your advices. Sorry for the two posts, I forgot that submitting an attachment created a new post.
So - nice patch; but it -looks- like this bit is missing:
> Please filter any interface headers that are not used; just experiment
> with removing them in each module and seeing if cd lotuswordpro ;
> make - continues to work.
By which I mean; that (almost certainly) we don't need -all- of these includes in each file; have you experimented with removing some of these lines to see if it fails ? ultimately, its possible that those are needed in other headers there, but ... the purpose was not just to remove this header, but remove redundant includes =)
Can you experiment with that ?
Created attachment 108025 [details]
Oh, I understand, now it should be correct.
Lovely; much better - see what a nice cleanup it is now =)
I've pushed to master - thanks for your contribution !
Please can you send a license statement to the mailing list of this form:
Which helps our book-keeping; thanks !
Migrating Whiteboard tags to Keywords: (EasyHack DifficultyBeginner SkillCpp TopicCleanup)