diff options
| author | Caolán McNamara <caolan.mcnamara@collabora.com> | 2023-10-07 15:57:19 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolan.mcnamara@collabora.com> | 2023-10-08 13:17:26 +0200 |
| commit | 889848163e8dcb32d58f85ddaf2748517a37b9b9 (patch) | |
| tree | 3963f58debe97c3224b0d7c7a0deb8fdd39dbcf1 /svx | |
| parent | 7ef3d937415185ef66e32dd3043783eddcd03db5 (diff) | |
cid#1547086 Logically dead code
since:
commit 757a73379dba01fde853443a09583d862c404531
Date: Fri Oct 6 09:31:28 2023 +0100
Related: cool#7373 add an SfxHintId::ThisIsAnSfxEventHint
luckily this is the only use of SfxHintId::NONE so easy to audit
Change-Id: Id593eb3b2c2d26c9552aa7bf6033afd4f3112120
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/157678
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Diffstat (limited to 'svx')
| -rw-r--r-- | svx/source/form/fmundo.cxx | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/svx/source/form/fmundo.cxx b/svx/source/form/fmundo.cxx index 80aea1c8a35e..ba693fb02a92 100644 --- a/svx/source/form/fmundo.cxx +++ b/svx/source/form/fmundo.cxx @@ -307,26 +307,26 @@ void FmXUndoEnvironment::Notify( SfxBroadcaster& /*rBC*/, const SfxHint& rHint ) break; } } - else if (rHint.GetId() != SfxHintId::NONE) + else if (rHint.GetId() == SfxHintId::ThisIsAnSfxEventHint) { - switch (rHint.GetId()) + switch (static_cast<const SfxEventHint&>(rHint).GetEventId()) { - case SfxHintId::Dying: - dispose(); - rModel.SetObjectShell( nullptr ); - break; - case SfxHintId::ModeChanged: + case SfxEventHintId::CreateDoc: + case SfxEventHintId::OpenDoc: ModeChanged(); break; default: break; } } - else if (rHint.GetId() == SfxHintId::ThisIsAnSfxEventHint) + else if (rHint.GetId() != SfxHintId::NONE) { - switch (static_cast<const SfxEventHint&>(rHint).GetEventId()) + switch (rHint.GetId()) { - case SfxEventHintId::CreateDoc: - case SfxEventHintId::OpenDoc: + case SfxHintId::Dying: + dispose(); + rModel.SetObjectShell( nullptr ); + break; + case SfxHintId::ModeChanged: ModeChanged(); break; default: break; |