diff options
author | Platon Pronko <platon7pronko@gmail.com> | 2020-10-19 17:39:08 +0300 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-10-20 08:06:32 +0200 |
commit | 150d4c8b66a5d4e32a97a2a7e25e6fb632cfd40c (patch) | |
tree | 8180b27ba851b9517e703adfe3cb6469cb680790 /vcl/source | |
parent | 1e822e401ea8fe950c7fb62172ac61d8396c98e9 (diff) |
remove unnecessary `if (!true) return;` statement from Desktop::Init()
Seems that this specific piece of code was introduced in
commit bc66bf4608557d757555aef1f46abf8a15c3538b by Noel Grandin,
where `if (true)` wrapper was removed and `if (!true) return;`
seems to have been left over.
But the original `if (true)` statement was introduced in commit
4d258787559426e1e2a0279888eb669622889899 by Michael Meeks, where
`if ( m_aBootstrapError == BE_OK )` before starting IPC thread
was replaced with `if (true)`. I asked on IRC and they were not
sure for the reason of this change, so I will add Michael Meeks
as a reviewer of this patch, just in case.
Change-Id: I1ffb781464fa2fde152eb21bc61e4b244e8812d3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104530
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Jenkins
Diffstat (limited to 'vcl/source')
0 files changed, 0 insertions, 0 deletions