[Poppler-bugs] [Bug 90796] [patch] Allow disabling C assertions with configure --disable-assert

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=90796

--- Comment #5 from William Bader <williambader at hotmail.com> ---
Created attachment 116815
  --> https://bugs.freedesktop.org/attachment.cgi?id=116815&action=edit
revised patch

This patch creates a ./configure option called --enable-release-mode that
adds-O2 -DNDEBUG to CFLAGS and CXXFLAGS just like the options in
cmake/modules/PopplerMacros.cmake that are enabled by cmake
-DCMAKE_BUILD_TYPE=Release.
$ grep -i release cmake/modules/PopplerMacros.cmake 
  set(CMAKE_CXX_FLAGS_RELEASE        "-O2 -DNDEBUG")
  set(CMAKE_C_FLAGS_RELEASE          "-O2 -DNDEBUG")
  set(CMAKE_CXX_FLAGS_RELEASE        "-O2 -DNDEBUG")
  set(CMAKE_C_FLAGS_RELEASE          "-O2 -DNDEBUG")
Is that better? It creates a configure option with a name similar to the cmake
option that adds the same flags as the cmake option.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/poppler-bugs/attachments/20150630/8bb565fa/attachment.html>