Re: [Wireshark-dev] 25 Warnings on Clang Scan Build

2014-10-15 Thread Bálint Réczey
2014-10-14 21:41 GMT+02:00 Alexis La Goutte alexis.lagou...@gmail.com:
 Hi,

 For the first time, you are under 25 warnings on Clang scan-build :-D
 ( https://www.wireshark.org/download/automated/analysis/ )

 A quick review about last warnings :
 3 coming from flex (Dead Assignement on ascend_scanner.c, k12text.c,
 ascend.c), need to fix flex on upstream...


 10 coming from lemon (sqlite), i have start to update lemon from
 upstream ( https://code.wireshark.org/review/#/c/3976/ ) but all
 warning is no fixed...

 and last warning is Dereference of null pointer not yet look all
 warning but some warning is false postive (packet-ber...)
Great news!

Cheers,
Balint
___
Sent via:Wireshark-dev mailing list wireshark-dev@wireshark.org
Archives:http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
 mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe


[Wireshark-dev] 25 Warnings on Clang Scan Build

2014-10-14 Thread Alexis La Goutte
Hi,

For the first time, you are under 25 warnings on Clang scan-build :-D
( https://www.wireshark.org/download/automated/analysis/ )

A quick review about last warnings :
3 coming from flex (Dead Assignement on ascend_scanner.c, k12text.c,
ascend.c), need to fix flex on upstream...


10 coming from lemon (sqlite), i have start to update lemon from
upstream ( https://code.wireshark.org/review/#/c/3976/ ) but all
warning is no fixed...

and last warning is Dereference of null pointer not yet look all
warning but some warning is false postive (packet-ber...)


But also missing analyse of qt folder (bug in this Clang buildbot release)...

Regards,
___
Sent via:Wireshark-dev mailing list wireshark-dev@wireshark.org
Archives:http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
 mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe