Skip to content
Snippets Groups Projects

Increase warning level & Fix all warnings

Merged Increase warning level & Fix all warnings
fix-all-warnings into master
All threads resolved!
Merged rahix requested to merge fix-all-warnings into master
All threads resolved!
Viewing commit 4107bd22
Prev Next
Show latest version
1 file
+ 5
− 0
Compare changes
  • Side-by-side
  • Inline
  • 4107bd22
    rahix
    hack(ble): Temporarily silence a warning · 4107bd22
    rahix authored 
    With the increased warning-level, GCC warns about three unused fields in
`bleSmpCfg`:

    - maxAttemptTimeout: Maximum 'Repeated attempts' timeout in msec
    - attemptDecTimeout: Time msec before attemptExp decreases
    - attemptExp: Exponent to raise attemptTimeout on maxAttempts

These should probably be set to sensible values but to make !360
mergeable, explicitly set them to zero for now.

Signed-off-by: default avatarRahix <rahix@rahix.de>
epicardium/ble/ble_main.c
+ 5
− 0
@@ -122,6 +122,11 @@ static const smpCfg_t bleSmpCfg =
16, /*! Maximum encryption key length */
3, /*! Attempts to trigger 'repeated attempts' timeout */
DM_AUTH_MITM_FLAG, /*! Device authentication requirements */
/* TODO: The following three parameters should probably get proper values */
0, /*! Maximum 'Repeated attempts' timeout in msec */
0, /*! Time msec before attemptExp decreases */
0, /*! Exponent to raise attemptTimeout on maxAttempts */
};
/* Configuration structure */