neels has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-msc/+/15342 )

Change subject: ran_dec logging: log message sizes on errors
......................................................................


Patch Set 1:

interesting to note that my gcc seems to not warn about this at all.

----
gcc -DHAVE_CONFIG_H -I. -I../../../../src/osmo-msc/src/libmsc -I../..   
-I../../../../src/osmo-msc/include -I../..  -fsanitize=address 
-fsanitize=undefined -Werror -Wno-error=deprecated 
-Wno-error=deprecated-declarations -Wno-error=cpp -Wall -I/usr/local/include/ 
-I/usr/local/include/ -I/usr/local/include/  -I/usr/local/include/ 
-I/usr/local/include/ -I/usr/local/include/asn1c -I/usr/local/include/ 
-I/usr/local/include/ -I/usr/local/include/ -I/usr/local/include/ 
-I/usr/local/include/ -I/usr/local/include/  -g -O0 -fsanitize=address 
-fsanitize=undefined -Werror -Wno-error=deprecated 
-Wno-error=deprecated-declarations -Wno-error=cpp -Werror=implicit 
-Werror=maybe-uninitialized -Werror=memset-transposed-args -Wnull-dereference 
-Werror=sizeof-array-argument -Werror=sizeof-pointer-memaccess -MT ran_msg_a.o 
-MD -MP -MF .deps/ran_msg_a.Tpo -c -o ran_msg_a.o 
../../../../src/osmo-msc/src/libmsc/ran_msg_a.c
mv -f .deps/ran_msg_a.Tpo .deps/ran_msg_a.Po
----

gcc (Debian 8.3.0-6) 8.3.0


--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/15342
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Id08e4ee5a4dbf552dbb107d8f0519110664f6acb
Gerrit-Change-Number: 15342
Gerrit-PatchSet: 1
Gerrit-Owner: neels <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: neels <[email protected]>
Gerrit-CC: pespin <[email protected]>
Gerrit-Comment-Date: Fri, 30 Aug 2019 12:30:05 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to