Re: [ros-dev] [ros-diffs] [sginsberg] 42829: - svchost: #ifdef _MSC_VER doesn't mean "using Microsoft's headers" anymore - ddraw, imm32, msxml3, oleaut32, riched20: Include typeof.h for typeof emulation when compiling under MSVC and remove from React

Fri, 11 Sep 2009 07:59:53 -0700 

Filip:


*NetworkAddressLength = (UINT)((j+1)/2);

Do not equal to

> +    if ((j % 2) == 0)     //////    if  even
> +    {
> +        *NetworkAddressLength = (UINT)(j/2);
> +    }
> +    else
> +    {
> +        *NetworkAddressLength = (UINT)((j/2)+1);
> +    }






2009/8/23 Filip Navara <[email protected]>

> On Fri, Aug 21, 2009 at 5:57 PM, <[email protected]> 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.
>
> _______________________________________________
> Ros-dev mailing list
> [email protected]
> http://www.reactos.org/mailman/listinfo/ros-dev
>



-- 
----------------------------------------
Wladimir A. Jiménez B.
http://www.kasbeel.cl
Linux User # 444661
Ubuntu User # 19201

_______________________________________________
Ros-dev mailing list
[email protected]
http://www.reactos.org/mailman/listinfo/ros-dev

Reply via email to