diff options
author | NRK <nrk@disroot.org> | 2023-05-09 02:34:28 +0200 |
---|---|---|
committer | NRK <nrk@disroot.org> | 2023-05-09 02:34:28 +0200 |
commit | 657080a7e55843e351fa6ce41e4ce315eab62b67 (patch) | |
tree | bc5c0ebba4e8742887552c8cb5d506b0f4b50b5a | |
parent | 733916ad70b5b2d92e7adfecdf53a74dd4617b74 (diff) | |
download | nsxiv-657080a7e55843e351fa6ce41e4ce315eab62b67.tar.zst |
config.mk: default to `-O2` and `-DNDEBUG` (#435)
assertions are for debugging purposes, and so shouldn't be enabled for
"release" builds. disable it by default by using `-DNDEBUG`.
`-O2` on gcc/clang will result it slightly better binary. on tcc it'll
be ignored. and since -O is specified by POSIX there shouldn't be any
portability concern.
additionally add a (commented out) recommended debug build for gcc/clang
with address and undefined sanitizers turned on.
Closes: https://codeberg.org/nsxiv/nsxiv/issues/424
Reviewed-on: https://codeberg.org/nsxiv/nsxiv/pulls/435
Reviewed-by: explosion-mental <explosion-mental@noreply.codeberg.org>
Reviewed-by: eylles <eylles@noreply.codeberg.org>
-rw-r--r-- | config.mk | 7 |
1 files changed, 5 insertions, 2 deletions
@@ -23,8 +23,11 @@ HAVE_LIBWEBP = $(OPT_DEP_DEFAULT) # Compiler and linker CC = c99 -# CFLAGS, any optimization flags goes here -CFLAGS = -Wall -pedantic +# CFLAGS, any additional compiler flags goes here +CFLAGS = -Wall -pedantic -O2 -DNDEBUG +# Uncomment for a debug build using gcc/clang +# CFLAGS = -Wall -pedantic -g3 -fsanitize=address,undefined +# LDFLAGS = $(CFLAGS) # icons that will be installed via `make icon` ICONS = 16x16.png 32x32.png 48x48.png 64x64.png 128x128.png |