[Libreoffice-bugs] [Bug 124176] Use pragma once instead of include guards

bugzilla-daemon at bugs.documentfoundation.org bugzilla-daemon at bugs.documentfoundation.org
Tue Feb 11 19:53:47 UTC 2020


https://bugs.documentfoundation.org/show_bug.cgi?id=124176

--- Comment #70 from Mohamed Sameh <mohamedsameh1998 at gmail.com> ---
(In reply to Muhammet Kara from comment #68)
> (In reply to Mohamed Sameh from comment #67)
> > I would like to work on this issue. I have a question, Can I fix any
> > arbitrary header file or is there a list I am supposed to follow ?
> 
> You can fix any C++ header (.hxx) file. You can find them by using a command
> like this on linux terminal while in the core directory: git grep #ifndef
> 
> Please make sure your clone of the core git repo is up to date, and do not
> create a large patch. (Not exceeding 50 lines of change might be a good idea
> for quick reviews.)

I submitted a commit to gerrit and here it is
https://gerrit.libreoffice.org/c/core/+/88473
did I follow the steps correctly ? And Is there any thing else I should do now
to finish contributing to this easy hack ?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice-bugs/attachments/20200211/d800e70e/attachment.htm>


More information about the Libreoffice-bugs mailing list