ACK,
It's not you, that's duplicate code.
Eric
On Feb 28, 2010, at 09:05:46, Gert Doering wrote:
> Hi,
>
> tun.c contains this code, in the
>
> #elif defined(TARGET_FREEBSD)||defined(TARGET_DRAGONFLY)
>
> part of do_ifconfig() (about line 983 in the IPv6 payload branch):
>
> ----------------- snip ----------------
> else {
> if (tt->topology == TOP_SUBNET)
> argv_printf (&argv,
> "%s %s %s netmask %s mtu %d up",
> IFCONFIG_PATH,
> actual,
> ifconfig_local,
> ifconfig_remote_netmask,
> tun_mtu
> );
> else
> argv_printf (&argv,
> "%s %s %s netmask %s mtu %d up",
> IFCONFIG_PATH,
> actual,
> ifconfig_local,
> ifconfig_remote_netmask,
> tun_mtu
> );
> }
> ----------------- snip ----------------
>
> Is it just me needing new glasses, or is this twice the very same code,
> with a little bit of different indentation?
>
> If other people see this too, I'll send in a patch that drops the
> duplicate bits. The tun.c code is still hard enough to read...
>
> gert
> --
> USENET is *not* the non-clickable part of WWW!
> //www.muc.de/~gert/
> Gert Doering - Munich, Germany [email protected]
> fax: +49-89-35655025 [email protected]
>
> ------------------------------------------------------------------------------
> Download Intel® Parallel Studio Eval
> Try the new software tools for yourself. Speed compiling, find bugs
> proactively, and fine-tune applications for parallel performance.
> See why Intel Parallel Studio got high marks during beta.
> http://p.sf.net/sfu/intel-sw-dev
> _______________________________________________
> Openvpn-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/openvpn-devel
>
---
Eric Crist
