[Libreoffice-commits] core.git: cui/source solenv/clang-format

2023-11-15 Thread Samuel Mehrbrodt (via logerrit)
 cui/source/options/optaboutconfig.cxx |  364 +-
 cui/source/options/optaboutconfig.hxx |   34 +--
 solenv/clang-format/excludelist   |2 
 3 files changed, 202 insertions(+), 198 deletions(-)

New commits:
commit f271ec5cd98ae32fd643df33174136d0a91a1941
Author: Samuel Mehrbrodt 
AuthorDate: Mon Nov 13 14:22:09 2023 +0100
Commit: Samuel Mehrbrodt 
CommitDate: Thu Nov 16 08:04:31 2023 +0100

Format optaboutconfig with clang-format

Change-Id: Ia696e096f972fe722920238c3710d20a3f03b221
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159374
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt 

diff --git a/cui/source/options/optaboutconfig.cxx 
b/cui/source/options/optaboutconfig.cxx
index 11806faa8477..362af44cfd7d 100644
--- a/cui/source/options/optaboutconfig.cxx
+++ b/cui/source/options/optaboutconfig.cxx
@@ -7,37 +7,37 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
 
+#include "optaboutconfig.hxx"
 #include 
 #include 
-#include "optaboutconfig.hxx"
 
-#include 
-#include 
-#include 
-#include 
 #include 
 #include 
 #include 
 #include 
 #include 
 #include 
+#include 
+#include 
 #include 
 #include 
-#include 
+#include 
 #include 
 #include 
 #include 
-#include 
-#include 
 #include 
+#include 
+#include 
+#include 
+#include 
+#include 
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
-#include 
-#include 
 
 #include 
 #include 
@@ -52,15 +52,16 @@ using namespace com::sun::star::container;
 
 struct Prop_Impl
 {
-OUStringName;
-OUStringProperty;
-Any Value;
-
-Prop_Impl( OUString sName, OUString sProperty, Any aValue )
-: Name(std::move( sName ))
-, Property(std::move( sProperty ))
-, Value(std::move( aValue ))
-{}
+OUString Name;
+OUString Property;
+Any Value;
+
+Prop_Impl(OUString sName, OUString sProperty, Any aValue)
+: Name(std::move(sName))
+, Property(std::move(sProperty))
+, Value(std::move(aValue))
+{
+}
 };
 
 struct UserData
@@ -72,20 +73,22 @@ struct UserData
 int aLineage;
 Reference aXNameAccess;
 
-explicit UserData( OUString aPropertyPath, OUString aTooltip, bool 
isReadOnly )
-: bIsPropertyPath( true )
-, bIsReadOnly( isReadOnly )
+explicit UserData(OUString aPropertyPath, OUString aTooltip, bool 
isReadOnly)
+: bIsPropertyPath(true)
+, bIsReadOnly(isReadOnly)
 , sPropertyPath(std::move(aPropertyPath))
 , sTooltip(std::move(aTooltip))
 , aLineage(0)
-{}
+{
+}
 
-explicit UserData( Reference const & rXNameAccess, int rIndex 
)
-: bIsPropertyPath( false )
-, bIsReadOnly( false )
+explicit UserData(Reference const& rXNameAccess, int rIndex)
+: bIsPropertyPath(false)
+, bIsReadOnly(false)
 , aLineage(rIndex)
-, aXNameAccess( rXNameAccess )
-{}
+, aXNameAccess(rXNameAccess)
+{
+}
 };
 
 CuiAboutConfigTabPage::CuiAboutConfigTabPage(weld::Window* pParent)
@@ -103,24 +106,20 @@ 
CuiAboutConfigTabPage::CuiAboutConfigTabPage(weld::Window* pParent)
  m_xPrefBox->get_height_rows(23));
 m_xPrefBox->connect_column_clicked(LINK(this, CuiAboutConfigTabPage, 
HeaderBarClick));
 
-m_xEditBtn->connect_clicked(LINK( this, CuiAboutConfigTabPage, 
StandardHdl_Impl));
-m_xResetBtn->connect_clicked(LINK( this, CuiAboutConfigTabPage, 
ResetBtnHdl_Impl));
+m_xEditBtn->connect_clicked(LINK(this, CuiAboutConfigTabPage, 
StandardHdl_Impl));
+m_xResetBtn->connect_clicked(LINK(this, CuiAboutConfigTabPage, 
ResetBtnHdl_Impl));
 m_xPrefBox->connect_row_activated(LINK(this, CuiAboutConfigTabPage, 
DoubleClickHdl_Impl));
 m_xPrefBox->connect_expanding(LINK(this, CuiAboutConfigTabPage, 
ExpandingHdl_Impl));
 m_xSearchBtn->connect_clicked(LINK(this, CuiAboutConfigTabPage, 
SearchHdl_Impl));
 
 m_options.AlgorithmType2 = util::SearchAlgorithms2::ABSOLUTE;
 m_options.transliterateFlags |= TransliterationFlags::IGNORE_CASE;
-m_options.searchFlag |= (util::SearchFlags::REG_NOT_BEGINOFLINE |
-util::SearchFlags::REG_NOT_ENDOFLINE);
+m_options.searchFlag
+|= (util::SearchFlags::REG_NOT_BEGINOFLINE | 
util::SearchFlags::REG_NOT_ENDOFLINE);
 
 float fWidth = m_xPrefBox->get_approximate_digit_width();
-std::vector aWidths
-{
-o3tl::narrowing(fWidth * 65),
-o3tl::narrowing(fWidth * 20),
-o3tl::narrowing(fWidth * 8)
-};
+std::vector aWidths{ o3tl::narrowing(fWidth * 65), 
o3tl::narrowing(fWidth * 20),
+  o3tl::narrowing(fWidth * 8) };
 m_xPrefBox->set_column_fixed_widths(aWidths);
 
 m_xPrefBox->connect_query_tooltip(LINK(this, CuiAboutConfigTabPage, 
QueryTooltip));
@@ -128,7 +127,7 @@ 

[Libreoffice-commits] core.git: cui/source solenv/clang-format

2020-10-18 Thread Muhammet Kara (via logerrit)
 cui/source/customize/SvxToolbarConfigPage.cxx |  474 --
 cui/source/inc/SvxToolbarConfigPage.hxx   |   53 +-
 solenv/clang-format/excludelist   |2 
 3 files changed, 255 insertions(+), 274 deletions(-)

New commits:
commit 35fa77d0137350e69b17418e2bc33f1fa0be2efb
Author: Muhammet Kara 
AuthorDate: Sun Oct 18 19:48:16 2020 +0300
Commit: Muhammet Kara 
CommitDate: Sun Oct 18 20:57:51 2020 +0200

