diff options
author | Justin Luth <justin.luth@collabora.com> | 2023-05-03 19:03:49 -0400 |
---|---|---|
committer | Justin Luth <jluth@mail.com> | 2023-05-04 02:46:53 +0200 |
commit | 83b1f6b58a30bdb589e9ce73deef39f021aebde1 (patch) | |
tree | fc09a0cb2143cbd448e504ff6a8926eaa9f9a85f /slideshow | |
parent | a55af69baefd7965da3a7faf5af1c049fe99f9b4 (diff) |
tdf#86630 sw page number wizard: avoid crash when no GetCurrFrame
The steps to get the crash were:
-create a header (and leave the cursor in the header)
-use the wizard to create a non-mirrored footer
I had seen this crash happening occassionally during development,
but hadn't yet identified the exact steps to consistently reproduce.
Thanks to QA who tracked that down.
Although it looks/feels like a hack, it actually works amazingly well.
The simple act of SetCursorInHdFt "fixes" GetCurrFrame/Get*PageNum,
but we can't always use that because we don't know whether we
are on an odd or even or first page when we start, so Goto*Text
is the best when non-mirroring.
Change-Id: I0e2f28d9c8fd2d1a10afede885b525c42ed08af4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/151345
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
Diffstat (limited to 'slideshow')
0 files changed, 0 insertions, 0 deletions