[prev in list] [next in list] [prev in thread] [next in thread]
List: haiku-bugs
Subject: [haiku-bugs] Re: [Haiku] #9460: Edit source code to remove compilation warnings, to allow -Werror (e
From: "Haiku" <trac () haiku-os ! org>
Date: 2023-12-31 13:34:07
Message-ID: 057.bbc321fe14eab6e2e73931b9ed69d595 () haiku-os ! org
[Download RAW message or body]
--===============5604143519825678474==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
#9460: Edit source code to remove compilation warnings, to allow -Werror (e=
asy)
----------------------------------+----------------------------
Reporter: mmadia | Owner: nobody
Type: enhancement | Status: new
Priority: normal | Milestone: Unscheduled
Component: - General | Version: R1/Development
Resolution: | Keywords: GCI
Blocked By: 12653, 12662, 12663 | Blocking:
Platform: All |
----------------------------------+----------------------------
Description changed by pulkomandy:
Old description:
> This is a generalized ticket to expose an area to contribute.
>
> Several portions of Haiku's code is not compiled with -Werror. Those
> portions can be found in [https://git.haiku-
> os.org/haiku/tree/build/jam/ArchitectureRules#n602
> build/jam/ArchitectureRules].
>
> Some of them only fail on GCC 2 vs. both GCC's. Take care to test with
> both compilers. As of hrev45297 these are those directories:
> {{{
> # EnableWerror src add-ons accelerants intel_extreme ;
> # EnableWerror src add-ons accelerants matrox ;
> # EnableWerror src add-ons accelerants nvidia ;
> # EnableWerror src add-ons accelerants radeon ;
> # EnableWerror src add-ons accelerants radeon_hd ;
> # EnableWerror src add-ons input_server filters ;
> # EnableWerror src add-ons input_server methods ;
> # EnableWerror src add-ons kernel bus_managers acpi ;
> # EnableWerror src add-ons kernel bus_managers firewire ;
> # EnableWerror src add-ons kernel bus_managers ps2 ; # gcc2
> # EnableWerror src add-ons kernel debugger ; # gcc2
> # EnableWerror src add-ons kernel drivers audio ;
> # EnableWerror src add-ons kernel drivers graphics ;
> # EnableWerror src add-ons kernel drivers input ;
> # EnableWerror src add-ons kernel drivers network ;
> # EnableWerror src add-ons kernel drivers power ;
> # EnableWerror src add-ons kernel file_systems ext2 ;
> # EnableWerror src add-ons kernel file_systems fat ;
> # EnableWerror src add-ons kernel file_systems googlefs ;
> # EnableWerror src add-ons kernel file_systems ntfs ;
> # EnableWerror src add-ons kernel file_systems reiserfs ;
> # EnableWerror src add-ons kernel network datalink_protocols ;
> # EnableWerror src add-ons kernel network stack ;
> # EnableWerror src add-ons media media-add-ons usb_webcam ;
> # EnableWerror src add-ons media media-add-ons video_producer_demo
> ;
> # EnableWerror src add-ons media plugins asf_reader ;
> # EnableWerror src add-ons media plugins avi_reader ;
> # EnableWerror src add-ons media plugins ffmpeg ;
> # EnableWerror src add-ons media plugins matroska ;
> # EnableWerror src add-ons media plugins mov_reader ;
> # EnableWerror src add-ons media plugins mp3_reader ;
> # EnableWerror src add-ons media plugins ogg ;
> # EnableWerror src add-ons media plugins raw_decoder ;
> # EnableWerror src add-ons media plugins speex ;
> # EnableWerror src add-ons media plugins wav_reader ;
> # EnableWerror src add-ons translators exr ;
> # EnableWerror src add-ons translators hpgs ;
> # EnableWerror src add-ons translators jpeg ; # gcc2
> # EnableWerror src add-ons translators png ; # gcc2
> # EnableWerror src add-ons translators stxt ;
> # EnableWerror src add-ons translators wonderbrush ;
> }}}
>
> The process would be:
>
> - Uncomment one of these lines in the ArchitectureRules file,
> - Build Haiku
> - Compilation will fail because of a compiler warning now being
> considered an error
> - Alter the code to fix the warning
> - Submit your change to Gerrit for code review
>
> Mention this ticket in the commit message of your changes (include a line
> saying "Fixes: #9640") to help us find what your work is related to.
>
> Eventually, all warnings will be fixed and we can enable Werror globally
> on all our sourcecode.
New description:
This is a generalized ticket to expose an area to contribute.
Several portions of Haiku's code is not compiled with -Werror. Those
portions can be found in [https://git.haiku-
os.org/haiku/tree/build/jam/ArchitectureRules#n602
build/jam/ArchitectureRules].
Some of them only fail on GCC 2 vs. both GCC's. Take care to test with
both compilers. As of hrev45297 these are those directories:
{{{
# EnableWerror src add-ons accelerants intel_extreme ;
# EnableWerror src add-ons accelerants matrox ;
# EnableWerror src add-ons accelerants nvidia ;
# EnableWerror src add-ons accelerants radeon ;
# EnableWerror src add-ons accelerants radeon_hd ;
# EnableWerror src add-ons input_server filters ;
# EnableWerror src add-ons input_server methods ;
# EnableWerror src add-ons kernel bus_managers acpi ;
# EnableWerror src add-ons kernel bus_managers firewire ;
# EnableWerror src add-ons kernel bus_managers ps2 ; # gcc2
# EnableWerror src add-ons kernel debugger ; # gcc2
# EnableWerror src add-ons kernel drivers audio ;
# EnableWerror src add-ons kernel drivers graphics ;
# EnableWerror src add-ons kernel drivers input ;
# EnableWerror src add-ons kernel drivers network ;
# EnableWerror src add-ons kernel drivers power ;
# EnableWerror src add-ons kernel file_systems ext2 ;
# EnableWerror src add-ons kernel file_systems fat ;
# EnableWerror src add-ons kernel file_systems googlefs ;
# EnableWerror src add-ons kernel file_systems ntfs ;
# EnableWerror src add-ons kernel file_systems reiserfs ;
# EnableWerror src add-ons kernel network datalink_protocols ;
# EnableWerror src add-ons kernel network stack ;
# EnableWerror src add-ons media media-add-ons usb_webcam ;
# EnableWerror src add-ons media media-add-ons video_producer_demo ;
# EnableWerror src add-ons media plugins asf_reader ;
# EnableWerror src add-ons media plugins avi_reader ;
# EnableWerror src add-ons media plugins ffmpeg ;
# EnableWerror src add-ons media plugins matroska ;
# EnableWerror src add-ons media plugins mov_reader ;
# EnableWerror src add-ons media plugins mp3_reader ;
# EnableWerror src add-ons media plugins ogg ;
# EnableWerror src add-ons media plugins raw_decoder ;
# EnableWerror src add-ons media plugins speex ;
# EnableWerror src add-ons media plugins wav_reader ;
# EnableWerror src add-ons translators exr ;
# EnableWerror src add-ons translators hpgs ;
# EnableWerror src add-ons translators jpeg ; # gcc2
# EnableWerror src add-ons translators png ; # gcc2
# EnableWerror src add-ons translators stxt ;
# EnableWerror src add-ons translators wonderbrush ;
}}}
The process would be:
- Uncomment one of these lines in the ArchitectureRules file,
- Build Haiku
- Compilation will fail because of a compiler warning now being considered
an error
- Alter the code to fix the warning
- Submit your change to Gerrit for code review
Mention this ticket in the commit message of your changes (include a line
saying "Fixes: #9640") to help us find what your work is related to.
Eventually, all warnings will be fixed and we can enable Werror globally
on all our sourcecode.
Note that there are exceptions to this: some code is "vendored", that is,
we copied it from other projects. For example, the NTFS driver contains
code from the NTFS-3G project. In this case, the fixes should be discussed
with the original developers of the code, and made on their project if
they accept it, to make it easier for us to merge in later versions of
their code. So, if you are new to Haiku and want to help with this, it's
better to start from parts of the sourcecode that are not in this
situation (ask us if you don't know how to determine this).
--
-- =
Ticket URL: <https://dev.haiku-os.org/ticket/9460#comment:13>
Haiku <https://dev.haiku-os.org>
The Haiku operating system.
--===============5604143519825678474==--
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic