diff options
author | Miklos Vajna <vmiklos@collabora.com> | 2025-04-30 15:00:43 +0200 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2025-05-07 09:37:18 +0200 |
commit | 0891df6b21fd95ec7c9614509d92829c0f17c353 (patch) | |
tree | 363454849c50f91abd1b1e1cbdf9bb3f53998a92 /sc/inc/cellformtmpl.hxx | |
parent | 19992b8f238dbfe398b4d7c625905c8ca9ad9845 (diff) |
sw::DocumentRedlineManager::AppendRedline() is 1158 lines long, which
makes it very hard to reason about the body of the function.
3 extractable pieces of it are the blocks which tweak the to-be-inserted
redline and possibly existing redlines of the redline table based on the
type of the to-be-inserted redline (insert, delete, format).
Extract the insert redline part into a new function, which alone is 261
lines long.
To keep this mechanical, pass all variables by non-const ref, then make 2
selected variables const, since those are only read.
Change-Id: I105524e010bab92afdf54bbba5b33dccdc5900dd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/184996
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
Diffstat (limited to 'sc/inc/cellformtmpl.hxx')
0 files changed, 0 insertions, 0 deletions