Wireshark mailing list archives

Re: [Wireshark-commits] rev 53531: /trunk/ /trunk/epan/dissectors/: packet-x11.c x11-declarations.h x11-enum.h x11-extension-errors.h x11-extension-implementation.h x11-glx-render-enum.h x11-register-info.h ...


From: Bill Meier <wmeier () newsguy com>
Date: Tue, 17 Dec 2013 14:25:35 -0500

On 11/23/2013 8:32 PM, morriss () wireshark org wrote:
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=53531

User: morriss
Date: 2013/11/24 01:32 AM

Log:
  Some patches from Peter Harris to make it possible to build the X11 dissector
  again (and some various other improvements):

When compiling X11 with dev CLang 4.8 (dev LLVM 2.4 but soon be be a release), the following warning message is generated:

warning: explicitly assigning a variable of type 'int' to
      itself [-Wself-assign]
    n = n; /* Avoid unreferenced warning */


This is basically just to give a heads up, since this is a very minor nit sompared to all the stuff (warning messages) that got fixed (Thanks).

Using 'n = n + 1' does seem to kill the warning message from Clang so maybe this can be used until the generator "ToDo" related to this code has been completed.

Bill
___________________________________________________________________________
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-request () wireshark org?subject=unsubscribe


Current thread: