gnucash

Commit Graph

Author	SHA1	Message	Date
Simon Arlott	d7b2b06bae	Avoid unnecessary memory allocation in dom_tree_to_gnc_numeric() Return a gnc_numeric instead of allocations that every caller has to free. This makes it easier to fix the use after free in the unit test function equals_node_val_vs_split_internal() where the expression in the return statement wants to use the allocated gnc_numeric.	3 years ago
Richard Cohen	2e861ab673	Valgrind: fix "Mismatched free/delete" - test-dom-converters1 ==32195== Mismatched free() / delete / delete [] ==32195== at 0x484620F: free (in /usr/libexec/valgrind/vgpreload_memcheck-amd64-linux.so) ==32195== by 0x146DB5: test_dom_tree_to_guid() (test-dom-converters1.cpp:251) ... ==32195== Address 0x8043780 is 0 bytes inside a block of size 16 alloc'd ==32195== at 0x4843FA3: operator new(unsigned long) (in /usr/libexec/valgrind/vgpreload_memcheck-amd64-linux.so) ==32195== by 0x503E06E: guid_malloc (guid.cpp:106) ==32195== by 0x503E0D5: guid_copy (guid.cpp:123) ==32195== by 0x503E1E7: guid_new (guid.cpp:154) ==32195== by 0x147188: get_random_guid (test-engine-stuff.cpp:207) ==32195== by 0x146D03: test_dom_tree_to_guid() (test-dom-converters1.cpp:237) ...	3 years ago
Richard Cohen	1cec0cb3f3	Use internal extern "C" { ... } for C++ - removes warnings compiling swig engine ... [ 10%] Generating swig-engine.cpp .../libgnucash/engine/engine-helpers.h:31: Warning 313: Unrecognized extern type "C++". .../libgnucash/engine/gnc-date.h:83: Warning 313: Unrecognized extern type "C++". .../libgnucash/engine/qofquery.h:90: Warning 302: Identifier 'QofQuery' redefined (ignored), .../libgnucash/engine/gnc-option.hpp:55: Warning 302: previous definition of 'QofQuery'. .../libgnucash/engine/gnc-commodity.h:56: Warning 313: Unrecognized extern type "C++". .../libgnucash/engine/gncBusiness.h:40: Warning 313: Unrecognized extern type "C++". .../libgnucash/engine/gncEntry.h:37: Warning 313: Unrecognized extern type "C++".	3 years ago
John Ralls	4359243c08	Move glib and gtk includes out of extern C for tests. Needed to compile with glib 2.67 and later.	5 years ago
Christopher Lam	de09259f13	Bug 797500 - valgrind errors - new/delete vs malloc/free Mismatched new/g_free() Change to new/guid_free() which calls delete	7 years ago
John Ralls	755773ac4c	Clean up timespec comments in the XML backend. Noting where the word must stay; most significantly for KVP types and for parsing Version 1 XML files.	8 years ago
Geert Janssens	b761b5a0dc	Bug 795377 - Reads and saves Gnucash 2.6.19 XML file, then can't reread it, due to bad date in old file This commit contains another round of cleanups in the timespec to time64 conversion. There were a number of false assumptions that time64 = 0 would be a bad date in the xml parser. This commit corrects enough of them to eliminate the bug. Further cleanup is probably advised but can be done at a later stage.	8 years ago
lmat	f77369bb96	Removing timspec from some xml parsing And several other sprawling references that are connected in one way or another.	8 years ago
lmat	34e0d6cfa0	kvp frame to template and correcting failure macro The template avoids the need to cast to and from void*, and adds flexibility to the targeted function's signature. test-stuff.h defines a macro, "failure" which is used as an identifier in the standard IO library, so I moved any inclusion of test-stuff.h to the last include position so that "failure" wouldn't be defined before the IO library was included.	8 years ago
Geert Janssens	1238b9d8cd	Prevent gcc from searching config.h in the current directory This will avoid a ninja-build from picking up a config.h generated by the autotools build (in the root build directory). Picking up the wrong config.h may lead to all kinds of subtle issues if the autotools run was done with different options than the cmake run.	9 years ago
Geert Janssens	83d14e1c1c	Restructure the src directory It is split into - /libgnucash (for the non-gui bits) - /gnucash (for the gui) - /common (misc source files used by both) - /bindings (currently only holds python bindings) This is the first step in restructuring the code. It will need much more fine tuning later on.	9 years ago

11 Commits (8a91fa8deb61302b0edc54aaadfd4ab37c9f72db)