Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set double to be the default for all compilation modes #59

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Set double to be the default for all compilation modes #59

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ option(BUILD_DOCUMENTATION "Build the documentation" OFF)
option(BUILD_SHARED_LIBS "Build libccd as a shared library" ON)

option(ENABLE_DOUBLE_PRECISION
"Enable double precision computations instead of single precision" OFF)
"Enable double precision computations instead of single precision" ON)

# Option for some bundle-like build system in order not to expose
# any FCL binary symbols in their public ABI
Expand Down
26 changes: 16 additions & 10 deletions configure.ac
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,17 +30,23 @@ AC_FUNC_FORK
AC_FUNC_REALLOC
AC_CHECK_FUNCS([clock_gettime])

use_double=no
AC_ARG_ENABLE(double-precision,
AS_HELP_STRING([--enable-double-precision],
[enable double precision computations instead of single precision]),
[use_double=yes])
if test $use_double = no
then
AC_DEFINE([CCD_SINGLE], [], [use single precision])
else
AC_DEFINE([CCD_DOUBLE], [], [use double precision])
fi
AS_HELP_STRING([--disable-double-precision],
[use single precision instead of double preicision]))
AS_IF([test "x$enable_double_precision" == "xno"],
[AC_DEFINE([CCD_SINGLE], [], [use single precision])],
[AC_DEFINE([CCD_DOUBLE], [], [use double precision])])
#use_double=no
#AC_ARG_ENABLE(double-precision,
# AS_HELP_STRING([--enable-double-precision],
# [enable double precision computations instead of single precision]),
# [use_double=yes])
#if test $use_double = no
#then
# AC_DEFINE([CCD_SINGLE], [], [use single precision])
#else
# AC_DEFINE([CCD_DOUBLE], [], [use double precision])
#fi


AC_CONFIG_FILES([Makefile
Expand Down
8 changes: 4 additions & 4 deletions doc/compile-and-install.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,8 @@ Run make and make install:
$ make && make install

configure script can change the way libccd is compiled and installed, most
significant option is ``--enable-double-precision`` which enables double
precision (single is default in this case).
significant option is ``--disable-double-precision`` which enables single
precision (double is default in this case).

3. Using CMake
---------------
Expand Down Expand Up @@ -88,12 +88,12 @@ Other build tools may be using by specifying a different generator. For example:

> cmake -G "Visual Studio 14 2015" ..

To compile using double precision, set the ``ENABLE_DOUBLE_PRECISION`` option:
To compile using single precision, set the ``ENABLE_DOUBLE_PRECISION`` option to ``OFF``:

.. code-block:: bash

$ mkdir build && cd build
$ cmake -G "Unix Makefiles" -DENABLE_DOUBLE_PRECISION=ON ..
$ cmake -G "Unix Makefiles" -DENABLE_DOUBLE_PRECISION=OFF ..
$ make && make install

To build libccd as a shared library, set the ``BUILD_SHARED_LIBS`` option:
Expand Down