| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
...which happens to define __GNUC__=4, __GNUC_MINOR__=2. (See also
<https://git.libreoffice.org/core/+/9a23239b710e44754e4a28643420c9ba91552645%5E%21>
"external/cppunit: Don't mis-apply GCC < 4.6 workaround for Clang". This change
requires <https://gerrit.libreoffice.org/c/cppunit/+/147383> "Use snprintf
instead of sprintf" to avoid deprecation warnings on macOS, which had originally
been hidden by the mis-applied #pragma.)
Change-Id: I4fcb38a79766238862795ba4019638862e65b041
Reviewed-on: https://gerrit.libreoffice.org/c/cppunit/+/147384
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
See
<https://git.libreoffice.org/core/+/bd8a213c8ceccf66ab296ef0aea0fa4c451047e5%5E%21>
"external/cppunit: Use snprintf instead of sprintf" for how sprintf would have
caused deprecation warnings on macOS.
(For MSVC, keep using sprintf_s. According to
<https://learn.microsoft.com/en-us/cpp/c-runtime-library/reference/snprintf-snprintf-snprintf-l-snwprintf-snwprintf-l?view=msvc-170>:
"Beginning with the UCRT in Visual Studio 2015 and Windows 10, snprintf is no
longer identical to _snprintf. The snprintf function behavior is now C99
standard conformant." So for older versions, snprintf would potentially not
store a terminating NUL, like _snprintf but unlike sprintf_s.)
Change-Id: Ibf8b228cd1dd57428ecf85ed10e0793b78ae90c9
Reviewed-on: https://gerrit.libreoffice.org/c/cppunit/+/147383
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Often a developer debugging a failing CppUnit test wants a core dump with the
place where an uncaught exception is thrown. So if the newly introduced
CPPUNIT_PROPAGATE_EXCEPTIONS environment variable is set (to any value), disable
the DefaultProtector that would otherwise catch such exceptions (and just report
some limited information about them).
Change-Id: Iac0e98837381578759384e45829fb4e2436f450a
Reviewed-on: https://gerrit.libreoffice.org/c/cppunit/+/144074
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
LibreOffice already benefits from this (see
<https://git.libreoffice.org/core/+/2f2246d22e2a8ccbc1dc3e6f5243734a61edf270%5E!>
"external/cppunit: Run tests in deterministic order", especially as otherwise
the order in which tests happened to get run differed between --disable-lto and
--enable-lto builds.
Change-Id: I87d6d7cb0f4c2f6a0ea1ac3ba3d48b4e089eb5c7
Reviewed-on: https://gerrit.libreoffice.org/c/cppunit/+/123963
Tested-by: Stephan Bergmann <sbergman@redhat.com>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
|
|
|
|
|
| |
The help string for the option already says that it is "useful for
development", which implies it is not a good default.
Change-Id: Ic0f3bb0eaefd146ad0e5e87ff97abd821462f1a5
|
| |
|
| |
|
|
|
|
| |
There are no API changes yet so we can just release an updated version.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
Change-Id: Ie72531ebdc858a0122886a7a3cfe49185dc0ff76
Reviewed-on: https://gerrit.libreoffice.org/c/cppunit/+/101688
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
| |
|
|
|
|
|
|
|
| |
Change-Id: I36ceddb5a8a1f8e96b36a2e851338366cda1808d
Reviewed-on: https://gerrit.libreoffice.org/85662
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
|
|
|
|
|
| |
Change-Id: I6ebc31181ccfe379be96e979cda3717a991cf513
Reviewed-on: https://gerrit.libreoffice.org/85663
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
Change-Id: I8a0d7a0b51b5c537dbcfa8fdd34e816605b1f32e
Reviewed-on: https://gerrit.libreoffice.org/71573
Reviewed-by: Tomáš Chvátal <tchvatal@suse.cz>
Tested-by: Tomáš Chvátal <tchvatal@suse.cz>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
doc/other_documentation.dox:
* add instructions to build documentation
* replace a dead link with it's archived version on WaybackMachine
doc/Money.dox:
* replace `configure.in` with `configure.ac`
* update MoneyApp.cpp with the version at `examples/money/MoneyApp.cpp`
* remove all occurrences of `#include "stdafx.h"` (stdafx.h does not exist in this example)
* replace all occurrences of CppUnit:: with CPPUNIT_NS::
* Undo a fix in commit 652d51e653568fbf652a248c7b9e01e72e6ec50f
* This commit fixed the following error:
```
Money( double amount, std::string currency )
: m_amount( amount )
, m_currency( m_currency )
```
But this error was introduced on purpose, so that the user can see that an assertion failed
* fix some spelling errors
* other minor edits
examples/money/MoneyTest.h:
* Updated to match doc/Money.dox
Change-Id: I4e90b1a3afadab94f7112c36fcb0d1c467169269
Reviewed-on: https://gerrit.libreoffice.org/67787
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
|
| |
|
|
|
|
|
|
|
|
|
| |
For more info, see: https://www.gnu.org/software/automake/manual/automake.html#Modernize-AM_005fINIT_005fAUTOMAKE-invocation
Change-Id: If27065b6d9c9245717ca24e0c0e1fa4c29bc3e5c
Reviewed-on: https://gerrit.libreoffice.org/67808
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
|
|
| |
belongs to the if body
|
|
|
|
|
|
|
|
|
|
| |
Provide an extension trait message_to_string that allows client code
to call ASSERT_MESSAGE with their own string types.
Also provide a default extension trait that accepts std::ostream
messages.
Change-Id: I516f97063c34d86bc91c40e0ec147d5393e7b6ea
|
|
|
|
|
|
|
|
|
|
|
|
| |
This helps avoid issues like <https://gerrit.libreoffice.org/plugins/gitiles/
core/+/9808486a89c6368f836579f8d8c0dda63fd0063c%5E%21> "Avoid
-Werror,-Wimplicit-fallthrough with clang-cl ...where CPPUNIT_FAIL is marked as
noreturn only for __GNUC__".
Change-Id: Idb33af7375f103f2dd7a7b4c3dbf20ce731b17ad
Reviewed-on: https://gerrit.libreoffice.org/67247
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
This is similar to googletest's TEST_F() macro, allows to avoid spelling
out the test name in a suite only once, not 3 times.
Change-Id: I90804d271d046d8158678617d8db75ed6ca7c420
Reviewed-on: https://gerrit.libreoffice.org/61732
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
...when an implicitly-defined copy function is used that may in a future C++
standard be defined as deleted because of the user-declared destructor. Just
declare all the four copy/move functions as defaulted for a consistent
interface.
(Originally addressed with LibreOffice commit
<https://gerrit.libreoffice.org/58042> "external/cppunit: silence
-Werror=deprecated-copy (GCC trunk towards GCC 9)".)
Change-Id: Ie38ac3a613e6fbc2e4045cb5b14c3f6d167c79c5
Reviewed-on: https://gerrit.libreoffice.org/58690
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The cppunit2junit XSL stylesheet has a hardcoded name for the testsuite.
Instead of putting a meaningless name, allow users to parametrize this
name when doing the transformation.
Change-Id: I2d0ebaf92cd0aa4d8ff3b3a2c423f4a8edd94807
Signed-off-by: Simon Barth <Simon.Pe.Barth@gmail.com>
Reviewed-on: https://gerrit.libreoffice.org/46357
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
|
|
|
|
|
|
| |
Change-Id: If42a50386402b7a601268cf8db80236c147009c1
Reviewed-on: https://gerrit.libreoffice.org/42206
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
|
|
|
|
|
|
|
|
|
|
|
| |
...instead of as characters, as those values often represent numerical values
(e.g., sal_uInt8 aka unsigned char in LibreOffice), and often even have values
outside the printing ASCII range
Change-Id: I72727e98f5af616f6b0c03c57fc1119c0c11c1fc
Reviewed-on: https://gerrit.libreoffice.org/40138
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|