diff options
| author | Michael Weghorn <m.weghorn@posteo.de> | 2025-02-10 10:01:32 +0100 |
|---|---|---|
| committer | Michael Weghorn <m.weghorn@posteo.de> | 2025-02-10 14:05:26 +0100 |
| commit | 59d34ccdbd27ddba09179c235ba89e542d818a41 (patch) | |
| tree | 77814405db8aa59fbf7136321e230abdb78823f3 /vcl/source | |
| parent | 38bab24446ca7967f604735a04452b4969758c8c (diff) | |
vcl: Use enum class instead of int defines for treeview button types
Change-Id: I2d19d71cb3ac5a2b7d8319635610369036d4ac0e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/181336
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Diffstat (limited to 'vcl/source')
| -rw-r--r-- | vcl/source/treelist/treelistbox.cxx | 33 |
1 files changed, 20 insertions, 13 deletions
diff --git a/vcl/source/treelist/treelistbox.cxx b/vcl/source/treelist/treelistbox.cxx index a7a482a59d4c..4ef41e271ec1 100644 --- a/vcl/source/treelist/treelistbox.cxx +++ b/vcl/source/treelist/treelistbox.cxx @@ -1364,6 +1364,9 @@ void SvTreeListBox::Resize() pImpl->ShowCursor( true ); } + +namespace { + /* Cases: A) entries have bitmaps @@ -1377,11 +1380,15 @@ void SvTreeListBox::Resize() 2. node buttons (can optionally also be on root items) + CheckButton 3. CheckButton */ +enum class TreeListButtonType +{ + NO_BUTTONS, + NODE_BUTTONS, + NODE_AND_CHECK_BUTTONS, + CHECK_BUTTONS, +}; -#define NO_BUTTONS 0 -#define NODE_BUTTONS 1 -#define NODE_AND_CHECK_BUTTONS 2 -#define CHECK_BUTTONS 3 +} #define TABFLAGS_TEXT (SvLBoxTabFlags::DYNAMIC | \ SvLBoxTabFlags::ADJUST_LEFT | \ @@ -1421,23 +1428,23 @@ void SvTreeListBox::SetTabs() ClearTabList(); - int nCase = NO_BUTTONS; + TreeListButtonType eButtonType = TreeListButtonType::NO_BUTTONS; if( !(nTreeFlags & SvTreeFlags::CHKBTN) ) { if( bHasButtons ) - nCase = NODE_BUTTONS; + eButtonType = TreeListButtonType::NODE_BUTTONS; } else { if( bHasButtons ) - nCase = NODE_AND_CHECK_BUTTONS; + eButtonType = TreeListButtonType::NODE_AND_CHECK_BUTTONS; else - nCase = CHECK_BUTTONS; + eButtonType = TreeListButtonType::CHECK_BUTTONS; } - switch( nCase ) + switch(eButtonType) { - case NO_BUTTONS : + case TreeListButtonType::NO_BUTTONS: nStartPos += nContextWidthDIV2; // because of centering AddTab( nStartPos, TABFLAGS_CONTEXTBMP ); nStartPos += nContextWidthDIV2; // right edge of context bitmap @@ -1447,7 +1454,7 @@ void SvTreeListBox::SetTabs() AddTab( nStartPos, TABFLAGS_TEXT ); break; - case NODE_BUTTONS : + case TreeListButtonType::NODE_BUTTONS: if( bHasButtonsAtRoot ) nStartPos += ( nIndent + (nNodeWidthPixel/2) ); else @@ -1463,7 +1470,7 @@ void SvTreeListBox::SetTabs() AddTab( nStartPos, TABFLAGS_TEXT ); break; - case NODE_AND_CHECK_BUTTONS : + case TreeListButtonType::NODE_AND_CHECK_BUTTONS: if( bHasButtonsAtRoot ) nStartPos += ( nIndent + nNodeWidthPixel ); else @@ -1480,7 +1487,7 @@ void SvTreeListBox::SetTabs() AddTab( nStartPos, TABFLAGS_TEXT ); break; - case CHECK_BUTTONS : + case TreeListButtonType::CHECK_BUTTONS: nStartPos += nCheckWidthDIV2; AddTab( nStartPos, TABFLAGS_CHECKBTN ); nStartPos += nCheckWidthDIV2; // right edge of CheckButton |