Skip to content

paranoid-2: Allow to compile code with paranoid compiler flags -Wall -Wextra -Wshadow -Werror -pedantic-errors#139

Merged
MichalPrincNXP merged 1 commit intoEmbeddedRPC:developfrom
jcdr:paranoid-2
Oct 6, 2020
Merged

paranoid-2: Allow to compile code with paranoid compiler flags -Wall -Wextra -Wshadow -Werror -pedantic-errors#139
MichalPrincNXP merged 1 commit intoEmbeddedRPC:developfrom
jcdr:paranoid-2

Conversation

@jcdr
Copy link
Contributor

@jcdr jcdr commented Oct 5, 2020

infra/erpc_simple_server.cpp: fix declaration of ‘err’ shadows a prev…ious local

erpc_simple_server.cpp: In member function ‘virtual erpc_status_t erpc::SimpleServer::run(erpc::RequestContext&)’:
erpc_simple_server.cpp:181:23: error: declaration of ‘err’ shadows a previous local [-Werror=shadow]
erpc_status_t err = runInternalBegin(&codec, buff, msgType, serviceId, methodId, sequence);
^~~
erpc_simple_server.cpp:169:19: note: shadowed declaration is here
erpc_status_t err = kErpcStatus_Success;
^~~

…ious local

erpc_simple_server.cpp: In member function ‘virtual erpc_status_t erpc::SimpleServer::run(erpc::RequestContext&)’:
erpc_simple_server.cpp:181:23: error: declaration of ‘err’ shadows a previous local [-Werror=shadow]
         erpc_status_t err = runInternalBegin(&codec, buff, msgType, serviceId, methodId, sequence);
                       ^~~
erpc_simple_server.cpp:169:19: note: shadowed declaration is here
     erpc_status_t err = kErpcStatus_Success;
                   ^~~
@jcdr
Copy link
Contributor Author

jcdr commented Oct 6, 2020

I will probably be too busy this week to continue improving this one. Please merge as is and I will open a new branch for the next pull request.

@MichalPrincNXP
Copy link
Member

I will probably be too busy this week to continue improving this one. Please merge as is and I will open a new branch for the next pull request.

OK, thanks anyway!

@MichalPrincNXP MichalPrincNXP merged commit 6129df6 into EmbeddedRPC:develop Oct 6, 2020
CBohler99 pushed a commit to VERO-Biotech/erpc that referenced this pull request Dec 17, 2025
…ic-Status-Updates-and-ERPCSIM-Debug-Build

[GDS4-3083] Update to Periodic Status and ERPCSIM CMAKE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants