cui/source/options/appearance.cxx | 3 --- 1 file changed, 3 deletions(-)
New commits: commit bef1b114eee58be202a82d45f9d0cb00529b8e03 Author: Dan Williams <[email protected]> AuthorDate: Tue Dec 16 10:32:59 2025 -0600 Commit: Hossein <[email protected]> CommitDate: Fri Dec 19 12:31:41 2025 +0100 cui: remove duplicate code The line: aCurrentEntryColor.nColor = m_xColorChangeBtn->GetSelectEntryColor(); was called twice with two different comments. Remove the line that has the inaccurate comment and leave the one that describes what's actually happening now. Change-Id: I566e8254425f4499cefa7c159468e531aaf9e355 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/195737 Tested-by: Jenkins Reviewed-by: Hossein <[email protected]> diff --git a/cui/source/options/appearance.cxx b/cui/source/options/appearance.cxx index 9304ab269f52..bee1a48ef650 100644 --- a/cui/source/options/appearance.cxx +++ b/cui/source/options/appearance.cxx @@ -352,9 +352,6 @@ IMPL_LINK_NOARG(SvxAppearanceTabPage, ColorValueChgHdl, ColorListBox&, void) if (nEntry >= WINDOWCOLOR) m_bRestartRequired = true; - // set the color in pColorConfig - aCurrentEntryColor.nColor = m_xColorChangeBtn->GetSelectEntryColor(); - // use nColor for caching the value of color in use. This avoids tedious refactoring which IMO // would use function calls to discriminate between colors. Those functions themself call some virtual functions // making the whole thing super slow (comparatively).
