Buildflags

[remicollet]

Hi,

CFLAGS is set in config.m4.
This is a bad practice as this don't honor default build flags used in downstream distribution.

As you only add warnings, probably you can omit those.

Else, please use CFLAGS="CFLAGS ..."

FYI, fedora use : "-O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic" and I often add "-Werror"

for now, I will use sed -e 's/CFLAGS/IGNORED/' -i config.m4

[remicollet]

FYI, with -Wall, we have various warnings (nothin serious, but easy to fix)

• warning: 'call_user_function_v' defined but not used [-Wunused-function]
• warning: 'php_sundown__get_flags' defined but not used [-Wunused-function]

[remicollet]

Another small issue: sundown/html/html.c is executable, please chmod it.

[chobie]

sundown/html/html.c is under the vmg/sundown repository. also hoedown keeps same permission.
we should report it to hoedown.