Skip to content

Commit

Permalink
Merge branch 'maint'
Browse files Browse the repository at this point in the history
  • Loading branch information
jralls committed Oct 28, 2015
2 parents 5537a7e + ce838d3 commit ac34d31
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 10 deletions.
10 changes: 6 additions & 4 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -1297,7 +1297,7 @@ then
#Save CFLAGS so that we can put the warnings in AM_CFLAGS
__cflags="$CFLAGS"
CFLAGS=
GNOME_COMPILE_WARNINGS
COMPILE_WARNINGS
AM_CFLAGS="$CFLAGS"
CFLAGS="$__cflags"

Expand Down Expand Up @@ -1438,7 +1438,7 @@ then
AM_CXXFLAGS="${AM_CXXFLAGS} -Wall"

else
# GNOME_COMPILE_WARNINGS will add -Wall; no need to set it again.
# COMPILE_WARNINGS will add -Wall; no need to set it again.
# also, only add it for GCC.
if test ${GCC}x = yesx
then
Expand Down Expand Up @@ -1552,12 +1552,14 @@ AC_ARG_ENABLE(python-bindings,
### Additional compiler warnings (or not) if we're running GCC
###-------------------------------------------------------------------------

# This has to come after AC_PROG_CC _AND_ GNOME_COMPILE_WARNINGS

# This has to come after AC_PROG_CC _AND_ COMPILE_WARNINGS
AC_LANG([C++])
AX_CHECK_COMPILE_FLAG([-Wno-deprecated-register],
[AM_CXXFLAGS="${AM_CXXFLAGS} -Wno-unused -Wno-deprecated-register"],
[AM_CXXFLAGS="${AM_CXXFLAGS} -Wno-unused"], [-Werror])
AC_LANG([C])

AC_MSG_CHECKING(what extra warning flags to pass to the C compiler)
if test ${GCC}x = yesx
then
Expand All @@ -1567,7 +1569,7 @@ then
#around to cleaning them up.
AM_CFLAGS="${AM_CFLAGS} -Wno-unused"
# other flags...
# These next two are included in the GNOME_COMPILE_WARNINGS
# These next two are included in the COMPILE_WARNINGS
#warnFLAGS="${warnFLAGS} -Wmissing-prototypes"
#warnFLAGS="${warnFLAGS} -Wmissing-declarations"
#warnFLAGS="${warnFLAGS} -Werror-implicit-function-declaration" # In -Wall
Expand Down
10 changes: 5 additions & 5 deletions macros/compiler-flags.m4
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
dnl GNOME_COMPILE_WARNINGS
dnl COMPILE_WARNINGS
dnl Turn on many useful compiler warnings
dnl For now, only works on GCC
AC_DEFUN([GNOME_COMPILE_WARNINGS],[
AC_ARG_ENABLE(compile-warnings,
AC_DEFUN([COMPILE_WARNINGS],[
AC_ARG_ENABLE(compile-warnings,
[ --enable-compile-warnings=[no/minimum/yes] Turn on compiler warnings.],,enable_compile_warnings=minimum)
AC_MSG_CHECKING(what warning flags to pass to the C compiler)
Expand Down Expand Up @@ -56,8 +56,8 @@ AC_DEFUN([GNOME_COMPILE_WARNINGS],[

dnl For C++, do basically the same thing.

AC_DEFUN([GNOME_CXX_WARNINGS],[
AC_ARG_ENABLE(cxx-warnings,
AC_DEFUN([CXX_WARNINGS],[
AC_ARG_ENABLE(cxx-warnings,
[ --enable-cxx-warnings=[no/minimum/yes] Turn on compiler warnings.],,enable_cxx_warnings=minimum)
AC_MSG_CHECKING(what warning flags to pass to the C++ compiler)
Expand Down
3 changes: 2 additions & 1 deletion src/gnome/dialog-commodities.c
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,8 @@ remove_clicked (CommoditiesDialog *cd)
g_list_free (accounts);

pdb = gnc_pricedb_get_db (cd->book);
if (gnc_pricedb_has_prices(pdb, commodity, NULL))
prices = gnc_pricedb_get_prices(pdb, commodity, NULL);
if (prices)
{
message = _("This commodity has price quotes. Are "
"you sure you want to delete the selected "
Expand Down

0 comments on commit ac34d31

Please sign in to comment.