Who made the modification in the first place, adding the dependency on the
math library? If this patch removes that dependency while still getting the
same result done, I would prefer getting rid of the dependency.

On Tue, Mar 8, 2011 at 4:12 PM, Mads Kiilerich <m...@kiilerich.com> wrote:

> # HG changeset patch
> # User Mads Kiilerich <m...@kiilerich.com>
> # Date 1299618735 -3600
> # Node ID a928c6f05ba6d96ea1e70bd5094178bf90505fbc
> # Parent  d8e5cace7b54eff8e33efc0b81c5441eba898de9
> libfreerdp orders: replace floating point math with bit operation
>
> OK? Or was there a magic reason this doesn't work?
>
> /Mads
>
> diff --git a/X11/Makefile.am b/X11/Makefile.am
> --- a/X11/Makefile.am
> +++ b/X11/Makefile.am
> @@ -23,4 +23,4 @@
>        ../libfreerdpkbd/libfreerdpkbd.la \
>        ../libfreerdputils/libfreerdputils.la \
>        ../libfreerdpchanman/libfreerdpchanman.la \
> -       @XCURSOR_LIBS@ @X_LIBS@ @X_EXTRA_LIBS@ -lm
> +       @XCURSOR_LIBS@ @X_LIBS@ @X_EXTRA_LIBS@
> diff --git a/cunit/Makefile.am b/cunit/Makefile.am
> --- a/cunit/Makefile.am
> +++ b/cunit/Makefile.am
> @@ -21,6 +21,6 @@
>        ../libfreerdpkbd/libfreerdpkbd.la \
>        ../libfreerdpgdi/libfreerdpgdi.la \
>        ../libfreerdpchanman/libfreerdpchanman.la \
> -       -lfusion -ldirect -lz -lcunit -lm -lncurses
> +       -lfusion -ldirect -lz -lcunit -lncurses
>
>
> diff --git a/dfb/Makefile.am b/dfb/Makefile.am
> --- a/dfb/Makefile.am
> +++ b/dfb/Makefile.am
> @@ -21,6 +21,6 @@
>        ../libfreerdpkbd/libfreerdpkbd.la \
>        ../libfreerdpgdi/libfreerdpgdi.la \
>        ../libfreerdpchanman/libfreerdpchanman.la \
> -       @DFB_LIBS@ -lfusion -ldirect -lz -lm
> +       @DFB_LIBS@ -lfusion -ldirect -lz
>
>
> diff --git a/libfreerdp/orders.c b/libfreerdp/orders.c
> --- a/libfreerdp/orders.c
> +++ b/libfreerdp/orders.c
> @@ -351,7 +351,7 @@
>        memset(rects, 0, size);
>
>        index = 0;
> -       data = ceil(((double)os->nentries) / 2.0);
> +       data = (os->nentries + 1) >> 1;
>        for (next = 1; (next <= os->nentries) && (next <= 45) && (data <
> os->datasize); next++)
>        {
>                if ((next - 1) % 2 == 0)
> @@ -561,7 +561,7 @@
>        memset(rects, 0, size);
>
>        index = 0;
> -       data = ceil(((double)os->nentries) / 2.0);
> +       data = (os->nentries + 1) >> 1;
>        for (next = 1; (next <= os->nentries) && (next <= 45) && (data <
> os->datasize); next++)
>        {
>                if ((next - 1) % 2 == 0)
> diff --git a/libfreerdp/tls.c b/libfreerdp/tls.c
> old mode 100644
> new mode 100755
>
>
> ------------------------------------------------------------------------------
> What You Don't Know About Data Connectivity CAN Hurt You
> This paper provides an overview of data connectivity, details
> its effect on application quality, and explores various alternative
> solutions. http://p.sf.net/sfu/progress-d2d
> _______________________________________________
> Freerdp-devel mailing list
> Freerdp-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/freerdp-devel
>
------------------------------------------------------------------------------
What You Don't Know About Data Connectivity CAN Hurt You
This paper provides an overview of data connectivity, details
its effect on application quality, and explores various alternative
solutions. http://p.sf.net/sfu/progress-d2d
_______________________________________________
Freerdp-devel mailing list
Freerdp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freerdp-devel

Reply via email to