gnucash maint: Fix review comments for locale restructuring

Geert Janssens gjanssens at code.gnucash.org
Mon Mar 22 07:17:46 EDT 2021


Updated	 via  https://github.com/Gnucash/gnucash/commit/2eadfb9b (commit)
	from  https://github.com/Gnucash/gnucash/commit/6537135e (commit)



commit 2eadfb9b4178a5e8afcc9c900b4b361176cdd4d1
Author: Geert Janssens <geert at kobaltwit.be>
Date:   Mon Mar 22 12:17:38 2021 +0100

    Fix review comments for locale restructuring

diff --git a/gnucash/CMakeLists.txt b/gnucash/CMakeLists.txt
index 32dec4a19..13a578f3f 100644
--- a/gnucash/CMakeLists.txt
+++ b/gnucash/CMakeLists.txt
@@ -47,7 +47,7 @@ set (gnucash_SOURCES
 
 if (MINGW)
   list(APPEND gnucash_SOURCES "gnucash-locale-windows.c")
-elseif(MAC_INTEGRATION)
+elseif (MAC_INTEGRATION)
     list(APPEND gnucash_SOURCES "gnucash-locale-macos.mm")
 endif()
 
@@ -75,8 +75,7 @@ set(gnucash_cli_SOURCES
 
 if (MINGW)
   list(APPEND gnucash_cli_SOURCES "gnucash-locale-locale.c")
-endif()
-if (MAC_INTEGRATION)
+elseif (MAC_INTEGRATION)
     list(APPEND gnucash_cli_SOURCES "gnucash-locale-macos.mm")
 endif()
 
diff --git a/gnucash/gnucash-locale-windows.c b/gnucash/gnucash-locale-windows.c
index 119bfd3ac..2c3b8a51a 100644
--- a/gnucash/gnucash-locale-windows.c
+++ b/gnucash/gnucash-locale-windows.c
@@ -32,7 +32,7 @@
  * it is set both the Windows and POSIX locales to that. If not
  * retrieve the Windows locale and set POSIX to match.
  */
-char *void
+char *
 set_platform_locale()
 {
     WCHAR lpLocaleName[LOCALE_NAME_MAX_LENGTH];



Summary of changes:
 gnucash/CMakeLists.txt           | 5 ++---
 gnucash/gnucash-locale-windows.c | 2 +-
 2 files changed, 3 insertions(+), 4 deletions(-)



More information about the gnucash-changes mailing list