Page MenuHomeFreeBSD

Use proper prototypes in struct boot_module_t
ClosedPublic

Authored by dim on Jan 11 2017, 7:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 13, 7:16 PM
Unknown Object (File)
Thu, Nov 13, 3:32 PM
Unknown Object (File)
Wed, Nov 5, 12:30 PM
Unknown Object (File)
Mon, Oct 27, 6:47 PM
Unknown Object (File)
Mon, Oct 27, 8:07 AM
Unknown Object (File)
Mon, Oct 27, 8:07 AM
Unknown Object (File)
Mon, Oct 27, 2:37 AM
Unknown Object (File)
Sep 30 2025, 4:36 PM
Subscribers

Details

Summary

With clang 4.0.0, we are getting the following warnings about struct
boot_module_t in efi's boot_module.h:

In file included from sys/boot/efi/boot1/ufs_module.c:41:
sys/boot/efi/boot1/boot_module.h:67:14: error: this function declaration is not a prototype [-Werror,-Wstrict-prototypes]
        void (*init)();
                    ^
                     void
sys/boot/efi/boot1/boot_module.h:92:16: error: this function declaration is not a prototype [-Werror,-Wstrict-prototypes]
        void (*status)();
                      ^
                       void
sys/boot/efi/boot1/boot_module.h:95:24: error: this function declaration is not a prototype [-Werror,-Wstrict-prototypes]
        dev_info_t *(*devices)();
                              ^
                               void
3 errors generated.

Fix this by adding 'void' to the parameter lists. No functional change.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6714
Build 6931: arc lint + arc unit

Event Timeline

dim retitled this revision from to Use proper prototypes in struct boot_module_t.
dim updated this object.
dim edited the test plan for this revision. (Show Details)
dim added reviewers: emaste, smh.
emaste edited edge metadata.
This revision is now accepted and ready to land.Jan 11 2017, 7:49 PM
imp added a reviewer: imp.
smh edited edge metadata.
This revision was automatically updated to reflect the committed changes.