gnucash maint: Multiple changes pushed

[Christopher Lam]

Updated	 via  https://github.com/Gnucash/gnucash/commit/5ced0d93 (commit)
	 via  https://github.com/Gnucash/gnucash/commit/40d886fa (commit)
	 via  https://github.com/Gnucash/gnucash/commit/4c37f6d4 (commit)
	 via  https://github.com/Gnucash/gnucash/commit/e8454992 (commit)
	from  https://github.com/Gnucash/gnucash/commit/fa666e73 (commit)



commit 5ced0d932a942526ef962f0748614c52cade1ac7
Merge: fa666e736 40d886fa9
Author: Christopher Lam <christopher.lck at gmail.com>
Date:   Thu Aug 12 17:39:45 2021 +0800

    Merge branch 'maint-account-cpp' into maint #1107

commit 40d886fa9d54d556efbd6f150fd82a5a92d48fd1
Author: Christopher Lam <christopher.lck at gmail.com>
Date:   Wed Aug 11 09:28:26 2021 +0800

    gnc_account_list_name_violations elements must be freed

commit 4c37f6d4ef1de455dd24839ab4a126c6f71c6c70
Author: Christopher Lam <christopher.lck at gmail.com>
Date:   Wed Aug 11 09:28:22 2021 +0800

    [account.cpp] gnc_g_list_stringjoin instead of repeated allocations

commit e84549926bbeaf6f19428f1aeac1b96eeb860b6e
Author: Christopher Lam <christopher.lck at gmail.com>
Date:   Tue Aug 10 12:55:47 2021 +0800

    [gnc-glib-utils] gnc_g_list_stringjoin to join a GList of strings
    
    It traverses the GList twice (once to calculate the length) but
    allocates only once.
    
    Includes snippet from
    https://www.joelonsoftware.com/2001/12/11/back-to-basics/



Summary of changes:
 gnucash/gnome-utils/dialog-preferences.c         |  2 +-
 gnucash/gnome-utils/gnc-file.c                   |  1 +
 libgnucash/core-utils/gnc-glib-utils.c           | 32 +++++++++++++
 libgnucash/core-utils/gnc-glib-utils.h           | 16 +++++++
 libgnucash/core-utils/test/test-gnc-glib-utils.c | 57 ++++++++++++++++++++++++
 libgnucash/engine/Account.cpp                    | 16 +------
 libgnucash/engine/Account.h                      |  4 +-
 libgnucash/engine/test/utest-Account.cpp         | 16 ++-----
 8 files changed, 113 insertions(+), 31 deletions(-)