See #4843
When this option is enabled, it requires a version of TinyEXIF newer
than 1.0.2 (which is the latest version that has been released),
specifically the change in cdcseacave/TinyEXIF@d75f772. The request to
release a new version of TinyEXIF is cdcseacave/TinyEXIF#18.
Merged run()/close() member functions again to close all docs before
destroying MainWindow even in case of an exception, using RAII
idiom (this fixes some failing tests/benchmarks).
Fixes a failing ASSERT() in ~DocObserverWidget().
We've fixed some issues resetting the ui::Theme to nullptr when we're
shutting down the App to run a new unit test later. This avoids using
a freed theme pointer.
This file can be used with --ignore-revs-file param or
blame.ignoreRevsFile config to ignore global formatting changes which
don't help to use git-blame.
This reverts commit d38565f64e4c0304b8ea2ae060c9e1c11ba5a863.
It added blank lines between one line member function definitions
inside class bodies/header files, something which is undesirable.
With the current code it makes no difference, but it will force new
definitions to be in its own block, e.g. we cannot define two
functions without a blank line between then.
Allows to format the code in better ways adjusting the comments to fit
the column limit. In few cases the comment will require to be moved to
other place (e.g. at the top of a statement rather than to the side of
it).
With this we prefer to format a function declaration as:
int function(arg1,
arg2,
arg3);
Rather than:
int function(
arg1, arg2, arg3);
Cons: This makes the definition of NewParams structures a bit worse.
With this we prefer to format function calls as:
int value = function(arg1,
arg2,
arg3);
Or:
int value =
function(arg1, arg2, arg3);
Rather than:
int value = function(
arg1, arg2, arg3);
Making PenaltyBreakAssignment bigger creates better looking
const/variable declarations, where the following term to the
assignment operator is in the same line.
This is required if a new case is added below in the future to avoid
compiler errors. It's better to always use { } if we're going to
define local variables for the case.
