On Fri, Aug 21, 2009 at 5:57 PM, <sginsberg(a)svn.reactos.org> wrote:
> --- trunk/reactos/drivers/network/ndis/ndis/config.c [iso-8859-1] (original)
> +++ trunk/reactos/drivers/network/ndis/ndis/config.c [iso-8859-1] Fri Aug 21 17:57:26 2009
> @@ -705,7 +705,14 @@
>
> Â Â while (j < str.Length && str.Buffer[j] != '\0') j++;
>
> - Â Â *NetworkAddressLength = (UINT)((j/2)+0.5);
> + Â Â if ((j % 2) == 0)
> + Â Â {
> + Â Â Â Â *NetworkAddressLength = (UINT)(j/2);
> + Â Â }
> + Â Â else
> + Â Â {
> + Â Â Â Â *NetworkAddressLength = (UINT)((j/2)+1);
> + Â Â }
Why not use *NetworkAddressLength = (UINT)((j+1)/2); unconditionally?
>
> Â Â if ((*NetworkAddressLength) == 0)
> Â Â {
>
F.