clang-format customize/SvxToolbarConfigPage

Change-Id: If9b3fcbaa8e900d73c8dd0e727b4059c3b30ac01
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104485
Tested-by: Jenkins
Reviewed-by: Muhammet Kara 

diff --git a/cui/source/customize/SvxToolbarConfigPage.cxx 
b/cui/source/customize/SvxToolbarConfigPage.cxx
index 740fa426feb0..57f78dcd6161 100644
--- a/cui/source/customize/SvxToolbarConfigPage.cxx
+++ b/cui/source/customize/SvxToolbarConfigPage.cxx
@@ -42,15 +42,19 @@
 
 #include 
 
-SvxToolbarConfigPage::SvxToolbarConfigPage(weld::Container* pPage, 
weld::DialogController* pController, const SfxItemSet& rSet)
+SvxToolbarConfigPage::SvxToolbarConfigPage(weld::Container* pPage,
+   weld::DialogController* pController,
+   const SfxItemSet& rSet)
 : SvxConfigPage(pPage, pController, rSet)
 {
 m_xGearBtn = m_xBuilder->weld_menu_button("toolbargearbtn");
 m_xGearBtn->show();
 m_xContainer->set_help_id(HID_SVX_CONFIG_TOOLBAR);
 
-m_xContentsListBox.reset(new 
SvxToolbarEntriesListBox(m_xBuilder->weld_tree_view("toolcontents"), this));
-m_xDropTargetHelper.reset(new SvxConfigPageFunctionDropTarget(*this, 
m_xContentsListBox->get_widget()));
+m_xContentsListBox.reset(
+new 
SvxToolbarEntriesListBox(m_xBuilder->weld_tree_view("toolcontents"), this));
+m_xDropTargetHelper.reset(
+new SvxConfigPageFunctionDropTarget(*this, 
m_xContentsListBox->get_widget()));
 
 weld::TreeView& rTreeView = m_xContentsListBox->get_widget();
 Size aSize(m_xFunctions->get_size_request());
@@ -58,44 +62,38 @@ SvxToolbarConfigPage::SvxToolbarConfigPage(weld::Container* 
pPage, weld::DialogC
 
 rTreeView.set_hexpand(true);
 rTreeView.set_vexpand(true);
-rTreeView.set_help_id( HID_SVX_CONFIG_TOOLBAR_CONTENTS );
+rTreeView.set_help_id(HID_SVX_CONFIG_TOOLBAR_CONTENTS);
 rTreeView.show();
 
-rTreeView.connect_changed(
-LINK( this, SvxToolbarConfigPage, SelectToolbarEntry ) );
-rTreeView.connect_popup_menu( LINK( this, SvxToolbarConfigPage, 
ContentContextMenuHdl ) );
+rTreeView.connect_changed(LINK(this, SvxToolbarConfigPage, 
SelectToolbarEntry));
+rTreeView.connect_popup_menu(LINK(this, SvxToolbarConfigPage, 
ContentContextMenuHdl));
 
 m_xFunctions->get_widget().connect_popup_menu(
-LINK( this, SvxToolbarConfigPage, FunctionContextMenuHdl ) );
+LINK(this, SvxToolbarConfigPage, FunctionContextMenuHdl));
 
-m_xTopLevelListBox->set_help_id ( HID_SVX_TOPLEVELLISTBOX );
-m_xSaveInListBox->set_help_id( HID_SVX_SAVE_IN );
-m_xMoveUpButton->set_help_id( HID_SVX_UP_TOOLBAR_ITEM );
-m_xMoveDownButton->set_help_id( HID_SVX_DOWN_TOOLBAR_ITEM );
-m_xDescriptionField->set_help_id ( HID_SVX_DESCFIELD );
+m_xTopLevelListBox->set_help_id(HID_SVX_TOPLEVELLISTBOX);
+m_xSaveInListBox->set_help_id(HID_SVX_SAVE_IN);
+m_xMoveUpButton->set_help_id(HID_SVX_UP_TOOLBAR_ITEM);
+m_xMoveDownButton->set_help_id(HID_SVX_DOWN_TOOLBAR_ITEM);
+m_xDescriptionField->set_help_id(HID_SVX_DESCFIELD);
 
-m_xCommandCategoryListBox->connect_changed(
-LINK( this, SvxToolbarConfigPage, SelectCategory ) );
+m_xCommandCategoryListBox->connect_changed(LINK(this, 
SvxToolbarConfigPage, SelectCategory));
 
-m_xGearBtn->connect_selected(
-LINK( this, SvxToolbarConfigPage, GearHdl ) );
+m_xGearBtn->connect_selected(LINK(this, SvxToolbarConfigPage, GearHdl));
 
-m_xMoveUpButton->connect_clicked( LINK( this, SvxToolbarConfigPage, 
MoveHdl) );
-m_xMoveDownButton->connect_clicked( LINK( this, SvxToolbarConfigPage, 
MoveHdl) );
+m_xMoveUpButton->connect_clicked(LINK(this, SvxToolbarConfigPage, 
MoveHdl));
+m_xMoveDownButton->connect_clicked(LINK(this, SvxToolbarConfigPage, 
MoveHdl));
 // Always enable Up and Down buttons
 // added for issue i53677 by shizhoubo
 m_xMoveDownButton->set_sensitive(true);
 m_xMoveUpButton->set_sensitive(true);
 
-m_xAddCommandButton->connect_clicked( LINK( this, SvxToolbarConfigPage, 
AddCommandHdl ) );
-m_xRemoveCommandButton->connect_clicked( LINK( this, SvxToolbarConfigPage, 
RemoveCommandHdl ) );
+m_xAddCommandButton->connect_clicked(LINK(this, SvxToolbarConfigPage, 
AddCommandHdl));
+m_xRemoveCommandButton->connect_clicked(LINK(this, SvxToolbarConfigPage, 
RemoveCommandHdl));
 
-m_xInsertBtn->connect_selected(
-LINK( this, SvxToolbarConfigPage, InsertHdl ) );
-

[Libreoffice-commits] core.git: cui/source solenv/clang-format

2020-10-18 Thread Muhammet Kara (via logerrit)
 cui/source/customize/SvxMenuConfigPage.cxx |  245 ++---
 cui/source/inc/SvxMenuConfigPage.hxx   |   47 ++---
 solenv/clang-format/excludelist|2 
 3 files changed, 144 insertions(+), 150 deletions(-)

New commits:
commit e190529f40988bbeb23b7b3abaeca6900f96213f
Author: Muhammet Kara 
AuthorDate: Sun Oct 18 18:55:37 2020 +0300
Commit: Muhammet Kara 
CommitDate: Sun Oct 18 20:53:59 2020 +0200

clang-format customize/SvxMenuConfigPage

Change-Id: I9ad8012d31c8e727fa4c8c840f15a93ca47541cc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104483
Tested-by: Jenkins
Reviewed-by: Muhammet Kara 

diff --git a/cui/source/customize/SvxMenuConfigPage.cxx 
b/cui/source/customize/SvxMenuConfigPage.cxx
index d3fed61015a2..3181a78bbb07 100644
--- a/cui/source/customize/SvxMenuConfigPage.cxx
+++ b/cui/source/customize/SvxMenuConfigPage.cxx
@@ -36,13 +36,15 @@
 
 #include 
 
-SvxMenuConfigPage::SvxMenuConfigPage(weld::Container* pPage, 
weld::DialogController* pController, const SfxItemSet& rSet, bool bIsMenuBar)
+SvxMenuConfigPage::SvxMenuConfigPage(weld::Container* pPage, 
weld::DialogController* pController,
+ const SfxItemSet& rSet, bool bIsMenuBar)
 : SvxConfigPage(pPage, pController, rSet)
 , m_bIsMenuBar(bIsMenuBar)
 {
 m_xGearBtn = m_xBuilder->weld_menu_button("menugearbtn");
 m_xGearBtn->show();
-m_xContentsListBox.reset(new 
SvxMenuEntriesListBox(m_xBuilder->weld_tree_view("menucontents"), this));
+m_xContentsListBox.reset(
+new SvxMenuEntriesListBox(m_xBuilder->weld_tree_view("menucontents"), 
this));
 weld::TreeView& rTreeView = m_xContentsListBox->get_widget();
 m_xDropTargetHelper.reset(new SvxConfigPageFunctionDropTarget(*this, 
rTreeView));
 rTreeView.connect_size_allocate(LINK(this, SvxMenuConfigPage, 
MenuEntriesSizeAllocHdl));
@@ -53,36 +55,32 @@ SvxMenuConfigPage::SvxMenuConfigPage(weld::Container* 
pPage, weld::DialogControl
 rTreeView.set_vexpand(true);
 rTreeView.show();
 
-rTreeView.connect_changed(
-LINK( this, SvxMenuConfigPage, SelectMenuEntry ) );
-rTreeView.connect_popup_menu( LINK( this, SvxMenuConfigPage, 
ContentContextMenuHdl ) );
+rTreeView.connect_changed(LINK(this, SvxMenuConfigPage, SelectMenuEntry));
+rTreeView.connect_popup_menu(LINK(this, SvxMenuConfigPage, 
ContentContextMenuHdl));
 
 m_xFunctions->get_widget().connect_popup_menu(
-LINK( this, SvxMenuConfigPage, FunctionContextMenuHdl ) );
+LINK(this, SvxMenuConfigPage, FunctionContextMenuHdl));
 
 m_xGearBtn->connect_selected(LINK(this, SvxMenuConfigPage, GearHdl));
 
 m_xCommandCategoryListBox->connect_changed(LINK(this, SvxMenuConfigPage, 
SelectCategory));
 
-m_xMoveUpButton->connect_clicked( LINK( this, SvxConfigPage, MoveHdl) );
-m_xMoveDownButton->connect_clicked( LINK( this, SvxConfigPage, MoveHdl) );
+m_xMoveUpButton->connect_clicked(LINK(this, SvxConfigPage, MoveHdl));
+m_xMoveDownButton->connect_clicked(LINK(this, SvxConfigPage, MoveHdl));
 
-m_xAddCommandButton->connect_clicked( LINK( this, SvxMenuConfigPage, 
AddCommandHdl ) );
-m_xRemoveCommandButton->connect_clicked( LINK( this, SvxMenuConfigPage, 
RemoveCommandHdl ) );
+m_xAddCommandButton->connect_clicked(LINK(this, SvxMenuConfigPage, 
AddCommandHdl));
+m_xRemoveCommandButton->connect_clicked(LINK(this, SvxMenuConfigPage, 
RemoveCommandHdl));
 
-m_xInsertBtn->connect_selected(
-LINK( this, SvxMenuConfigPage, InsertHdl ) );
-m_xModifyBtn->connect_selected(
-LINK( this, SvxMenuConfigPage, ModifyItemHdl ) );
-m_xResetBtn->connect_clicked(
-LINK( this, SvxMenuConfigPage, ResetMenuHdl ) );
+m_xInsertBtn->connect_selected(LINK(this, SvxMenuConfigPage, InsertHdl));
+m_xModifyBtn->connect_selected(LINK(this, SvxMenuConfigPage, 
ModifyItemHdl));
+m_xResetBtn->connect_clicked(LINK(this, SvxMenuConfigPage, ResetMenuHdl));
 
 // These operations are not possible on menus/context menus yet
 m_xModifyBtn->remove_item("changeIcon");
 m_xModifyBtn->remove_item("resetIcon");
 m_xModifyBtn->remove_item("restoreItem");
 
-if ( !bIsMenuBar )
+if (!bIsMenuBar)
 {
 //TODO: Remove this when the gear button is implemented for context 
menus
 m_xGearBtn->set_sensitive(false);
@@ -102,7 +100,8 @@ void SvxMenuConfigPage::ListModified()
 pEntries->clear();
 
 for (int i = 0; i < m_xContentsListBox->n_children(); ++i)
-
pEntries->push_back(reinterpret_cast(m_xContentsListBox->get_id(i).toInt64()));
+pEntries->push_back(
+
reinterpret_cast(m_xContentsListBox->get_id(i).toInt64()));
 
 GetSaveInData()->SetModified();
 GetTopLevelSelection()->SetModified();
@@ -140,36 +139,32 @@ void SvxMenuConfigPage::Init()
 m_xTopLevelListBox->set_active(0);
 SelectElement();
 
-m_xCommandCategoryListBox->Init(
-

[Libreoffice-commits] core.git: cui/source solenv/clang-format

2020-10-18 Thread Muhammet Kara (via logerrit)
 cui/source/customize/SvxConfigPageHelper.cxx |  224 ---
 cui/source/inc/SvxConfigPageHelper.hxx   |   77 +++--
 solenv/clang-format/excludelist  |2 
 3 files changed, 132 insertions(+), 171 deletions(-)

New commits:
commit 11485a500196af7a2533787f4faf32e14693f480
Author: Muhammet Kara 
AuthorDate: Sun Oct 18 17:15:45 2020 +0300
Commit: Muhammet Kara 
CommitDate: Sun Oct 18 17:43:53 2020 +0200

clang-format customize/SvxConfigPageHelper

Change-Id: Ia802bfc4a6cd8d970fbb3c37a55959729ea19257
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104481
Tested-by: Jenkins
Reviewed-by: Muhammet Kara 

diff --git a/cui/source/customize/SvxConfigPageHelper.cxx 
b/cui/source/customize/SvxConfigPageHelper.cxx
index 033219f4d9a0..d54bb7cdaf1a 100644
--- a/cui/source/customize/SvxConfigPageHelper.cxx
+++ b/cui/source/customize/SvxConfigPageHelper.cxx
@@ -27,52 +27,43 @@
 #include 
 #include 
 
-static sal_Int16 theImageType =
-css::ui::ImageType::COLOR_NORMAL |
-css::ui::ImageType::SIZE_DEFAULT;
+static sal_Int16 theImageType = css::ui::ImageType::COLOR_NORMAL | 
css::ui::ImageType::SIZE_DEFAULT;
 
-void SvxConfigPageHelper::RemoveEntry( SvxEntries* pEntries, SvxConfigEntry 
const * pChildEntry )
+void SvxConfigPageHelper::RemoveEntry(SvxEntries* pEntries, SvxConfigEntry 
const* pChildEntry)
 {
 SvxEntries::iterator iter = pEntries->begin();
 
-while ( iter != pEntries->end() )
+while (iter != pEntries->end())
 {
-if ( pChildEntry == *iter )
+if (pChildEntry == *iter)
 {
-pEntries->erase( iter );
+pEntries->erase(iter);
 break;
 }
 ++iter;
 }
 }
 
-OUString SvxConfigPageHelper::replaceSaveInName( const OUString& rMessage, 
const OUString& rSaveInName )
+OUString SvxConfigPageHelper::replaceSaveInName(const OUString& rMessage,
+const OUString& rSaveInName)
 {
 OUString name = rMessage.replaceFirst("%SAVE IN SELECTION%", rSaveInName);
 
 return name;
 }
 
-OUString SvxConfigPageHelper::stripHotKey( const OUString& str )
-{
-return str.replaceFirst("~", "");
-}
+OUString SvxConfigPageHelper::stripHotKey(const OUString& str) { return 
str.replaceFirst("~", ""); }
 
-OUString SvxConfigPageHelper::replaceSixteen( const OUString& str, sal_Int32 
nReplacement )
+OUString SvxConfigPageHelper::replaceSixteen(const OUString& str, sal_Int32 
nReplacement)
 {
-return str.replaceAll( OUString::number( 16 ), OUString::number( 
nReplacement ));
+return str.replaceAll(OUString::number(16), 
OUString::number(nReplacement));
 }
 
-sal_Int16 SvxConfigPageHelper::GetImageType()
-{
-return theImageType;
-}
+sal_Int16 SvxConfigPageHelper::GetImageType() { return theImageType; }
 
 void SvxConfigPageHelper::InitImageType()
 {
-theImageType =
-css::ui::ImageType::COLOR_NORMAL |
-css::ui::ImageType::SIZE_DEFAULT;
+theImageType = css::ui::ImageType::COLOR_NORMAL | 
css::ui::ImageType::SIZE_DEFAULT;
 
 if (SvtMiscOptions().GetCurrentSymbolsSize() == SFX_SYMBOLS_SIZE_LARGE)
 {
@@ -84,30 +75,29 @@ void SvxConfigPageHelper::InitImageType()
 }
 }
 
-css::uno::Reference< css::graphic::XGraphic > SvxConfigPageHelper::GetGraphic(
-const css::uno::Reference< css::ui::XImageManager >& xImageManager,
-const OUString& rCommandURL )
+css::uno::Reference
+SvxConfigPageHelper::GetGraphic(const 
css::uno::Reference& xImageManager,
+const OUString& rCommandURL)
 {
-css::uno::Reference< css::graphic::XGraphic > result;
+css::uno::Reference result;
 
-if ( xImageManager.is() )
+if (xImageManager.is())
 {
 // TODO handle large graphics
-css::uno::Sequence< css::uno::Reference< css::graphic::XGraphic > > 
aGraphicSeq;
+css::uno::Sequence> 
aGraphicSeq;
 
-css::uno::Sequence aImageCmdSeq { rCommandURL };
+css::uno::Sequence aImageCmdSeq{ rCommandURL };
 
 try
 {
-aGraphicSeq =
-xImageManager->getImages( GetImageType(), aImageCmdSeq );
+aGraphicSeq = xImageManager->getImages(GetImageType(), 
aImageCmdSeq);
 
-if ( aGraphicSeq.hasElements() )
+if (aGraphicSeq.hasElements())
 {
-result =  aGraphicSeq[0];
+result = aGraphicSeq[0];
 }
 }
-catch ( css::uno::Exception& )
+catch (css::uno::Exception&)
 {
 // will return empty XGraphic
 }
@@ -116,22 +106,19 @@ css::uno::Reference< css::graphic::XGraphic > 
SvxConfigPageHelper::GetGraphic(
 return result;
 }
 
-OUString
-SvxConfigPageHelper::generateCustomName(
-const OUString& prefix,
-SvxEntries* entries,
-sal_Int32 suffix /*= 1*/ )
+OUString SvxConfigPageHelper::generateCustomName(const OUString& prefix, 
SvxEntries* entries,
+  

[Libreoffice-commits] core.git: cui/source solenv/clang-format

2020-10-18 Thread Muhammet Kara (via logerrit)
 cui/source/customize/CommandCategoryListBox.cxx |  309 +---
 cui/source/inc/CommandCategoryListBox.hxx   |   50 ++-
 solenv/clang-format/excludelist |2 
 3 files changed, 196 insertions(+), 165 deletions(-)

New commits:
commit 2242774bc8a35cda736d65d0f4b647cf394ca6ee
Author: Muhammet Kara 
AuthorDate: Sun Oct 18 16:21:59 2020 +0300
Commit: Muhammet Kara 
CommitDate: Sun Oct 18 16:08:31 2020 +0200

clang-format CommandCategoryListBox

Change-Id: If7ae64a0b9be01b3fa73310a4d9834ae3c151c10
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104479
Reviewed-by: Yusuf Keten 
Reviewed-by: Muhammet Kara 
Tested-by: Jenkins

diff --git a/cui/source/customize/CommandCategoryListBox.cxx 
b/cui/source/customize/CommandCategoryListBox.cxx
index 1337c3dd48dd..5b8903f3d199 100644
--- a/cui/source/customize/CommandCategoryListBox.cxx
+++ b/cui/source/customize/CommandCategoryListBox.cxx
@@ -51,39 +51,36 @@
 #include  //for SaveInData
 
 CommandCategoryListBox::CommandCategoryListBox(std::unique_ptr 
xControl)
-: pStylesInfo( nullptr )
+: pStylesInfo(nullptr)
 , m_xControl(std::move(xControl))
 {
 //Initialize search util
 m_searchOptions.AlgorithmType2 = css::util::SearchAlgorithms2::ABSOLUTE;
 m_searchOptions.transliterateFlags |= TransliterationFlags::IGNORE_CASE;
 m_searchOptions.searchFlag |= (css::util::SearchFlags::REG_NOT_BEGINOFLINE
-| css::util::SearchFlags::REG_NOT_ENDOFLINE);
+   | 
css::util::SearchFlags::REG_NOT_ENDOFLINE);
 }
 
-CommandCategoryListBox::~CommandCategoryListBox()
-{
-ClearAll();
-}
+CommandCategoryListBox::~CommandCategoryListBox() { ClearAll(); }
 
 void CommandCategoryListBox::ClearAll()
 {
 // Clear objects from m_aGroupInfo vector to avoid memory leak
-for (const auto & It : m_aGroupInfo)
+for (const auto& It : m_aGroupInfo)
 {
-if ( It->nKind == SfxCfgKind::GROUP_STYLES && It->pObject )
+if (It->nKind == SfxCfgKind::GROUP_STYLES && It->pObject)
 {
 SfxStyleInfo_Impl* pStyle = 
static_cast(It->pObject);
 delete pStyle;
 }
-else if ( It->nKind == SfxCfgKind::FUNCTION_SCRIPT && It->pObject )
+else if (It->nKind == SfxCfgKind::FUNCTION_SCRIPT && It->pObject)
 {
 OUString* pScriptURI = static_cast(It->pObject);
 delete pScriptURI;
 }
-else if ( It->nKind == SfxCfgKind::GROUP_SCRIPTCONTAINER && 
It->pObject)
+else if (It->nKind == SfxCfgKind::GROUP_SCRIPTCONTAINER && It->pObject)
 {
-css::uno::XInterface* xi = static_cast(It->pObject);
+css::uno::XInterface* xi = 
static_cast(It->pObject);
 if (xi != nullptr)
 {
 xi->release();
@@ -95,10 +92,9 @@ void CommandCategoryListBox::ClearAll()
 m_xControl->clear();
 }
 
-void CommandCategoryListBox::Init(
-const css::uno::Reference< css::uno::XComponentContext >& xContext,
-const css::uno::Reference< css::frame::XFrame >& xFrame,
-const OUString& sModuleLongName)
+void CommandCategoryListBox::Init(const 
css::uno::Reference& xContext,
+  const 
css::uno::Reference& xFrame,
+  const OUString& sModuleLongName)
 {
 // User will not see incomplete UI
 m_xControl->freeze();
@@ -108,9 +104,10 @@ void CommandCategoryListBox::Init(
 m_xFrame = xFrame;
 
 m_sModuleLongName = sModuleLongName;
-m_xGlobalCategoryInfo = css::ui::theUICategoryDescription::get( m_xContext 
);
-
m_xModuleCategoryInfo.set(m_xGlobalCategoryInfo->getByName(m_sModuleLongName), 
css::uno::UNO_QUERY_THROW);
-m_xUICmdDescription   = css::frame::theUICommandDescription::get( 
m_xContext );
+m_xGlobalCategoryInfo = css::ui::theUICategoryDescription::get(m_xContext);
+
m_xModuleCategoryInfo.set(m_xGlobalCategoryInfo->getByName(m_sModuleLongName),
+  css::uno::UNO_QUERY_THROW);
+m_xUICmdDescription = css::frame::theUICommandDescription::get(m_xContext);
 
 // Support style commands
 css::uno::Reference xController;
@@ -125,16 +122,20 @@ void CommandCategoryListBox::Init(
 
 try
 {
-css::uno::Reference< css::frame::XDispatchInformationProvider > 
xProvider(m_xFrame, css::uno::UNO_QUERY_THROW);
-css::uno::Sequence< sal_Int16 > lGroups = 
xProvider->getSupportedCommandGroups();
+css::uno::Reference 
xProvider(
+m_xFrame, css::uno::UNO_QUERY_THROW);
+css::uno::Sequence lGroups = 
xProvider->getSupportedCommandGroups();
 
 sal_Int32 nGroupsLength = lGroups.getLength();
 
-if ( nGroupsLength > 0 )
+if (nGroupsLength > 0)
 {
 // Add the category of "All commands"
-m_aGroupInfo.push_back( std::make_unique( 
SfxCfgKind::GROUP_ALLFUNCTIONS, 0 ) 

[Libreoffice-commits] core.git: cui/source solenv/clang-format

2020-03-07 Thread Muhammet Kara (via logerrit)
 cui/source/customize/acccfg.cxx | 1645 
 cui/source/inc/acccfg.hxx   |  149 +--
 solenv/clang-format/blacklist   |2 
 3 files changed, 900 insertions(+), 896 deletions(-)

New commits:
commit 472d5cfb11bc68779f9e1754a22e4d93725ea255
Author: Muhammet Kara 
AuthorDate: Sat Mar 7 19:16:24 2020 +0300
Commit: Muhammet Kara 
CommitDate: Sat Mar 7 21:57:08 2020 +0100

remove acccfg.{cxx,hxx} from clang-format blacklist

Change-Id: Iaa85fe6332f057642dfd01eee7a1d2b1e27c49b3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90168
Tested-by: Jenkins
Reviewed-by: Muhammet Kara 

diff --git a/cui/source/customize/acccfg.cxx b/cui/source/customize/acccfg.cxx
index 98f67d601025..7e0a4820fc2e 100644
--- a/cui/source/customize/acccfg.cxx
+++ b/cui/source/customize/acccfg.cxx
@@ -17,7 +17,6 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  */
 
-
 // include own files
 
 #include 
@@ -35,7 +34,6 @@
 #include 
 #include 
 
-
 // include interface declarations
 #include 
 #include 
@@ -67,713 +65,712 @@
 
 using namespace css;
 
-static const char FOLDERNAME_UICONFIG   [] = "Configurations2";
-
-static const char MEDIATYPE_PROPNAME[] = "MediaType";
-
-static const sal_uInt16 KEYCODE_ARRAY[] =
-{
-KEY_F1,
-KEY_F2,
-KEY_F3,
-KEY_F4,
-KEY_F5,
-KEY_F6,
-KEY_F7,
-KEY_F8,
-KEY_F9,
-KEY_F10,
-KEY_F11,
-KEY_F12,
-KEY_F13,
-KEY_F14,
-KEY_F15,
-KEY_F16,
-
-KEY_DOWN,
-KEY_UP,
-KEY_LEFT,
-KEY_RIGHT,
-KEY_HOME,
-KEY_END,
-KEY_PAGEUP,
-KEY_PAGEDOWN,
-KEY_RETURN,
-KEY_ESCAPE,
-KEY_BACKSPACE,
-KEY_INSERT,
-KEY_DELETE,
-
-KEY_OPEN,
-KEY_CUT,
-KEY_COPY,
-KEY_PASTE,
-KEY_UNDO,
-KEY_REPEAT,
-KEY_FIND,
-KEY_PROPERTIES,
-KEY_FRONT,
-KEY_CONTEXTMENU,
-KEY_MENU,
-KEY_HELP,
-
-KEY_SHIFT | KEY_F1,
-KEY_SHIFT | KEY_F2,
-KEY_SHIFT | KEY_F3,
-KEY_SHIFT | KEY_F4,
-KEY_SHIFT | KEY_F5,
-KEY_SHIFT | KEY_F6,
-KEY_SHIFT | KEY_F7,
-KEY_SHIFT | KEY_F8,
-KEY_SHIFT | KEY_F9,
-KEY_SHIFT | KEY_F10,
-KEY_SHIFT | KEY_F11,
-KEY_SHIFT | KEY_F12,
-KEY_SHIFT | KEY_F13,
-KEY_SHIFT | KEY_F14,
-KEY_SHIFT | KEY_F15,
-KEY_SHIFT | KEY_F16,
-
-KEY_SHIFT | KEY_DOWN,
-KEY_SHIFT | KEY_UP,
-KEY_SHIFT | KEY_LEFT,
-KEY_SHIFT | KEY_RIGHT,
-KEY_SHIFT | KEY_HOME,
-KEY_SHIFT | KEY_END,
-KEY_SHIFT | KEY_PAGEUP,
-KEY_SHIFT | KEY_PAGEDOWN,
-KEY_SHIFT | KEY_RETURN,
-KEY_SHIFT | KEY_SPACE,
-KEY_SHIFT | KEY_ESCAPE,
-KEY_SHIFT | KEY_BACKSPACE,
-KEY_SHIFT | KEY_INSERT,
-KEY_SHIFT | KEY_DELETE,
-KEY_SHIFT | KEY_EQUAL,
-
-KEY_MOD1 | KEY_0,
-KEY_MOD1 | KEY_1,
-KEY_MOD1 | KEY_2,
-KEY_MOD1 | KEY_3,
-KEY_MOD1 | KEY_4,
-KEY_MOD1 | KEY_5,
-KEY_MOD1 | KEY_6,
-KEY_MOD1 | KEY_7,
-KEY_MOD1 | KEY_8,
-KEY_MOD1 | KEY_9,
-KEY_MOD1 | KEY_A,
-KEY_MOD1 | KEY_B,
-KEY_MOD1 | KEY_C,
-KEY_MOD1 | KEY_D,
-KEY_MOD1 | KEY_E,
-KEY_MOD1 | KEY_F,
-KEY_MOD1 | KEY_G,
-KEY_MOD1 | KEY_H,
-KEY_MOD1 | KEY_I,
-KEY_MOD1 | KEY_J,
-KEY_MOD1 | KEY_K,
-KEY_MOD1 | KEY_L,
-KEY_MOD1 | KEY_M,
-KEY_MOD1 | KEY_N,
-KEY_MOD1 | KEY_O,
-KEY_MOD1 | KEY_P,
-KEY_MOD1 | KEY_Q,
-KEY_MOD1 | KEY_R,
-KEY_MOD1 | KEY_S,
-KEY_MOD1 | KEY_T,
-KEY_MOD1 | KEY_U,
-KEY_MOD1 | KEY_V,
-KEY_MOD1 | KEY_W,
-KEY_MOD1 | KEY_X,
-KEY_MOD1 | KEY_Y,
-KEY_MOD1 | KEY_Z,
-KEY_MOD1 | KEY_SEMICOLON,
-KEY_MOD1 | KEY_QUOTERIGHT,
-KEY_MOD1 | KEY_BRACKETLEFT,
-KEY_MOD1 | KEY_BRACKETRIGHT,
-KEY_MOD1 | KEY_POINT,
-KEY_MOD1 | KEY_COMMA,
-KEY_MOD1 | KEY_TILDE,
-KEY_MOD1 | KEY_TAB,
-
-KEY_MOD1 | KEY_F1,
-KEY_MOD1 | KEY_F2,
-KEY_MOD1 | KEY_F3,
-KEY_MOD1 | KEY_F4,
-KEY_MOD1 | KEY_F5,
-KEY_MOD1 | KEY_F6,
-KEY_MOD1 | KEY_F7,
-KEY_MOD1 | KEY_F8,
-KEY_MOD1 | KEY_F9,
-KEY_MOD1 | KEY_F10,
-KEY_MOD1 | KEY_F11,
-KEY_MOD1 | KEY_F12,
-KEY_MOD1 | KEY_F13,
-KEY_MOD1 | KEY_F14,
-KEY_MOD1 | KEY_F15,
-KEY_MOD1 | KEY_F16,
-
-KEY_MOD1 | KEY_DOWN,
-KEY_MOD1 | KEY_UP,
-KEY_MOD1 | KEY_LEFT,
-KEY_MOD1 | KEY_RIGHT,
-KEY_MOD1 | KEY_HOME,
-KEY_MOD1 | KEY_END,
-KEY_MOD1 | KEY_PAGEUP,
-KEY_MOD1 | KEY_PAGEDOWN,
-KEY_MOD1 | KEY_RETURN,
-KEY_MOD1 | KEY_SPACE,
-KEY_MOD1 | KEY_BACKSPACE,
-KEY_MOD1 | KEY_INSERT,
-KEY_MOD1 | KEY_DELETE,
-
-KEY_MOD1 | KEY_ADD,
-KEY_MOD1 | KEY_SUBTRACT,
-KEY_MOD1 | KEY_MULTIPLY,
-KEY_MOD1 | KEY_DIVIDE,
-KEY_MOD1 | KEY_EQUAL,
-
-KEY_SHIFT | KEY_MOD1 | KEY_0,
-KEY_SHIFT | KEY_MOD1 | KEY_1,
-KEY_SHIFT | KEY_MOD1 | KEY_2,
-KEY_SHIFT | KEY_MOD1 | KEY_3,
-KEY_SHIFT | KEY_MOD1 | KEY_4,
-KEY_SHIFT | KEY_MOD1 | KEY_5,
-KEY_SHIFT | KEY_MOD1 | KEY_6,
-KEY_SHIFT | KEY_MOD1 | 

[Libreoffice-commits] core.git: cui/source solenv/clang-format

2020-02-06 Thread Muhammet Kara (via logerrit)
 cui/source/dialogs/dlgname.cxx  |2 +-
 cui/source/dialogs/zoom.cxx |   39 ---
 cui/source/inc/about.hxx|4 ++--
 cui/source/inc/dialmgr.hxx  |2 +-
 cui/source/inc/newtabledlg.hxx  |1 -
 cui/source/inc/showcols.hxx |4 ++--
 cui/source/options/certpath.hxx |2 +-
 cui/source/options/tsaurls.hxx  |2 +-
 solenv/clang-format/blacklist   |8 
 9 files changed, 28 insertions(+), 36 deletions(-)

New commits:
commit e9152974af2598856ab57268cb02c363f8730579
Author: Muhammet Kara 
AuthorDate: Fri Feb 7 01:46:25 2020 +0300
Commit: Muhammet Kara 
CommitDate: Fri Feb 7 08:06:24 2020 +0100

clang-format cui with under 5-percent lines of change

Files which could become clang-format conformant with
under 5-percent lines of change relative to the total
count of lines in the file are found by using bin/find-clang-format.py,
and fixed with /opt/lo/bin/clang-format -i 

There will be follow-up patches to fix all 'under-5-percent' files.

Change-Id: I2886b6da85043218505ff9f3e40a8e28289c6f37
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88152
Tested-by: Jenkins
Reviewed-by: Muhammet Kara 

diff --git a/cui/source/dialogs/dlgname.cxx b/cui/source/dialogs/dlgname.cxx
index 0e201d5e22dd..133f1753fecd 100644
--- a/cui/source/dialogs/dlgname.cxx
+++ b/cui/source/dialogs/dlgname.cxx
@@ -77,7 +77,7 @@ IMPL_LINK_NOARG(SvxObjectNameDialog, ModifyHdl, weld::Entry&, 
void)
 // Dialog for editing Object Title and Description
 
 SvxObjectTitleDescDialog::SvxObjectTitleDescDialog(weld::Window* pParent, 
const OUString& rTitle,
-const OUString& rDescription)
+   const OUString& 
rDescription)
 : GenericDialogController(pParent, "cui/ui/objecttitledescdialog.ui", 
"ObjectTitleDescDialog")
 , m_xEdtTitle(m_xBuilder->weld_entry("object_title_entry"))
 , m_xEdtDescription(m_xBuilder->weld_text_view("desc_entry"))
diff --git a/cui/source/dialogs/zoom.cxx b/cui/source/dialogs/zoom.cxx
index 380ad1404da4..b80080dfa352 100644
--- a/cui/source/dialogs/zoom.cxx
+++ b/cui/source/dialogs/zoom.cxx
@@ -28,7 +28,6 @@
 
 namespace
 {
-
 const sal_uInt16 SPECIAL_FACTOR = 0x;
 
 } // anonymous namespace
@@ -50,7 +49,7 @@ void SvxZoomDialog::SetFactor(sal_uInt16 nNewFactor, 
ZoomButtonId nButtonId)
 
 if (nButtonId == ZoomButtonId::NONE)
 {
-if ( nNewFactor == 100 )
+if (nNewFactor == 100)
 {
 m_x100Btn->set_active(true);
 m_x100Btn->grab_focus();
@@ -66,7 +65,7 @@ void SvxZoomDialog::SetFactor(sal_uInt16 nNewFactor, 
ZoomButtonId nButtonId)
 else
 {
 m_xUserEdit->set_value(nNewFactor, FieldUnit::PERCENT);
-switch(nButtonId)
+switch (nButtonId)
 {
 case ZoomButtonId::OPTIMAL:
 {
@@ -86,7 +85,8 @@ void SvxZoomDialog::SetFactor(sal_uInt16 nNewFactor, 
ZoomButtonId nButtonId)
 m_xWholePageBtn->grab_focus();
 break;
 }
-default: break;
+default:
+break;
 }
 }
 }
@@ -108,7 +108,7 @@ void SvxZoomDialog::HideButton(ZoomButtonId nButtonId)
 break;
 
 default:
-OSL_FAIL("Wrong button number!" );
+OSL_FAIL("Wrong button number!");
 }
 }
 
@@ -118,10 +118,7 @@ void SvxZoomDialog::SetLimits(sal_uInt16 nMin, sal_uInt16 
nMax)
 m_xUserEdit->set_range(nMin, nMax, FieldUnit::PERCENT);
 }
 
-const SfxItemSet* SvxZoomDialog::GetOutputItemSet() const
-{
-return m_pOutSet.get();
-}
+const SfxItemSet* SvxZoomDialog::GetOutputItemSet() const { return 
m_pOutSet.get(); }
 
 SvxZoomDialog::SvxZoomDialog(weld::Window* pParent, const SfxItemSet& rCoreSet)
 : SfxDialogController(pParent, "cui/ui/zoomdialog.ui", "ZoomDialog")
@@ -141,22 +138,24 @@ SvxZoomDialog::SvxZoomDialog(weld::Window* pParent, const 
SfxItemSet& rCoreSet)
 , m_xBookModeChk(m_xBuilder->weld_check_button("bookmode"))
 , m_xOKBtn(m_xBuilder->weld_button("ok"))
 {
-Link aLink = LINK(this, SvxZoomDialog, UserHdl);
+Link aLink = LINK(this, SvxZoomDialog, UserHdl);
 m_x100Btn->connect_toggled(aLink);
 m_xOptimalBtn->connect_toggled(aLink);
 m_xPageWidthBtn->connect_toggled(aLink);
 m_xWholePageBtn->connect_toggled(aLink);
 m_xUserBtn->connect_toggled(aLink);
 
-Link aViewLayoutLink = LINK(this, SvxZoomDialog, 
ViewLayoutUserHdl);
+Link aViewLayoutLink = LINK(this, 
SvxZoomDialog, ViewLayoutUserHdl);
 m_xAutomaticBtn->connect_toggled(aViewLayoutLink);
 m_xSingleBtn->connect_toggled(aViewLayoutLink);
 m_xColumnsBtn->connect_toggled(aViewLayoutLink);
 
-Link aViewLayoutSpinLink = LINK(this, 
SvxZoomDialog, ViewLayoutSpinHdl);
+Link aViewLayoutSpinLink
+= LINK(this, SvxZoomDialog, ViewLayoutSpinHdl);
 

[Libreoffice-commits] core.git: cui/source solenv/clang-format

2019-05-23 Thread Muhammet Kara (via logerrit)
 cui/source/options/personalization.cxx |  111 ++---
 cui/source/options/personalization.hxx |   24 +++
 solenv/clang-format/blacklist  |2 
 3 files changed, 61 insertions(+), 76 deletions(-)

New commits:
commit 4c1479d63972aaea0b1bfee1ffb56b530ca05655
Author: Muhammet Kara 
AuthorDate: Thu May 23 00:41:44 2019 +0300
Commit: Muhammet Kara 
CommitDate: Thu May 23 18:21:06 2019 +0200

clang-format SvxPersonalizationTabPage

Now is the time to get it out of the blacklist
since it is a tiny class again.

And also remove some leftovers.

Change-Id: Ia38dd3054ddefa43a7e0d917d358e7d9d1b750e4
Reviewed-on: https://gerrit.libreoffice.org/72837
Tested-by: Jenkins
Reviewed-by: Muhammet Kara 

diff --git a/cui/source/options/personalization.cxx 
b/cui/source/options/personalization.cxx
index 0ab9a28fecc8..e9d9380065d1 100644
--- a/cui/source/options/personalization.cxx
+++ b/cui/source/options/personalization.cxx
@@ -43,100 +43,90 @@ using namespace ::com::sun::star::uno;
 using namespace ::com::sun::star::ucb;
 using namespace ::com::sun::star::beans;
 
-struct PersonaInfo
-{
-OUString sSlug;
-OUString sName;
-OUString sPreviewURL;
-OUString sHeaderURL;
-OUString sFooterURL;
-OUString sTextColor;
-};
-
-SvxPersonalizationTabPage::SvxPersonalizationTabPage( vcl::Window *pParent, 
const SfxItemSet  )
-: SfxTabPage( pParent, "PersonalizationTabPage", 
"cui/ui/personalization_tab.ui",  )
+SvxPersonalizationTabPage::SvxPersonalizationTabPage(vcl::Window* pParent, 
const SfxItemSet& rSet)
+: SfxTabPage(pParent, "PersonalizationTabPage", 
"cui/ui/personalization_tab.ui", )
 {
 // persona
-get( m_pNoPersona, "no_persona" );
-get( m_pDefaultPersona, "default_persona" );
+get(m_pNoPersona, "no_persona");
+get(m_pDefaultPersona, "default_persona");
 
 for (sal_uInt32 i = 0; i < MAX_DEFAULT_PERSONAS; ++i)
 {
 OUString sDefaultId("default" + OUString::number(i));
-get( m_vDefaultPersonaImages[i], OUStringToOString(sDefaultId, 
RTL_TEXTENCODING_UTF8) );
-m_vDefaultPersonaImages[i]->SetClickHdl( LINK( this, 
SvxPersonalizationTabPage, DefaultPersona ) );
+get(m_vDefaultPersonaImages[i], OUStringToOString(sDefaultId, 
RTL_TEXTENCODING_UTF8));
+m_vDefaultPersonaImages[i]->SetClickHdl(
+LINK(this, SvxPersonalizationTabPage, DefaultPersona));
 }
 
 LoadDefaultImages();
 }
 
-SvxPersonalizationTabPage::~SvxPersonalizationTabPage()
-{
-disposeOnce();
-}
+SvxPersonalizationTabPage::~SvxPersonalizationTabPage() { disposeOnce(); }
 
 void SvxPersonalizationTabPage::dispose()
 {
 m_pNoPersona.clear();
 m_pDefaultPersona.clear();
-for (VclPtr & i : m_vDefaultPersonaImages)
+for (VclPtr& i : m_vDefaultPersonaImages)
 i.clear();
 SfxTabPage::dispose();
 }
 
-
-VclPtr SvxPersonalizationTabPage::Create( TabPageParent pParent, 
const SfxItemSet *rSet )
+VclPtr SvxPersonalizationTabPage::Create(TabPageParent pParent, 
const SfxItemSet* rSet)
 {
-return VclPtr::Create( pParent.pParent, *rSet );
+return VclPtr::Create(pParent.pParent, *rSet);
 }
 
-bool SvxPersonalizationTabPage::FillItemSet( SfxItemSet * )
+bool SvxPersonalizationTabPage::FillItemSet(SfxItemSet*)
 {
 // persona
-OUString aPersona( "default" );
-if ( m_pNoPersona->IsChecked() )
+OUString aPersona("default");
+if (m_pNoPersona->IsChecked())
 aPersona = "no";
 
 bool bModified = false;
-uno::Reference< uno::XComponentContext > xContext( 
comphelper::getProcessComponentContext() );
-if ( xContext.is() &&
-( aPersona != officecfg::Office::Common::Misc::Persona::get( 
xContext ) ||
-  m_aPersonaSettings != 
officecfg::Office::Common::Misc::PersonaSettings::get( xContext ) ) )
+uno::Reference 
xContext(comphelper::getProcessComponentContext());
+if (xContext.is()
+&& (aPersona != officecfg::Office::Common::Misc::Persona::get(xContext)
+|| m_aPersonaSettings
+   != 
officecfg::Office::Common::Misc::PersonaSettings::get(xContext)))
 {
 bModified = true;
 }
 
 // write
-std::shared_ptr< comphelper::ConfigurationChanges > batch( 
comphelper::ConfigurationChanges::create() );
-if( aPersona == "no" )
+std::shared_ptr batch(
+comphelper::ConfigurationChanges::create());
+if (aPersona == "no")
 m_aPersonaSettings.clear();
-officecfg::Office::Common::Misc::Persona::set( aPersona, batch );
-officecfg::Office::Common::Misc::PersonaSettings::set( m_aPersonaSettings, 
batch );
+officecfg::Office::Common::Misc::Persona::set(aPersona, batch);
+officecfg::Office::Common::Misc::PersonaSettings::set(m_aPersonaSettings, 
batch);
 batch->commit();
 
-if ( bModified )
+if (bModified)
 {
 // broadcast the change
-DataChangedEvent aDataChanged(