Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 20546: /trunk/epan/dissectors/ /trun

From: Joerg Mayer <jmayer@xxxxxxxxx>
Date: Fri, 26 Jan 2007 10:51:48 +0100
On Thu, Jan 25, 2007 at 10:32:29AM -0800, Gerald Combs wrote:
> > On Wed, Jan 24, 2007 at 07:22:15PM +0000, gerald@xxxxxxxxxxxxx wrote:
> >> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=20546

> The commit included whitespace changes and ViewVC is configured to omit
> those in its diff output.  As I recall, this was set after someone
> requested it a while back.  I'd be happy to change it back if that would
> be preferred.

That was my first impression too, but then I saw the following lines (in
the context diff mode):

!         goto end_of_wlan;
        }
-                       }
      } else {

And that '-' line doesn't look like a pure whitespace change. but from looking
at it again, it seems that the '-' line is wrong in the context diff output
and the colored output is correct.

Ciao
    Joerg
-- 
Joerg Mayer                                           <jmayer@xxxxxxxxx>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.