Re: [Wireshark-commits] master e143570: Define INET6 for all platforms. Show the addresses as a tooltip in capture interfaces.

[Joerg Mayer <jmayer () loplof de>]

On Sat, Jun 14, 2014 at 01:27:12PM +0000, Wireshark code review wrote:
> URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e14357032f38416a733596d3f003d51710eb1e52
...
> e143570 by Irene Ruengeler (ruengeler () wireshark org):
>
>     Define INET6 for all platforms.
>     Show the addresses as a tooltip in capture interfaces.
>     
>     Change-Id: I911784e09ed9479229a7d6f8a7f1476e2e1e6224
>     Reviewed-on: https://code.wireshark.org/review/2155
>     Reviewed-by: Evan Huus <eapache () gmail com>
>     Reviewed-by: Alexis La Goutte <alexis.lagoutte () gmail com>

Why not take the clean approach and remove all the
"#ifdef AF_INET6/#ifndef AF_INET6/defined(INET6)" checks and the definition
of INET6 itself instead?

Thanks
   Jörg
-- 
Joerg Mayer                                           <jmayer () loplof de>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
___________________________________________________________________________
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