Wireshark · Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 35975: /trunk/epan/dfilter/ /trunk/epan/dfilter/: dfilte

Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 35975: /trunk/epan/dfilter/ /trunk/e

From: Stig Bjørlykke <stig@xxxxxxxxxxxxx>

Date: Thu, 17 Feb 2011 09:25:39 +0100

On Thu, Feb 17, 2011 at 9:15 AM,  <guy@xxxxxxxxxxxxx> wrote:
>  OK, let's try a couple more explicit checks against NULL, to see whether
>  that de-confuses Microsoft's code analyzer.

Do we really want such changes?
Personally I think it makes the code harder to read.


-- 
Stig Bjørlykke

Follow-Ups:
- Re: [Wireshark-dev] [Wireshark-commits] rev 35975: /trunk/epan/dfilter/ /trunk/epan/dfilter/: dfilter-macro.c dfilter.c
  - From: Guy Harris

Prev by Date: Re: [Wireshark-dev] Static analysis added to the buildbot
Next by Date: Re: [Wireshark-dev] [Wireshark-commits] rev 35975: /trunk/epan/dfilter/ /trunk/epan/dfilter/: dfilter-macro.c dfilter.c
Previous by thread: Re: [Wireshark-dev] RFC: Should ip.addr also match on ARP packets
Next by thread: Re: [Wireshark-dev] [Wireshark-commits] rev 35975: /trunk/epan/dfilter/ /trunk/epan/dfilter/: dfilter-macro.c dfilter.c
Index(es):
- Date
- Thread