Skip to content

Commit 8e437be

Browse files
remove redundant function declaration (#1241)
* remove redundant function declaration Remove warning message from "gcc -Wredundant-decls" in FreeRTOS_TCP_IP.c. Signed-off-by: Florian La Roche <Florian.LaRoche@gmail.com> * Add -Wredundant-decls to CI checks * Update formatting CI check Github runnner to ubuntu-latest * Revert "Update formatting CI check Github runnner to ubuntu-latest" This reverts commit 28ff474. * Fix spell check * Revert "Revert "Update formatting CI check Github runnner to ubuntu-latest"" This reverts commit 8ff4ccf. --------- Signed-off-by: Florian La Roche <Florian.LaRoche@gmail.com> Co-authored-by: tony-josi-aws <tonyjosi@amazon.com>
1 parent 3733583 commit 8e437be

File tree

4 files changed

+3
-9
lines changed

4 files changed

+3
-9
lines changed

.github/.cSpellWords.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1568,6 +1568,7 @@ Wpacked
15681568
Wpedantic
15691569
Wpragma
15701570
wrdata
1571+
Wredundant
15711572
Wreserved
15721573
writen
15731574
Wsign

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ jobs:
131131
exclude-dirs: source/portable/NetworkInterface/STM32
132132

133133
formatting:
134-
runs-on: ubuntu-20.04
134+
runs-on: ubuntu-latest
135135
steps:
136136
- uses: actions/checkout@v4
137137
- name: Check formatting

source/FreeRTOS_TCP_IP.c

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -87,14 +87,6 @@
8787
/* coverity[misra_c_2012_rule_8_9_violation] */
8888
_static FreeRTOS_Socket_t * xSocketToListen = NULL;
8989

90-
#if ( ipconfigHAS_DEBUG_PRINTF != 0 )
91-
92-
/*
93-
* For logging and debugging: make a string showing the TCP flags.
94-
*/
95-
const char * prvTCPFlagMeaning( UBaseType_t xFlags );
96-
#endif /* ipconfigHAS_DEBUG_PRINTF != 0 */
97-
9890
static IPv46_Address_t xGetSourceAddrFromBuffer( const uint8_t * const pucEthernetBuffer );
9991

10092
/*-----------------------------------------------------------*/

test/build-combination/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ target_compile_options(freertos_plus_tcp
111111
$<$<COMPILE_LANG_AND_ID:C,Clang,GNU>:-Werror>
112112
$<$<COMPILE_LANG_AND_ID:C,Clang,GNU>:-Wpedantic>
113113
$<$<COMPILE_LANG_AND_ID:C,Clang,GNU>:-Wconversion>
114+
$<$<COMPILE_LANG_AND_ID:C,Clang,GNU>:-Wredundant-decls>
114115
$<$<COMPILE_LANG_AND_ID:C,Clang,GNU>:-Wunused-variable>
115116
$<$<COMPILE_LANG_AND_ID:C,Clang>:-Weverything>
116117

0 commit comments

Comments
 (0)