include/oox/helper/helper.hxx                                |    4 -
 oox/source/drawingml/chart/axisconverter.cxx                 |   20 +++---
 oox/source/drawingml/chart/chartspaceconverter.cxx           |    2 
 oox/source/drawingml/chart/objectformatter.cxx               |    4 -
 oox/source/drawingml/chart/seriesconverter.cxx               |   28 ++++-----
 oox/source/drawingml/connectorshapecontext.cxx               |   12 +--
 oox/source/drawingml/customshapegeometry.cxx                 |   26 ++++----
 oox/source/drawingml/customshapeproperties.cxx               |   24 +++----
 oox/source/drawingml/diagram/datamodelcontext.cxx            |   20 +++---
 oox/source/drawingml/diagram/diagramdefinitioncontext.cxx    |   14 ++--
 oox/source/drawingml/diagram/diagramlayoutatoms.cxx          |   12 +--
 oox/source/drawingml/diagram/layoutnodecontext.cxx           |   12 +--
 oox/source/drawingml/embeddedwavaudiofile.cxx                |    4 -
 oox/source/drawingml/fillproperties.cxx                      |   18 ++---
 oox/source/drawingml/graphicshapecontext.cxx                 |   14 ++--
 oox/source/drawingml/hyperlinkcontext.cxx                    |    8 +-
 oox/source/drawingml/lineproperties.cxx                      |   22 +++----
 oox/source/drawingml/shape.cxx                               |   22 +++----
 oox/source/drawingml/shapecontext.cxx                        |    8 +-
 oox/source/drawingml/shapegroupcontext.cxx                   |    6 -
 oox/source/drawingml/table/tablecell.cxx                     |    6 -
 oox/source/drawingml/table/tablecellcontext.cxx              |    4 -
 oox/source/drawingml/table/tablecontext.cxx                  |    2 
 oox/source/drawingml/table/tablerowcontext.cxx               |    2 
 oox/source/drawingml/table/tablestylecontext.cxx             |    4 -
 oox/source/drawingml/table/tablestylelistfragmenthandler.cxx |    2 
 oox/source/drawingml/textbodypropertiescontext.cxx           |    4 -
 oox/source/drawingml/textcharacterproperties.cxx             |   18 ++---
 oox/source/drawingml/textcharacterpropertiescontext.cxx      |    8 +-
 oox/source/drawingml/texteffectscontext.cxx                  |    2 
 oox/source/drawingml/textfield.cxx                           |    2 
 oox/source/drawingml/textfieldcontext.cxx                    |    4 -
 oox/source/drawingml/textparagraph.cxx                       |    2 
 oox/source/drawingml/textparagraphpropertiescontext.cxx      |   30 ++++-----
 oox/source/drawingml/textrun.cxx                             |    2 
 oox/source/drawingml/textspacingcontext.cxx                  |    4 -
 oox/source/drawingml/texttabstoplistcontext.cxx              |    2 
 oox/source/drawingml/themeelementscontext.cxx                |    4 -
 oox/source/drawingml/themefragmenthandler.cxx                |    2 
 oox/source/drawingml/transform2dcontext.cxx                  |   20 +++---
 oox/source/helper/attributelist.cxx                          |    2 
 oox/source/ppt/layoutfragmenthandler.cxx                     |    2 
 oox/source/ppt/pptgraphicshapecontext.cxx                    |    6 -
 oox/source/ppt/pptshape.cxx                                  |   10 +--
 oox/source/ppt/pptshapecontext.cxx                           |    6 -
 oox/source/ppt/pptshapegroupcontext.cxx                      |    8 +-
 oox/source/ppt/presPropsfragmenthandler.cxx                  |    4 -
 oox/source/ppt/slidefragmenthandler.cxx                      |    2 
 oox/source/shape/LockedCanvasContext.cxx                     |    4 -
 oox/source/shape/WpsContext.cxx                              |   12 +--
 oox/source/vml/vmlformatting.cxx                             |   34 +++++------
 oox/source/vml/vmlshape.cxx                                  |   16 ++---
 oox/source/vml/vmlshapecontext.cxx                           |   26 ++++----
 oox/source/vml/vmltextbox.cxx                                |   22 +++----
 oox/source/vml/vmltextboxcontext.cxx                         |    8 +-
 sc/source/filter/oox/SparklineFragment.cxx                   |    4 -
 sc/source/filter/oox/autofilterbuffer.cxx                    |    2 
 sc/source/filter/oox/drawingfragment.cxx                     |    2 
 58 files changed, 287 insertions(+), 287 deletions(-)

New commits:
commit 0feeb94f97332a8e803e1936d66e0f234bd51973
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Tue Jun 21 13:02:53 2022 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Wed Jun 22 11:48:28 2022 +0200

    rename oox::OptValue::get to value
    
    as a step in replacing OptValue with std::optional
    
    Change-Id: Ia5d05c28a88beaced11ae1d0414de66106cc9e20
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136269
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/include/oox/helper/helper.hxx b/include/oox/helper/helper.hxx
index 0c8aa2e6c358..9d5b7c3e2549 100644
--- a/include/oox/helper/helper.hxx
+++ b/include/oox/helper/helper.hxx
@@ -180,7 +180,7 @@ public:
     bool         has_value() const { return mbHasValue; }
     bool         operator!() const { return !mbHasValue; }
 
-    const Type&  get() const { return maValue; }
+    const Type&  value() const { return maValue; }
     const Type&  value_or( const Type& rDefValue ) const { return mbHasValue ? 
maValue : rDefValue; }
 
     Type&        operator*() { assert(mbHasValue); return maValue; }
@@ -201,7 +201,7 @@ template< typename Type >
 void assignIfUsed( OptValue<Type>& rDestValue, const OptValue<Type>& 
rSourceValue )
 {
     if( rSourceValue.has_value() )
-        rDestValue = rSourceValue.get();
+        rDestValue = rSourceValue.value();
 }
 
 
diff --git a/oox/source/drawingml/chart/axisconverter.cxx 
b/oox/source/drawingml/chart/axisconverter.cxx
index bb6b278af8ac..8d6ba3fab509 100644
--- a/oox/source/drawingml/chart/axisconverter.cxx
+++ b/oox/source/drawingml/chart/axisconverter.cxx
@@ -55,12 +55,12 @@ namespace {
 
 void lclSetValueOrClearAny( Any& orAny, const OptValue< double >& rofValue )
 {
-    if( rofValue.has_value() ) orAny <<= rofValue.get(); else orAny.clear();
+    if( rofValue.has_value() ) orAny <<= rofValue.value(); else orAny.clear();
 }
 
 bool lclIsLogarithmicScale( const AxisModel& rAxisModel )
 {
-    return rAxisModel.mofLogBase.has_value() && (2.0 <= 
rAxisModel.mofLogBase.get()) && (rAxisModel.mofLogBase.get() <= 1000.0);
+    return rAxisModel.mofLogBase.has_value() && (2.0 <= 
rAxisModel.mofLogBase.value()) && (rAxisModel.mofLogBase.value() <= 1000.0);
 }
 
 sal_Int32 lclGetApiTimeUnit( sal_Int32 nTimeUnit )
@@ -78,8 +78,8 @@ sal_Int32 lclGetApiTimeUnit( sal_Int32 nTimeUnit )
 
 void lclConvertTimeInterval( Any& orInterval, const OptValue< double >& 
rofUnit, sal_Int32 nTimeUnit )
 {
-    if( rofUnit.has_value() && (1.0 <= rofUnit.get()) && (rofUnit.get() <= 
SAL_MAX_INT32) )
-        orInterval <<= css::chart::TimeInterval( static_cast< sal_Int32 >( 
rofUnit.get() ), lclGetApiTimeUnit( nTimeUnit ) );
+    if( rofUnit.has_value() && (1.0 <= rofUnit.value()) && (rofUnit.value() <= 
SAL_MAX_INT32) )
+        orInterval <<= css::chart::TimeInterval( static_cast< sal_Int32 >( 
rofUnit.value() ), lclGetApiTimeUnit( nTimeUnit ) );
     else
         orInterval.clear();
 }
@@ -265,7 +265,7 @@ void AxisConverter::convertFromModel(const 
Reference<XCoordinateSystem>& rxCoord
                     lclConvertTimeInterval( 
aScaleData.TimeIncrement.MinorTimeInterval, mrModel.mofMinorUnit, 
mrModel.mnMinorTimeUnit );
                     // base time unit
                     if( mrModel.monBaseTimeUnit.has_value() )
-                        aScaleData.TimeIncrement.TimeResolution <<= 
lclGetApiTimeUnit( mrModel.monBaseTimeUnit.get() );
+                        aScaleData.TimeIncrement.TimeResolution <<= 
lclGetApiTimeUnit( mrModel.monBaseTimeUnit.value() );
                     else
                         aScaleData.TimeIncrement.TimeResolution.clear();
                 }
@@ -276,7 +276,7 @@ void AxisConverter::convertFromModel(const 
Reference<XCoordinateSystem>& rxCoord
                     if (mrModel.mxTextProp.is()
                         && 
mrModel.mxTextProp->getTextProperties().moRotation.has_value())
                         bTextOverlap
-                            = 
mrModel.mxTextProp->getTextProperties().moRotation.get() == 0;
+                            = 
mrModel.mxTextProp->getTextProperties().moRotation.value() == 0;
                     aAxisProp.setProperty(PROP_TextOverlap, bTextOverlap);
                     /* do not break text into several lines unless the 
rotation is 0 degree,
                        or the rotation is 90 degree and the inner size of the 
chart is not fixed,
@@ -309,7 +309,7 @@ void AxisConverter::convertFromModel(const 
Reference<XCoordinateSystem>& rxCoord
                 // major increment
                 IncrementData& rIncrementData = aScaleData.IncrementData;
                 if( mrModel.mofMajorUnit.has_value() && 
aScaleData.Scaling.is() )
-                    rIncrementData.Distance <<= aScaleData.Scaling->doScaling( 
mrModel.mofMajorUnit.get() );
+                    rIncrementData.Distance <<= aScaleData.Scaling->doScaling( 
mrModel.mofMajorUnit.value() );
                 else
                     lclSetValueOrClearAny( rIncrementData.Distance, 
mrModel.mofMajorUnit );
                 // minor increment
@@ -322,9 +322,9 @@ void AxisConverter::convertFromModel(const 
Reference<XCoordinateSystem>& rxCoord
                     if( mrModel.mofMinorUnit.has_value() )
                         rIntervalCount <<= sal_Int32( 9 );
                 }
-                else if( mrModel.mofMajorUnit.has_value() && 
mrModel.mofMinorUnit.has_value() && (0.0 < mrModel.mofMinorUnit.get()) && 
(mrModel.mofMinorUnit.get() <= mrModel.mofMajorUnit.get()) )
+                else if( mrModel.mofMajorUnit.has_value() && 
mrModel.mofMinorUnit.has_value() && (0.0 < mrModel.mofMinorUnit.value()) && 
(mrModel.mofMinorUnit.value() <= mrModel.mofMajorUnit.value()) )
                 {
-                    double fCount = mrModel.mofMajorUnit.get() / 
mrModel.mofMinorUnit.get() + 0.5;
+                    double fCount = mrModel.mofMajorUnit.value() / 
mrModel.mofMinorUnit.value() + 0.5;
                     if( (1.0 <= fCount) && (fCount < 1001.0) )
                         rIntervalCount <<= static_cast< sal_Int32 >( fCount );
                 }
@@ -377,7 +377,7 @@ void AxisConverter::convertFromModel(const 
Reference<XCoordinateSystem>& rxCoord
 
         // calculate automatic origin depending on scaling mode of crossing 
axis
         bool bCrossingLogScale = pCrossingAxis && lclIsLogarithmicScale( 
*pCrossingAxis );
-        double fCrossingPos = bManualCrossing ? mrModel.mofCrossesAt.get() : 
(bCrossingLogScale ? 1.0 : 0.0);
+        double fCrossingPos = bManualCrossing ? mrModel.mofCrossesAt.value() : 
(bCrossingLogScale ? 1.0 : 0.0);
         aAxisProp.setProperty( PROP_CrossoverValue, fCrossingPos );
 
         // axis title ---------------------------------------------------------
diff --git a/oox/source/drawingml/chart/chartspaceconverter.cxx 
b/oox/source/drawingml/chart/chartspaceconverter.cxx
index 7b9044c47442..c83ed37e9c02 100644
--- a/oox/source/drawingml/chart/chartspaceconverter.cxx
+++ b/oox/source/drawingml/chart/chartspaceconverter.cxx
@@ -212,7 +212,7 @@ void ChartSpaceConverter::convertFromModel( const 
Reference< XShapes >& rxExtern
         if( mrModel.mxPlotArea.is() && mrModel.mxPlotArea->maAxes.size() > 0 &&
             mrModel.mxPlotArea->maAxes[0]->monBaseTimeUnit.has_value() )
         {
-            bIsMonthBasedTimeUnit = 
mrModel.mxPlotArea->maAxes[0]->monBaseTimeUnit.get() == XML_months;
+            bIsMonthBasedTimeUnit = 
mrModel.mxPlotArea->maAxes[0]->monBaseTimeUnit.value() == XML_months;
         }
 
         if (!bIsMonthBasedTimeUnit) switch( mrModel.mnDispBlanksAs )
diff --git a/oox/source/drawingml/chart/objectformatter.cxx 
b/oox/source/drawingml/chart/objectformatter.cxx
index 2ef67ebc683e..f716ece56f2c 100644
--- a/oox/source/drawingml/chart/objectformatter.cxx
+++ b/oox/source/drawingml/chart/objectformatter.cxx
@@ -839,7 +839,7 @@ LineFormatter::LineFormatter( ObjectFormatterData& rData, 
const AutoFormatEntry*
     }
     // change line width according to chart auto style
     if( mxAutoLine->moLineWidth.has_value() )
-        mxAutoLine->moLineWidth = mxAutoLine->moLineWidth.get() * 
pAutoFormatEntry->mnRelLineWidth / 100;
+        mxAutoLine->moLineWidth = mxAutoLine->moLineWidth.value() * 
pAutoFormatEntry->mnRelLineWidth / 100;
 }
 
 void LineFormatter::convertFormatting( ShapePropertyMap& rPropMap, const 
ModelRef< Shape >& rxShapeProp, sal_Int32 nSeriesIdx )
@@ -915,7 +915,7 @@ TextFormatter::TextFormatter( ObjectFormatterData& rData, 
const AutoTextEntry* p
     {
         mxAutoText->assignUsed( *pTextProps );
         if( pTextProps->moHeight.has_value() )
-            mxAutoText->moHeight = pTextProps->moHeight.get() * 
pAutoTextEntry->mnRelFontSize / 100;
+            mxAutoText->moHeight = pTextProps->moHeight.value() * 
pAutoTextEntry->mnRelFontSize / 100;
     }
 }
 
diff --git a/oox/source/drawingml/chart/seriesconverter.cxx 
b/oox/source/drawingml/chart/seriesconverter.cxx
index e618e0695cf7..383e5bd3fa4b 100644
--- a/oox/source/drawingml/chart/seriesconverter.cxx
+++ b/oox/source/drawingml/chart/seriesconverter.cxx
@@ -197,12 +197,12 @@ void importBorderProperties( PropertySet& rPropSet, 
Shape& rShape, const Graphic
 {
     LineProperties& rLP = rShape.getLineProperties();
     // no fill has the same effect as no border so skip it
-    if (rLP.maLineFill.moFillType.get() == XML_noFill)
+    if (rLP.maLineFill.moFillType.value() == XML_noFill)
         return;
 
     if (rLP.moLineWidth.has_value())
     {
-        sal_Int32 nWidth = convertEmuToHmm(rLP.moLineWidth.get());
+        sal_Int32 nWidth = convertEmuToHmm(rLP.moLineWidth.value());
         rPropSet.setProperty(PROP_LabelBorderWidth, uno::Any(nWidth));
         rPropSet.setProperty(PROP_LabelBorderStyle, 
uno::Any(drawing::LineStyle_SOLID));
     }
@@ -215,7 +215,7 @@ void importFillProperties( PropertySet& rPropSet, Shape& 
rShape, const GraphicHe
 {
     FillProperties& rFP = rShape.getFillProperties();
 
-    if (rFP.moFillType.has_value() && rFP.moFillType.get() == XML_solidFill)
+    if (rFP.moFillType.has_value() && rFP.moFillType.value() == XML_solidFill)
     {
         rPropSet.setProperty(PROP_LabelFillStyle, drawing::FillStyle_SOLID);
 
@@ -223,13 +223,13 @@ void importFillProperties( PropertySet& rPropSet, Shape& 
rShape, const GraphicHe
         ::Color nColor = aColor.getColor(rGraphicHelper);
         rPropSet.setProperty(PROP_LabelFillColor, uno::Any(nColor));
     }
-    else if(rFP.moFillType.has_value() && rFP.moFillType.get() == XML_pattFill)
+    else if(rFP.moFillType.has_value() && rFP.moFillType.value() == 
XML_pattFill)
     {
         rPropSet.setProperty(PROP_LabelFillStyle, drawing::FillStyle_HATCH);
         rPropSet.setProperty(PROP_LabelFillBackground, true);
 
         Color aHatchColor( rFP.maPatternProps.maPattFgColor );
-        drawing::Hatch aHatch = 
createHatch(rFP.maPatternProps.moPattPreset.get(), 
aHatchColor.getColor(rGraphicHelper, 0));
+        drawing::Hatch aHatch = 
createHatch(rFP.maPatternProps.moPattPreset.value(), 
aHatchColor.getColor(rGraphicHelper, 0));
 
         OUString sHatchName = rModelObjHelper.insertFillHatch(aHatch);
         rPropSet.setProperty(PROP_LabelFillHatchName, sHatchName);
@@ -360,8 +360,8 @@ void DataLabelConverter::convertFromModel( const Reference< 
XDataSeries >& rxDat
 
                         if (eType == 
DataPointCustomLabelFieldType::DataPointCustomLabelFieldType_CELLRANGE && 
oaCellRange.has_value())
                         {
-                            xCustomLabel->setCellRange( oaCellRange.get() );
-                            xCustomLabel->setString( oaLabelText.get() );
+                            xCustomLabel->setCellRange( oaCellRange.value() );
+                            xCustomLabel->setString( oaLabelText.value() );
                             xCustomLabel->setDataLabelsRange( true );
                         }
                         else
@@ -671,13 +671,13 @@ void TrendlineConverter::convertFromModel( const 
Reference< XDataSeries >& rxDat
             bool hasIntercept = mrModel.mfIntercept.has_value();
             aPropSet.setProperty( PROP_ForceIntercept, hasIntercept);
             if (hasIntercept)
-                aPropSet.setProperty( PROP_InterceptValue,  
mrModel.mfIntercept.get());
+                aPropSet.setProperty( PROP_InterceptValue,  
mrModel.mfIntercept.value());
 
             // Extrapolation
             if (mrModel.mfForward.has_value())
-                aPropSet.setProperty( PROP_ExtrapolateForward, 
mrModel.mfForward.get() );
+                aPropSet.setProperty( PROP_ExtrapolateForward, 
mrModel.mfForward.value() );
             if (mrModel.mfBackward.has_value())
-                aPropSet.setProperty( PROP_ExtrapolateBackward, 
mrModel.mfBackward.get() );
+                aPropSet.setProperty( PROP_ExtrapolateBackward, 
mrModel.mfBackward.value() );
 
             // trendline formatting
             getFormatter().convertFrameFormatting( aPropSet, 
mrModel.mxShapeProp, OBJECTTYPE_TRENDLINE );
@@ -725,14 +725,14 @@ void DataPointConverter::convertFromModel( const 
Reference< XDataSeries >& rxDat
         PropertySet aPropSet( rxDataSeries->getDataPointByIndex( 
mrModel.mnIndex ) );
 
         // data point marker
-        if( ( mrModel.monMarkerSymbol.has_value() && 
mrModel.monMarkerSymbol.get() != rSeries.mnMarkerSymbol ) ||
-            ( mrModel.monMarkerSize.has_value() && mrModel.monMarkerSize.get() 
!= rSeries.mnMarkerSize ) )
+        if( ( mrModel.monMarkerSymbol.has_value() && 
mrModel.monMarkerSymbol.value() != rSeries.mnMarkerSymbol ) ||
+            ( mrModel.monMarkerSize.has_value() && 
mrModel.monMarkerSize.value() != rSeries.mnMarkerSize ) )
             rTypeGroup.convertMarker( aPropSet, 
mrModel.monMarkerSymbol.value_or( rSeries.mnMarkerSymbol ),
                     mrModel.monMarkerSize.value_or( rSeries.mnMarkerSize ), 
mrModel.mxMarkerProp );
 
         // data point pie explosion
-        if( mrModel.monExplosion.has_value() && mrModel.monExplosion.get() != 
rSeries.mnExplosion )
-            rTypeGroup.convertPieExplosion( aPropSet, 
mrModel.monExplosion.get() );
+        if( mrModel.monExplosion.has_value() && mrModel.monExplosion.value() 
!= rSeries.mnExplosion )
+            rTypeGroup.convertPieExplosion( aPropSet, 
mrModel.monExplosion.value() );
 
         // point formatting
         if( mrModel.mxShapeProp.is() )
diff --git a/oox/source/drawingml/connectorshapecontext.cxx 
b/oox/source/drawingml/connectorshapecontext.cxx
index b1ea90c357af..ebafb5116f8b 100644
--- a/oox/source/drawingml/connectorshapecontext.cxx
+++ b/oox/source/drawingml/connectorshapecontext.cxx
@@ -70,15 +70,15 @@ ConnectorShapePropertiesContext::onCreateContext(sal_Int32 
aElementToken,
     switch (getBaseToken(aElementToken))
     {
         case XML_cNvPr:
-            mpConnectorShapePtr->setId(rAttribs.getString(XML_id).get());
-            mpConnectorShapePtr->setName(rAttribs.getString(XML_name).get());
+            mpConnectorShapePtr->setId(rAttribs.getString(XML_id).value());
+            mpConnectorShapePtr->setName(rAttribs.getString(XML_name).value());
             break;
         case XML_stCxn:
         {
             ConnectorShapeProperties aConnectorShapeProps;
             aConnectorShapeProps.mbStartShape = true;
-            aConnectorShapeProps.maDestShapeId = 
rAttribs.getString(XML_id).get();
-            aConnectorShapeProps.mnDestGlueId = 
rAttribs.getInteger(XML_idx).get();
+            aConnectorShapeProps.maDestShapeId = 
rAttribs.getString(XML_id).value();
+            aConnectorShapeProps.mnDestGlueId = 
rAttribs.getInteger(XML_idx).value();
             mrConnectorShapePropertiesList.push_back(aConnectorShapeProps);
             break;
         }
@@ -86,8 +86,8 @@ ConnectorShapePropertiesContext::onCreateContext(sal_Int32 
aElementToken,
         {
             ConnectorShapeProperties aConnectorShapeProps;
             aConnectorShapeProps.mbStartShape = false;
-            aConnectorShapeProps.maDestShapeId = 
rAttribs.getString(XML_id).get();
-            aConnectorShapeProps.mnDestGlueId = 
rAttribs.getInteger(XML_idx).get();
+            aConnectorShapeProps.maDestShapeId = 
rAttribs.getString(XML_id).value();
+            aConnectorShapeProps.mnDestGlueId = 
rAttribs.getInteger(XML_idx).value();
             mrConnectorShapePropertiesList.push_back(aConnectorShapeProps);
             break;
         }
diff --git a/oox/source/drawingml/customshapegeometry.cxx 
b/oox/source/drawingml/customshapegeometry.cxx
index 2f508382cdc9..de240281833c 100644
--- a/oox/source/drawingml/customshapegeometry.cxx
+++ b/oox/source/drawingml/customshapegeometry.cxx
@@ -621,8 +621,8 @@ ContextHandlerRef GeomGuideListContext::onCreateContext( 
sal_Int32 aElementToken
     if ( aElementToken == A_TOKEN( gd ) )   // CT_GeomGuide
     {
         CustomShapeGuide aGuide;
-        aGuide.maName = rAttribs.getString( XML_name ).get();
-        aGuide.maFormula = convertToOOEquation( mrCustomShapeProperties, 
rAttribs.getString( XML_fmla ).get() );
+        aGuide.maName = rAttribs.getString( XML_name ).value();
+        aGuide.maFormula = convertToOOEquation( mrCustomShapeProperties, 
rAttribs.getString( XML_fmla ).value() );
         mrGuideList.push_back( aGuide );
     }
     return this;
@@ -647,8 +647,8 @@ public:
 AdjPoint2DContext::AdjPoint2DContext( ContextHandler2Helper const & rParent, 
const AttributeList& rAttribs, CustomShapeProperties& rCustomShapeProperties, 
EnhancedCustomShapeParameterPair& rAdjPoint2D )
 : ContextHandler2( rParent )
 {
-    rAdjPoint2D.First = GetAdjCoordinate( rCustomShapeProperties, 
rAttribs.getString( XML_x ).get() );
-    rAdjPoint2D.Second = GetAdjCoordinate( rCustomShapeProperties, 
rAttribs.getString( XML_y ).get() );
+    rAdjPoint2D.First = GetAdjCoordinate( rCustomShapeProperties, 
rAttribs.getString( XML_x ).value() );
+    rAdjPoint2D.Second = GetAdjCoordinate( rCustomShapeProperties, 
rAttribs.getString( XML_y ).value() );
 }
 
 namespace {
@@ -823,7 +823,7 @@ ConnectionSiteContext::ConnectionSiteContext( 
ContextHandler2Helper const & rPar
 , mrConnectionSite( rConnectionSite )
 , mrCustomShapeProperties( rCustomShapeProperties )
 {
-    mrConnectionSite.ang = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_ang ).get() );
+    mrConnectionSite.ang = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_ang ).value() );
 }
 
 ContextHandlerRef ConnectionSiteContext::onCreateContext( sal_Int32 
aElementToken, const AttributeList& rAttribs )
@@ -1099,8 +1099,8 @@ ContextHandlerRef Path2DContext::onCreateContext( 
sal_Int32 aElementToken,
             EnhancedCustomShapeParameterPair aScale;
             EnhancedCustomShapeParameterPair aAngles;
 
-            aScale.First = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_wR ).get() );
-            aScale.Second = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_hR ).get() );
+            aScale.First = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_wR ).value() );
+            aScale.Second = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_hR ).value() );
 
             CustomShapeGuide aGuide;
             sal_Int32 nArcNum = mrCustomShapeProperties.getArcNum();
@@ -1108,7 +1108,7 @@ ContextHandlerRef Path2DContext::onCreateContext( 
sal_Int32 aElementToken,
             // start angle
             aGuide.maName = "arctosa" + OUString::number( nArcNum );
             aGuide.maFormula = "("
-                + GetFormulaParameter( GetAdjCoordinate( 
mrCustomShapeProperties, rAttribs.getString( XML_stAng ).get() ) )
+                + GetFormulaParameter( GetAdjCoordinate( 
mrCustomShapeProperties, rAttribs.getString( XML_stAng ).value() ) )
                 + ")/60000.0";
             aAngles.First.Value <<= 
CustomShapeProperties::SetCustomShapeGuideValue( 
mrCustomShapeProperties.getGuideList(), aGuide );
             aAngles.First.Type = EnhancedCustomShapeParameterType::EQUATION;
@@ -1116,7 +1116,7 @@ ContextHandlerRef Path2DContext::onCreateContext( 
sal_Int32 aElementToken,
             // swing angle
             aGuide.maName = "arctosw" + OUString::number( nArcNum );
             aGuide.maFormula = "("
-                + GetFormulaParameter( GetAdjCoordinate( 
mrCustomShapeProperties, rAttribs.getString( XML_swAng ).get() ) )
+                + GetFormulaParameter( GetAdjCoordinate( 
mrCustomShapeProperties, rAttribs.getString( XML_swAng ).value() ) )
                 + ")/60000.0";
             aAngles.Second.Value <<= 
CustomShapeProperties::SetCustomShapeGuideValue( 
mrCustomShapeProperties.getGuideList(), aGuide );
             aAngles.Second.Type = EnhancedCustomShapeParameterType::EQUATION;
@@ -1234,10 +1234,10 @@ ContextHandlerRef 
CustomShapeGeometryContext::onCreateContext( sal_Int32 aElemen
         case A_TOKEN( rect ):           // CT_GeomRectList geometry rect list
         {
             GeomRect aGeomRect;
-            aGeomRect.l = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_l ).get() );
-            aGeomRect.t = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_t ).get() );
-            aGeomRect.r = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_r ).get() );
-            aGeomRect.b = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_b ).get() );
+            aGeomRect.l = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_l ).value() );
+            aGeomRect.t = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_t ).value() );
+            aGeomRect.r = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_r ).value() );
+            aGeomRect.b = GetAdjCoordinate( mrCustomShapeProperties, 
rAttribs.getString( XML_b ).value() );
             mrCustomShapeProperties.getTextRect() = aGeomRect;
         }
         break;
diff --git a/oox/source/drawingml/customshapeproperties.cxx 
b/oox/source/drawingml/customshapeproperties.cxx
index ff666b25ab6c..b0b6d34d23fe 100644
--- a/oox/source/drawingml/customshapeproperties.cxx
+++ b/oox/source/drawingml/customshapeproperties.cxx
@@ -216,8 +216,8 @@ void CustomShapeProperties::pushToPropSet(
 
         if ( maTextRect.has_value() ) {
             Sequence< EnhancedCustomShapeTextFrame > aTextFrames{
-                { /* tl */ { maTextRect.get().l, maTextRect.get().t },
-                  /* br */ { maTextRect.get().r, maTextRect.get().b } }
+                { /* tl */ { maTextRect.value().l, maTextRect.value().t },
+                  /* br */ { maTextRect.value().r, maTextRect.value().b } }
             };
             aPath.setProperty( PROP_TextFrames, aTextFrames);
         }
@@ -288,20 +288,20 @@ void CustomShapeProperties::pushToPropSet(
                 aHandle.setProperty( PROP_Position, maAdjustHandleList[ i 
].pos);
                 if ( maAdjustHandleList[ i ].gdRef1.has_value() )
                 {
-                    sal_Int32 nIndex = GetCustomShapeGuideValue( 
maAdjustmentGuideList, maAdjustHandleList[ i ].gdRef1.get() );
+                    sal_Int32 nIndex = GetCustomShapeGuideValue( 
maAdjustmentGuideList, maAdjustHandleList[ i ].gdRef1.value() );
                     if ( nIndex >= 0 )
                         aHandle.setProperty( PROP_RefR, nIndex);
                 }
                 if ( maAdjustHandleList[ i ].gdRef2.has_value() )
                 {
-                    sal_Int32 nIndex = GetCustomShapeGuideValue( 
maAdjustmentGuideList, maAdjustHandleList[ i ].gdRef2.get() );
+                    sal_Int32 nIndex = GetCustomShapeGuideValue( 
maAdjustmentGuideList, maAdjustHandleList[ i ].gdRef2.value() );
                     if ( nIndex >= 0 )
                         aHandle.setProperty( PROP_RefAngle, nIndex);
                 }
                 if ( maAdjustHandleList[ i ].min1.has_value() )
-                    aHandle.setProperty( PROP_RadiusRangeMinimum, 
maAdjustHandleList[ i ].min1.get());
+                    aHandle.setProperty( PROP_RadiusRangeMinimum, 
maAdjustHandleList[ i ].min1.value());
                 if ( maAdjustHandleList[ i ].max1.has_value() )
-                    aHandle.setProperty( PROP_RadiusRangeMaximum, 
maAdjustHandleList[ i ].max1.get());
+                    aHandle.setProperty( PROP_RadiusRangeMaximum, 
maAdjustHandleList[ i ].max1.value());
 
                 /* TODO: AngleMin & AngleMax
                 if ( maAdjustHandleList[ i ].min2.has() )
@@ -317,24 +317,24 @@ void CustomShapeProperties::pushToPropSet(
                 {
                     // TODO: PROP_RefX and PROP_RefY are not yet part of our 
file format,
                     // so the handles will not work after save/reload
-                    sal_Int32 nIndex = GetCustomShapeGuideValue( 
maAdjustmentGuideList, maAdjustHandleList[ i ].gdRef1.get() );
+                    sal_Int32 nIndex = GetCustomShapeGuideValue( 
maAdjustmentGuideList, maAdjustHandleList[ i ].gdRef1.value() );
                     if ( nIndex >= 0 )
                         aHandle.setProperty( PROP_RefX, nIndex);
                 }
                 if ( maAdjustHandleList[ i ].gdRef2.has_value() )
                 {
-                    sal_Int32 nIndex = GetCustomShapeGuideValue( 
maAdjustmentGuideList, maAdjustHandleList[ i ].gdRef2.get() );
+                    sal_Int32 nIndex = GetCustomShapeGuideValue( 
maAdjustmentGuideList, maAdjustHandleList[ i ].gdRef2.value() );
                     if ( nIndex >= 0 )
                         aHandle.setProperty( PROP_RefY, nIndex);
                 }
                 if ( maAdjustHandleList[ i ].min1.has_value() )
-                    aHandle.setProperty( PROP_RangeXMinimum, 
maAdjustHandleList[ i ].min1.get());
+                    aHandle.setProperty( PROP_RangeXMinimum, 
maAdjustHandleList[ i ].min1.value());
                 if ( maAdjustHandleList[ i ].max1.has_value() )
-                    aHandle.setProperty( PROP_RangeXMaximum, 
maAdjustHandleList[ i ].max1.get());
+                    aHandle.setProperty( PROP_RangeXMaximum, 
maAdjustHandleList[ i ].max1.value());
                 if ( maAdjustHandleList[ i ].min2.has_value() )
-                    aHandle.setProperty( PROP_RangeYMinimum, 
maAdjustHandleList[ i ].min2.get());
+                    aHandle.setProperty( PROP_RangeYMinimum, 
maAdjustHandleList[ i ].min2.value());
                 if ( maAdjustHandleList[ i ].max2.has_value() )
-                    aHandle.setProperty( PROP_RangeYMaximum, 
maAdjustHandleList[ i ].max2.get());
+                    aHandle.setProperty( PROP_RangeYMaximum, 
maAdjustHandleList[ i ].max2.value());
             }
             aHandlesRange[ i ] = aHandle.makePropertyValueSequence();
         }
diff --git a/oox/source/drawingml/diagram/datamodelcontext.cxx 
b/oox/source/drawingml/diagram/datamodelcontext.cxx
index e0ea3c93dc35..86f330467aa2 100644
--- a/oox/source/drawingml/diagram/datamodelcontext.cxx
+++ b/oox/source/drawingml/diagram/datamodelcontext.cxx
@@ -57,12 +57,12 @@ public:
                     svx::diagram::Connection& rConnection=mrConnection.back();
 
                     rConnection.mnXMLType = 
static_cast<svx::diagram::TypeConstant>(rAttribs.getToken( XML_type, XML_parOf 
));
-                    rConnection.msModelId = rAttribs.getString( XML_modelId 
).get();
-                    rConnection.msSourceId = rAttribs.getString( XML_srcId 
).get();
-                    rConnection.msDestId  = rAttribs.getString( XML_destId 
).get();
-                    rConnection.msPresId  = rAttribs.getString( XML_presId 
).get();
-                    rConnection.msSibTransId  = rAttribs.getString( 
XML_sibTransId ).get();
-                    rConnection.msParTransId  = rAttribs.getString( 
XML_parTransId ).get();
+                    rConnection.msModelId = rAttribs.getString( XML_modelId 
).value();
+                    rConnection.msSourceId = rAttribs.getString( XML_srcId 
).value();
+                    rConnection.msDestId  = rAttribs.getString( XML_destId 
).value();
+                    rConnection.msPresId  = rAttribs.getString( XML_presId 
).value();
+                    rConnection.msSibTransId  = rAttribs.getString( 
XML_sibTransId ).value();
+                    rConnection.msParTransId  = rAttribs.getString( 
XML_parTransId ).value();
                     rConnection.mnSourceOrder = rAttribs.getInteger( 
XML_srcOrd, 0 );
                     rConnection.mnDestOrder = rAttribs.getInteger( 
XML_destOrd, 0 );
 
@@ -117,7 +117,7 @@ public:
                 // need to convert from oox::OptValue to std::optional since 
1st is not available in svx
                 const OptValue< sal_Int32 > aOptVal(rAttribs.getToken( XML_val 
));
                 if(aOptVal.has_value())
-                    mrPoint.moHierarchyBranch = aOptVal.get();
+                    mrPoint.moHierarchyBranch = aOptVal.value();
                 break;
             }
             case DGM_TOKEN( orgChart ):
@@ -214,7 +214,7 @@ public:
         mrPoint( rPoint ),
         mrDiagramData( rDiagramData )
     {
-        mrPoint.msModelId = rAttribs.getString( XML_modelId ).get();
+        mrPoint.msModelId = rAttribs.getString( XML_modelId ).value();
 
         // the default type is XML_node
         const sal_Int32 nType  = rAttribs.getToken( XML_type, XML_node );
@@ -222,7 +222,7 @@ public:
 
         // ignore the cxnId unless it is this type. See 5.15.3.1.3 in Primer
         if( ( nType == XML_parTrans ) || ( nType == XML_sibTrans ) )
-            mrPoint.msCnxId = rAttribs.getString( XML_cxnId ).get();
+            mrPoint.msCnxId = rAttribs.getString( XML_cxnId ).value();
     }
 
     virtual ContextHandlerRef
@@ -371,7 +371,7 @@ DataModelContext::onCreateContext( ::sal_Int32 aElement,
     case A_TOKEN( ext ):
         break;
     case DSP_TOKEN( dataModelExt ):
-        mpDataModel->getExtDrawings().push_back( rAttribs.getString( XML_relId 
).get() );
+        mpDataModel->getExtDrawings().push_back( rAttribs.getString( XML_relId 
).value() );
         break;
     default:
         break;
diff --git a/oox/source/drawingml/diagram/diagramdefinitioncontext.cxx 
b/oox/source/drawingml/diagram/diagramdefinitioncontext.cxx
index 02d3777036d4..d569ed6d6dbe 100644
--- a/oox/source/drawingml/diagram/diagramdefinitioncontext.cxx
+++ b/oox/source/drawingml/diagram/diagramdefinitioncontext.cxx
@@ -37,14 +37,14 @@ DiagramDefinitionContext::DiagramDefinitionContext( 
ContextHandler2Helper const
     : ContextHandler2( rParent )
     , mpLayout( pLayout )
 {
-    mpLayout->setDefStyle( rAttributes.getString( XML_defStyle ).get() );
-    OUString sValue = rAttributes.getString( XML_minVer ).get();
+    mpLayout->setDefStyle( rAttributes.getString( XML_defStyle ).value() );
+    OUString sValue = rAttributes.getString( XML_minVer ).value();
     if( sValue.isEmpty() )
     {
         sValue = "http://schemas.openxmlformats.org/drawingml/2006/diagram";;
     }
     mpLayout->setMinVer( sValue );
-    mpLayout->setUniqueId( rAttributes.getString( XML_uniqueId ).get() );
+    mpLayout->setUniqueId( rAttributes.getString( XML_uniqueId ).value() );
 }
 
 DiagramDefinitionContext::~DiagramDefinitionContext()
@@ -61,18 +61,18 @@ DiagramDefinitionContext::onCreateContext( ::sal_Int32 
aElement,
     switch( aElement )
     {
     case DGM_TOKEN( title ):
-        mpLayout->setTitle( rAttribs.getString( XML_val ).get() );
+        mpLayout->setTitle( rAttribs.getString( XML_val ).value() );
         break;
     case DGM_TOKEN( desc ):
-        mpLayout->setDesc( rAttribs.getString( XML_val ).get() );
+        mpLayout->setDesc( rAttribs.getString( XML_val ).value() );
         break;
     case DGM_TOKEN( layoutNode ):
     {
         LayoutNodePtr pNode = 
std::make_shared<LayoutNode>(mpLayout->getDiagram());
         mpLayout->getNode() = pNode;
         pNode->setChildOrder( rAttribs.getToken( XML_chOrder, XML_b ) );
-        pNode->setMoveWith( rAttribs.getString( XML_moveWith ).get() );
-        pNode->setStyleLabel( rAttribs.getString( XML_styleLbl ).get() );
+        pNode->setMoveWith( rAttribs.getString( XML_moveWith ).value() );
+        pNode->setStyleLabel( rAttribs.getString( XML_styleLbl ).value() );
         return new LayoutNodeContext( *this, rAttribs, pNode );
     }
      case DGM_TOKEN( clrData ):
diff --git a/oox/source/drawingml/diagram/diagramlayoutatoms.cxx 
b/oox/source/drawingml/diagram/diagramlayoutatoms.cxx
index 08e5269b6b6c..3714c239f9ef 100644
--- a/oox/source/drawingml/diagram/diagramlayoutatoms.cxx
+++ b/oox/source/drawingml/diagram/diagramlayoutatoms.cxx
@@ -1516,12 +1516,12 @@ void AlgAtom::layoutShape(const ShapePtr& rShape, const 
std::vector<Constraint>&
                                 if (nDir == XML_fromL || nDir == XML_fromR)
                                 {
                                     oox::OptValue<sal_Int32> oWidth = 
findProperty(aProperties, aCurrShape->getInternalName(), XML_w);
-                                    bScaleDownEmptySpacing = 
oWidth.has_value() && oWidth.get() > 0;
+                                    bScaleDownEmptySpacing = 
oWidth.has_value() && oWidth.value() > 0;
                                 }
                                 if (!bScaleDownEmptySpacing && (nDir == 
XML_fromT || nDir == XML_fromB))
                                 {
                                     oox::OptValue<sal_Int32> oHeight = 
findProperty(aProperties, aCurrShape->getInternalName(), XML_h);
-                                    bScaleDownEmptySpacing = 
oHeight.has_value() && oHeight.get() > 0;
+                                    bScaleDownEmptySpacing = 
oHeight.has_value() && oHeight.value() > 0;
                                 }
                                 if (bScaleDownEmptySpacing && 
aCurrShape->getChildren().empty())
                                 {
@@ -1572,9 +1572,9 @@ void AlgAtom::layoutShape(const ShapePtr& rShape, const 
std::vector<Constraint>&
                 oox::OptValue<sal_Int32> oHeight = findProperty(aProperties, 
aCurrShape->getInternalName(), XML_h);
                 awt::Size aSize = aChildSize;
                 if (oWidth.has_value())
-                    aSize.Width = oWidth.get();
+                    aSize.Width = oWidth.value();
                 if (oHeight.has_value())
-                    aSize.Height = oHeight.get();
+                    aSize.Height = oHeight.value();
                 aTotalSize.Width += aSize.Width;
                 aTotalSize.Height += aSize.Height;
             }
@@ -1599,9 +1599,9 @@ void AlgAtom::layoutShape(const ShapePtr& rShape, const 
std::vector<Constraint>&
 
                 awt::Size aSize = aChildSize;
                 if (oWidth.has_value())
-                    aSize.Width = oWidth.get();
+                    aSize.Width = oWidth.value();
                 if (oHeight.has_value())
-                    aSize.Height = oHeight.get();
+                    aSize.Height = oHeight.value();
                 if (aChildrenToShrink.empty()
                     || aChildrenToShrink.find(aCurrShape->getInternalName())
                            != aChildrenToShrink.end())
diff --git a/oox/source/drawingml/diagram/layoutnodecontext.cxx 
b/oox/source/drawingml/diagram/layoutnodecontext.cxx
index 93f927531cf6..74ad212b318d 100644
--- a/oox/source/drawingml/diagram/layoutnodecontext.cxx
+++ b/oox/source/drawingml/diagram/layoutnodecontext.cxx
@@ -102,7 +102,7 @@ public:
         : ContextHandler2( rParent )
         , mpNode( pNode )
         {
-            msName = rAttribs.getString( XML_name ).get();
+            msName = rAttribs.getString( XML_name ).value();
         }
 
     virtual ContextHandlerRef
@@ -143,7 +143,7 @@ public:
     ForEachContext( ContextHandler2Helper const & rParent, const 
AttributeList& rAttribs, const ForEachAtomPtr& pAtom )
         : LayoutNodeContext( rParent, rAttribs, pAtom )
         {
-            pAtom->setRef(rAttribs.getString(XML_ref).get());
+            pAtom->setRef(rAttribs.getString(XML_ref).value());
             pAtom->iterator().loadFromXAttr( rAttribs.getFastAttributeList() );
 
             LayoutAtomMap& rLayoutAtomMap = 
pAtom->getLayoutNode().getDiagram().getLayout()->getLayoutAtomMap();
@@ -164,7 +164,7 @@ public:
 
     virtual ContextHandlerRef onCreateContext( ::sal_Int32 aElement, const 
AttributeList& rAttribs ) override
         {
-            mVariables[ getBaseToken(aElement) ] = rAttribs.getString( XML_val 
).get();
+            mVariables[ getBaseToken(aElement) ] = rAttribs.getString( XML_val 
).value();
             return this;
         }
 private:
@@ -181,7 +181,7 @@ LayoutNodeContext::LayoutNodeContext( ContextHandler2Helper 
const & rParent,
     , mpNode( pAtom )
 {
     assert( pAtom && "Node must NOT be NULL" );
-    mpNode->setName( rAttribs.getString( XML_name ).get() );
+    mpNode->setName( rAttribs.getString( XML_name ).value() );
 }
 
 LayoutNodeContext::~LayoutNodeContext()
@@ -216,8 +216,8 @@ LayoutNodeContext::onCreateContext( ::sal_Int32 aElement,
             }
         }
 
-        pNode->setMoveWith( rAttribs.getString( XML_moveWith ).get() );
-        pNode->setStyleLabel( rAttribs.getString( XML_styleLbl ).get() );
+        pNode->setMoveWith( rAttribs.getString( XML_moveWith ).value() );
+        pNode->setStyleLabel( rAttribs.getString( XML_styleLbl ).value() );
         return new LayoutNodeContext( *this, rAttribs, pNode );
     }
     case DGM_TOKEN( shape ):
diff --git a/oox/source/drawingml/embeddedwavaudiofile.cxx 
b/oox/source/drawingml/embeddedwavaudiofile.cxx
index 613e34ccbbe5..8f88733cfce9 100644
--- a/oox/source/drawingml/embeddedwavaudiofile.cxx
+++ b/oox/source/drawingml/embeddedwavaudiofile.cxx
@@ -28,9 +28,9 @@ namespace oox::drawingml
 OUString getEmbeddedWAVAudioFile(const core::Relations& rRelations, const 
AttributeList& rAttribs)
 {
     if (rAttribs.getBool(XML_builtIn, false))
-        return rAttribs.getString(XML_name).get();
+        return rAttribs.getString(XML_name).value();
     else
-        return 
rRelations.getFragmentPathFromRelId(rAttribs.getString(R_TOKEN(embed)).get());
+        return 
rRelations.getFragmentPathFromRelId(rAttribs.getString(R_TOKEN(embed)).value());
 }
 }
 
diff --git a/oox/source/drawingml/fillproperties.cxx 
b/oox/source/drawingml/fillproperties.cxx
index df2f09f03364..11f42457bc8d 100644
--- a/oox/source/drawingml/fillproperties.cxx
+++ b/oox/source/drawingml/fillproperties.cxx
@@ -345,7 +345,7 @@ void FillProperties::assignUsed( const FillProperties& 
rSourceProps )
 Color FillProperties::getBestSolidColor() const
 {
     Color aSolidColor;
-    if( moFillType.has_value() ) switch( moFillType.get() )
+    if( moFillType.has_value() ) switch( moFillType.value() )
     {
         case XML_solidFill:
             aSolidColor = maFillColor;
@@ -375,8 +375,8 @@ void FillProperties::pushToPropMap( ShapePropertyMap& 
rPropMap,
         return;
 
     FillStyle eFillStyle = FillStyle_NONE;
-    OSL_ASSERT((moFillType.get() & sal_Int32(0xFFFF0000))==0);
-    switch( moFillType.get() )
+    OSL_ASSERT((moFillType.value() & sal_Int32(0xFFFF0000))==0);
+    switch( moFillType.value() )
     {
         case XML_noFill:
         {
@@ -444,7 +444,7 @@ void FillProperties::pushToPropMap( ShapePropertyMap& 
rPropMap,
                     aGradient.YOffset = getLimitedValue<sal_Int16, sal_Int32>(
                         nCenterY / PER_PERCENT, 0, 100);
 
-                    if( maGradientProps.moGradientPath.get() == XML_circle )
+                    if( maGradientProps.moGradientPath.value() == XML_circle )
                     {
                         // Style should be radial at least when the horizontal 
center is at 50%.
                         // Otherwise import as a linear gradient, because it 
is the most similar to the MSO radial style.
@@ -785,7 +785,7 @@ void FillProperties::pushToPropMap( ShapePropertyMap& 
rPropMap,
                     }
                     else if ( eBitmapMode == BitmapMode_STRETCH && 
maBlipProps.moFillRect.has_value() )
                     {
-                        geometry::IntegerRectangle2D aFillRect( 
maBlipProps.moFillRect.get() );
+                        geometry::IntegerRectangle2D aFillRect( 
maBlipProps.moFillRect.value() );
                         awt::Size aOriginalSize( 
rGraphicHelper.getOriginalSize( xGraphic ) );
                         if ( aOriginalSize.Width && aOriginalSize.Height )
                         {
@@ -814,7 +814,7 @@ void FillProperties::pushToPropMap( ShapePropertyMap& 
rPropMap,
                 }
 
                 if (maBlipProps.moAlphaModFix.has_value())
-                    rPropMap.setProperty(ShapeProperty::FillTransparency, 
static_cast<sal_Int16>(100 - (maBlipProps.moAlphaModFix.get() / PER_PERCENT)));
+                    rPropMap.setProperty(ShapeProperty::FillTransparency, 
static_cast<sal_Int16>(100 - (maBlipProps.moAlphaModFix.value() / 
PER_PERCENT)));
             }
         break;
 
@@ -826,7 +826,7 @@ void FillProperties::pushToPropMap( ShapePropertyMap& 
rPropMap,
                 if( aColor.isUsed() && maPatternProps.moPattPreset.has_value() 
)
                 {
                     eFillStyle = FillStyle_HATCH;
-                    rPropMap.setProperty( ShapeProperty::FillHatch, 
createHatch( maPatternProps.moPattPreset.get(), aColor.getColor( 
rGraphicHelper, nPhClr ) ) );
+                    rPropMap.setProperty( ShapeProperty::FillHatch, 
createHatch( maPatternProps.moPattPreset.value(), aColor.getColor( 
rGraphicHelper, nPhClr ) ) );
                     if( aColor.hasTransparency() )
                         rPropMap.setProperty( ShapeProperty::FillTransparency, 
aColor.getTransparency() );
 
@@ -899,7 +899,7 @@ void GraphicProperties::pushToPropMap( PropertyMap& 
rPropMap, const GraphicHelpe
         // cropping
         if ( maBlipProps.moClipRect.has_value() )
         {
-            geometry::IntegerRectangle2D oClipRect( 
maBlipProps.moClipRect.get() );
+            geometry::IntegerRectangle2D oClipRect( 
maBlipProps.moClipRect.value() );
             awt::Size aOriginalSize( rGraphicHelper.getOriginalSize( xGraphic 
) );
             if ( aOriginalSize.Width && aOriginalSize.Height )
             {
@@ -955,7 +955,7 @@ void GraphicProperties::pushToPropMap( PropertyMap& 
rPropMap, const GraphicHelpe
 
         if ( maBlipProps.moAlphaModFix.has_value() )
         {
-            rPropMap.setProperty(PROP_Transparency, static_cast<sal_Int16>(100 
- (maBlipProps.moAlphaModFix.get() / PER_PERCENT)));
+            rPropMap.setProperty(PROP_Transparency, static_cast<sal_Int16>(100 
- (maBlipProps.moAlphaModFix.value() / PER_PERCENT)));
         }
     }
     rPropMap.setProperty(PROP_GraphicColorMode, eColorMode);
diff --git a/oox/source/drawingml/graphicshapecontext.cxx 
b/oox/source/drawingml/graphicshapecontext.cxx
index d90980a87824..bab0a7a65d91 100644
--- a/oox/source/drawingml/graphicshapecontext.cxx
+++ b/oox/source/drawingml/graphicshapecontext.cxx
@@ -96,7 +96,7 @@ ContextHandlerRef GraphicShapeContext::onCreateContext( 
sal_Int32 aElementToken,
     case XML_videoFile:
         {
             OUString rPath = getRelations().getFragmentPathFromRelId(
-                    rAttribs.getString(R_TOKEN(link)).get() );
+                    rAttribs.getString(R_TOKEN(link)).value() );
             if (!rPath.isEmpty())
             {
                 Reference<XInputStream> xMediaStream = 
lcl_GetMediaStream(rPath, getFilter());
@@ -110,7 +110,7 @@ ContextHandlerRef GraphicShapeContext::onCreateContext( 
sal_Int32 aElementToken,
             else
             {
                 rPath = getRelations().getExternalTargetFromRelId(
-                    rAttribs.getString(R_TOKEN(link)).get());
+                    rAttribs.getString(R_TOKEN(link)).value());
                 if (!rPath.isEmpty()) // linked media file
                     mpShapePtr->getGraphicProperties().m_sMediaPackageURL
                         = getFilter().getAbsoluteUrl(rPath);
@@ -154,7 +154,7 @@ ContextHandlerRef 
GraphicalObjectFrameContext::onCreateContext( sal_Int32 aEleme
 
         case XML_graphicData :          // CT_GraphicalObjectData
         {
-            OUString sUri( rAttribs.getString( XML_uri ).get() );
+            OUString sUri( rAttribs.getString( XML_uri ).value() );
             if ( sUri == 
"http://schemas.openxmlformats.org/presentationml/2006/ole"; ||
                     sUri == "http://purl.oclc.org/ooxml/presentationml/ole"; )
                 return new OleObjectGraphicDataContext( *this, mpShapePtr );
@@ -277,10 +277,10 @@ ContextHandlerRef 
DiagramGraphicDataContext::onCreateContext( ::sal_Int32 aEleme
     {
     case DGM_TOKEN( relIds ):
     {
-        msDm = rAttribs.getString( R_TOKEN( dm ) ).get();
-        msLo = rAttribs.getString( R_TOKEN( lo ) ).get();
-        msQs = rAttribs.getString( R_TOKEN( qs ) ).get();
-        msCs = rAttribs.getString( R_TOKEN( cs ) ).get();
+        msDm = rAttribs.getString( R_TOKEN( dm ) ).value();
+        msLo = rAttribs.getString( R_TOKEN( lo ) ).value();
+        msQs = rAttribs.getString( R_TOKEN( qs ) ).value();
+        msCs = rAttribs.getString( R_TOKEN( cs ) ).value();
         loadDiagram(mpShapePtr,
                     getFilter(),
                     getFragmentPathFromRelId( msDm ),
diff --git a/oox/source/drawingml/hyperlinkcontext.cxx 
b/oox/source/drawingml/hyperlinkcontext.cxx
index 352eb7bf5efa..1193a6c3af95 100644
--- a/oox/source/drawingml/hyperlinkcontext.cxx
+++ b/oox/source/drawingml/hyperlinkcontext.cxx
@@ -42,7 +42,7 @@ HyperLinkContext::HyperLinkContext( ContextHandler2Helper 
const & rParent,
     , maProperties(aProperties)
 {
     OUString sURL, sHref;
-    OUString aRelId = rAttribs.getString( R_TOKEN( id ) ).get();
+    OUString aRelId = rAttribs.getString( R_TOKEN( id ) ).value();
     if ( !aRelId.isEmpty() )
     {
         sHref = getRelations().getExternalTargetFromRelId( aRelId );
@@ -56,13 +56,13 @@ HyperLinkContext::HyperLinkContext( ContextHandler2Helper 
const & rParent,
             sURL = getRelations().getInternalTargetFromRelId( aRelId );
         }
     }
-    OUString sTooltip = rAttribs.getString( R_TOKEN( tooltip ) ).get();
+    OUString sTooltip = rAttribs.getString( R_TOKEN( tooltip ) ).value();
     if ( !sTooltip.isEmpty() )
         maProperties.setProperty(PROP_Representation, sTooltip);
-    OUString sFrame = rAttribs.getString( R_TOKEN( tgtFrame ) ).get();
+    OUString sFrame = rAttribs.getString( R_TOKEN( tgtFrame ) ).value();
     if( !sFrame.isEmpty() )
         maProperties.setProperty(PROP_TargetFrame, sFrame);
-    OUString aAction = rAttribs.getString( XML_action ).get();
+    OUString aAction = rAttribs.getString( XML_action ).value();
     if ( !aAction.isEmpty() )
     {
         // reserved values of the unrestricted string aAction:
diff --git a/oox/source/drawingml/lineproperties.cxx 
b/oox/source/drawingml/lineproperties.cxx
index 12f74b845ef5..e1cef771c24e 100644
--- a/oox/source/drawingml/lineproperties.cxx
+++ b/oox/source/drawingml/lineproperties.cxx
@@ -314,8 +314,8 @@ void lclPushMarkerProperties( ShapePropertyMap& rPropMap,
             const double fArrowLineHalfWidth = ::std::max< double >( 100.0 * 
0.5 * nLineWidth / nMarkerWidth, 1 );
 
             ::std::vector< awt::Point > aPoints;
-            OSL_ASSERT((rArrowProps.moArrowType.get() & 
sal_Int32(0xFFFF0000))==0);
-            switch( rArrowProps.moArrowType.get() )
+            OSL_ASSERT((rArrowProps.moArrowType.value() & 
sal_Int32(0xFFFF0000))==0);
+            switch( rArrowProps.moArrowType.value() )
             {
                 case XML_triangle:
                     aPoints.push_back( OOX_ARROW_POINT(  50,   0 ) );
@@ -437,25 +437,25 @@ void LineProperties::pushToPropMap( ShapePropertyMap& 
rPropMap,
         return;
 
     // line style (our core only supports none and solid)
-    drawing::LineStyle eLineStyle = (maLineFill.moFillType.get() == 
XML_noFill) ? drawing::LineStyle_NONE : drawing::LineStyle_SOLID;
+    drawing::LineStyle eLineStyle = (maLineFill.moFillType.value() == 
XML_noFill) ? drawing::LineStyle_NONE : drawing::LineStyle_SOLID;
 
     // line width in 1/100mm
     sal_Int32 nLineWidth = getLineWidth(); // includes conversion from EMUs to 
1/100mm
     rPropMap.setProperty( ShapeProperty::LineWidth, nLineWidth );
 
     // line cap type
-    LineCap eLineCap = moLineCap.has_value() ? lclGetLineCap( moLineCap.get() 
) : LineCap_BUTT;
+    LineCap eLineCap = moLineCap.has_value() ? lclGetLineCap( 
moLineCap.value() ) : LineCap_BUTT;
     if( moLineCap.has_value() )
         rPropMap.setProperty( ShapeProperty::LineCap, eLineCap );
 
     // create line dash from preset dash token or dash stop vector (not for 
invisible line)
     if( (eLineStyle != drawing::LineStyle_NONE) &&
-        ((moPresetDash.has_value() && moPresetDash.get() != XML_solid) || 
!maCustomDash.empty()) )
+        ((moPresetDash.has_value() && moPresetDash.value() != XML_solid) || 
!maCustomDash.empty()) )
     {
         LineDash aLineDash;
         aLineDash.Style = lclGetDashStyle( moLineCap.value_or( XML_flat ) );
 
-        if(moPresetDash.has_value() && moPresetDash.get() != XML_solid)
+        if(moPresetDash.has_value() && moPresetDash.value() != XML_solid)
             lclConvertPresetDash(aLineDash, moPresetDash.value_or(XML_dash));
         else // !maCustomDash.empty()
         {
@@ -487,7 +487,7 @@ void LineProperties::pushToPropMap( ShapePropertyMap& 
rPropMap,
 
     // line joint type
     if( moLineJoint.has_value() )
-        rPropMap.setProperty( ShapeProperty::LineJoint, lclGetLineJoint( 
moLineJoint.get() ) );
+        rPropMap.setProperty( ShapeProperty::LineJoint, lclGetLineJoint( 
moLineJoint.value() ) );
 
     // line color and transparence
     Color aLineColor = maLineFill.getBestSolidColor();
@@ -506,9 +506,9 @@ void LineProperties::pushToPropMap( ShapePropertyMap& 
rPropMap,
 drawing::LineStyle LineProperties::getLineStyle() const
 {
     // rules to calculate the line style inferred from the code in 
LineProperties::pushToPropMap
-    if (maLineFill.moFillType.get() == XML_noFill)
+    if (maLineFill.moFillType.value() == XML_noFill)
         return drawing::LineStyle_NONE;
-    if ((moPresetDash.has_value() && moPresetDash.get() != XML_solid) ||
+    if ((moPresetDash.has_value() && moPresetDash.value() != XML_solid) ||
         (!moPresetDash && !maCustomDash.empty()))
        return drawing::LineStyle_DASH;
     return drawing::LineStyle_SOLID;
@@ -517,7 +517,7 @@ drawing::LineStyle LineProperties::getLineStyle() const
 drawing::LineCap LineProperties::getLineCap() const
 {
     if( moLineCap.has_value() )
-        return lclGetLineCap( moLineCap.get() );
+        return lclGetLineCap( moLineCap.value() );
 
     return drawing::LineCap_BUTT;
 }
@@ -525,7 +525,7 @@ drawing::LineCap LineProperties::getLineCap() const
 drawing::LineJoint LineProperties::getLineJoint() const
 {
     if( moLineJoint.has_value() )
-        return lclGetLineJoint( moLineJoint.get() );
+        return lclGetLineJoint( moLineJoint.value() );
 
     return drawing::LineJoint_NONE;
 }
diff --git a/oox/source/drawingml/shape.cxx b/oox/source/drawingml/shape.cxx
index 774ca831e35d..236e0bea3c82 100644
--- a/oox/source/drawingml/shape.cxx
+++ b/oox/source/drawingml/shape.cxx
@@ -643,17 +643,17 @@ static void lcl_createPresetShape(const 
uno::Reference<drawing::XShape>& xShape,
         std::shared_ptr<TextRun> pRun = pParagraph->getRuns()[0];
         TextCharacterProperties& pProperties = 
pRun->getTextCharacterProperties();
 
-        if (pProperties.moBold.has_value() && pProperties.moBold.get())
+        if (pProperties.moBold.has_value() && pProperties.moBold.value())
         {
             xSet->setPropertyValue( UNO_NAME_CHAR_WEIGHT, uno::Any( 
css::awt::FontWeight::BOLD ) );
         }
-        if (pProperties.moItalic.has_value() && pProperties.moItalic.get())
+        if (pProperties.moItalic.has_value() && pProperties.moItalic.value())
         {
             xSet->setPropertyValue( UNO_NAME_CHAR_POSTURE, uno::Any( 
css::awt::FontSlant::FontSlant_ITALIC ) );
         }
         if (pProperties.moHeight.has_value())
         {
-            sal_Int32 nHeight = pProperties.moHeight.get() / 100;
+            sal_Int32 nHeight = pProperties.moHeight.value() / 100;
             xSet->setPropertyValue( UNO_NAME_CHAR_HEIGHT, uno::Any( nHeight ) 
);
         }
         if (pProperties.maFillProperties.maFillColor.isUsed())
@@ -1240,7 +1240,7 @@ Reference< XShape > const & Shape::createAndInsert(
             aShapeProps.assignUsed( 
mpTextBody->getTextProperties().maPropertyMap );
             // Push char properties as well - specifically useful when this is 
a placeholder
             if( mpMasterTextListStyle &&  
mpMasterTextListStyle->getListStyle()[0].getTextCharacterProperties().moHeight.has_value()
 )
-                aShapeProps.setProperty(PROP_CharHeight, GetFontHeight( 
mpMasterTextListStyle->getListStyle()[0].getTextCharacterProperties().moHeight.get()
 ));
+                aShapeProps.setProperty(PROP_CharHeight, GetFontHeight( 
mpMasterTextListStyle->getListStyle()[0].getTextCharacterProperties().moHeight.value()
 ));
         }
 
         // applying properties
@@ -1259,7 +1259,7 @@ Reference< XShape > const & Shape::createAndInsert(
             mpTablePropertiesPtr->pushToPropSet( rFilterBase, xSet, 
mpMasterTextListStyle );
 
         FillProperties aFillProperties = getActualFillProperties(pTheme, 
&rShapeOrParentShapeFillProps);
-        if (getFillProperties().moFillType.has_value() && 
getFillProperties().moFillType.get() == XML_grpFill)
+        if (getFillProperties().moFillType.has_value() && 
getFillProperties().moFillType.value() == XML_grpFill)
             getFillProperties().assignUsed(aFillProperties);
         if(!bIsCroppedGraphic)
             aFillProperties.pushToPropMap( aShapeProps, rGraphicHelper, 
mnRotation, nFillPhClr, nFillPhClrTheme, mbFlipH, mbFlipV, bIsCustomShape );
@@ -1359,7 +1359,7 @@ Reference< XShape > const & Shape::createAndInsert(
                         css::table::BorderLine2 aBorderLine = 
xPropertySet->getPropertyValue(PropertyMap::getPropertyName(nBorder)).get<css::table::BorderLine2>();
                         aBorderLine.Color = 
aShapeProps.getProperty(PROP_LineColor).get<sal_Int32>();
                         if (aLineProperties.moLineWidth.has_value())
-                            aBorderLine.LineWidth = 
convertEmuToHmm(aLineProperties.moLineWidth.get());
+                            aBorderLine.LineWidth = 
convertEmuToHmm(aLineProperties.moLineWidth.value());
                         aShapeProps.setProperty(nBorder, aBorderLine);
                     }
                     aShapeProps.erase(PROP_LineColor);
@@ -1413,7 +1413,7 @@ Reference< XShape > const & Shape::createAndInsert(
                     if (aEffectProperties.maShadow.moShadowDir.has_value())
                     {
                         css::table::ShadowLocation nLocation = 
css::table::ShadowLocation_NONE;
-                        switch (aEffectProperties.maShadow.moShadowDir.get())
+                        switch (aEffectProperties.maShadow.moShadowDir.value())
                         {
                         case 13500000:
                             nLocation = css::table::ShadowLocation_TOP_LEFT;
@@ -1685,7 +1685,7 @@ Reference< XShape > const & Shape::createAndInsert(
                 mpCustomShapePropertiesPtr->setMirroredY( true );
             if( getTextBody() )
             {
-                sal_Int32 nTextCameraZRotation = static_cast< sal_Int32 >( 
getTextBody()->get3DProperties().maCameraRotation.mnRevolution.get() );
+                sal_Int32 nTextCameraZRotation = static_cast< sal_Int32 >( 
getTextBody()->get3DProperties().maCameraRotation.mnRevolution.value() );
                 mpCustomShapePropertiesPtr->setTextCameraZRotateAngle( 
nTextCameraZRotation / 60000 );
 
                 sal_Int32 nTextRotateAngle = static_cast< sal_Int32 >( 
getTextBody()->getTextProperties().moRotation.value_or( 0 ) );
@@ -1812,7 +1812,7 @@ Reference< XShape > const & Shape::createAndInsert(
         if ( aEffectProperties.maGlow.moGlowRad.has_value() )
         {
             uno::Reference<beans::XPropertySet> propertySet (mxShape, 
uno::UNO_QUERY);
-            propertySet->setPropertyValue("GlowEffectRadius", 
Any(convertEmuToHmm(aEffectProperties.maGlow.moGlowRad.get())));
+            propertySet->setPropertyValue("GlowEffectRadius", 
Any(convertEmuToHmm(aEffectProperties.maGlow.moGlowRad.value())));
             propertySet->setPropertyValue("GlowEffectColor", 
Any(aEffectProperties.maGlow.moGlowColor.getColor(rGraphicHelper)));
             propertySet->setPropertyValue("GlowEffectTransparency", 
Any(aEffectProperties.maGlow.moGlowColor.getTransparency()));
         }
@@ -1822,7 +1822,7 @@ Reference< XShape > const & Shape::createAndInsert(
         {
             uno::Reference<beans::XPropertySet> propertySet(mxShape, 
uno::UNO_QUERY);
             propertySet->setPropertyValue(
-                "SoftEdgeRadius", 
Any(convertEmuToHmm(aEffectProperties.maSoftEdge.moRad.get())));
+                "SoftEdgeRadius", 
Any(convertEmuToHmm(aEffectProperties.maSoftEdge.moRad.value())));
         }
     }
 
@@ -2196,7 +2196,7 @@ FillProperties Shape::getActualFillProperties(const 
Theme* pTheme, const FillPro
 
     // Parent shape's properties
     if ( pParentShapeFillProps != nullptr)
-        if( getFillProperties().moFillType.has_value() && 
getFillProperties().moFillType.get() == XML_grpFill )
+        if( getFillProperties().moFillType.has_value() && 
getFillProperties().moFillType.value() == XML_grpFill )
             aFillProperties.assignUsed( *pParentShapeFillProps );
 
     return aFillProperties;
diff --git a/oox/source/drawingml/shapecontext.cxx 
b/oox/source/drawingml/shapecontext.cxx
index e0cbf9343c1d..5cf910b3c07b 100644
--- a/oox/source/drawingml/shapecontext.cxx
+++ b/oox/source/drawingml/shapecontext.cxx
@@ -67,9 +67,9 @@ ContextHandlerRef ShapeContext::onCreateContext( sal_Int32 
aElementToken, const
     case XML_cNvPr:
     {
         mpShapePtr->setHidden( rAttribs.getBool( XML_hidden, false ) );
-        mpShapePtr->setId( rAttribs.getString( XML_id ).get() );
-        mpShapePtr->setName( rAttribs.getString( XML_name ).get() );
-        mpShapePtr->setDescription( rAttribs.getString( XML_descr ).get() );
+        mpShapePtr->setId( rAttribs.getString( XML_id ).value() );
+        mpShapePtr->setName( rAttribs.getString( XML_name ).value() );
+        mpShapePtr->setDescription( rAttribs.getString( XML_descr ).value() );
         break;
     }
     case XML_hlinkMouseOver:
@@ -78,7 +78,7 @@ ContextHandlerRef ShapeContext::onCreateContext( sal_Int32 
aElementToken, const
     case XML_ph:
         mpShapePtr->setSubType( rAttribs.getToken( XML_type, XML_obj ) );
         if( rAttribs.hasAttribute( XML_idx ) )
-            mpShapePtr->setSubTypeIndex( rAttribs.getString( XML_idx 
).get().toInt32() );
+            mpShapePtr->setSubTypeIndex( rAttribs.getString( XML_idx 
).value().toInt32() );
         break;
     // nvSpPr CT_ShapeNonVisual end
 
diff --git a/oox/source/drawingml/shapegroupcontext.cxx 
b/oox/source/drawingml/shapegroupcontext.cxx
index 1a9cb4113871..d0c53a34ad26 100644
--- a/oox/source/drawingml/shapegroupcontext.cxx
+++ b/oox/source/drawingml/shapegroupcontext.cxx
@@ -60,14 +60,14 @@ ContextHandlerRef ShapeGroupContext::onCreateContext( 
sal_Int32 aElementToken, c
     case XML_cNvPr:
     {
         mpGroupShapePtr->setHidden( rAttribs.getBool( XML_hidden, false ) );
-        mpGroupShapePtr->setId( rAttribs.getString( XML_id ).get() );
-        mpGroupShapePtr->setName( rAttribs.getString( XML_name ).get() );
+        mpGroupShapePtr->setId( rAttribs.getString( XML_id ).value() );
+        mpGroupShapePtr->setName( rAttribs.getString( XML_name ).value() );
         break;
     }
     case XML_ph:
         mpGroupShapePtr->setSubType( rAttribs.getToken( XML_type, 
FastToken::DONTKNOW ) );
         if( rAttribs.hasAttribute( XML_idx ) )
-            mpGroupShapePtr->setSubTypeIndex( rAttribs.getString( XML_idx 
).get().toInt32() );
+            mpGroupShapePtr->setSubTypeIndex( rAttribs.getString( XML_idx 
).value().toInt32() );
         break;
     // nvSpPr CT_ShapeNonVisual end
 
diff --git a/oox/source/drawingml/table/tablecell.cxx 
b/oox/source/drawingml/table/tablecell.cxx
index f2e8fc336937..8cff610763fc 100644
--- a/oox/source/drawingml/table/tablecell.cxx
+++ b/oox/source/drawingml/table/tablecell.cxx
@@ -64,7 +64,7 @@ static void applyLineAttributes( const 
::oox::core::XmlFilterBase& rFilterBase,
         sal_Int32 nPropId )
 {
     BorderLine2 aBorderLine;
-    if ( rLineProperties.maLineFill.moFillType.has_value() && 
rLineProperties.maLineFill.moFillType.get() != XML_noFill )
+    if ( rLineProperties.maLineFill.moFillType.has_value() && 
rLineProperties.maLineFill.moFillType.value() != XML_noFill )
     {
         Color aColor = rLineProperties.maLineFill.getBestSolidColor();
         aBorderLine.Color = sal_Int32(aColor.getColor( 
rFilterBase.getGraphicHelper() ));
@@ -84,7 +84,7 @@ static void applyLineAttributes( const 
::oox::core::XmlFilterBase& rFilterBase,
 
     if ( rLineProperties.moPresetDash.has_value() )
     {
-        switch ( rLineProperties.moPresetDash.get() )
+        switch ( rLineProperties.moPresetDash.value() )
         {
         case XML_dot:
         case XML_sysDot:
@@ -502,7 +502,7 @@ void TableCell::pushToXCell( const 
::oox::core::XmlFilterBase& rFilterBase, cons
         applyLineAttributes( rFilterBase, xPropSet, aLinePropertiesInsideV, 
PROP_RightBorder );
     }
 
-    if (rProperties.getBgColor().isUsed() && 
!maFillProperties.maFillColor.isUsed() && maFillProperties.moFillType.get() == 
XML_noFill)
+    if (rProperties.getBgColor().isUsed() && 
!maFillProperties.maFillColor.isUsed() && maFillProperties.moFillType.value() 
== XML_noFill)
     {
         maFillProperties.moFillType = XML_solidFill;
         maFillProperties.maFillColor = rProperties.getBgColor();
diff --git a/oox/source/drawingml/table/tablecellcontext.cxx 
b/oox/source/drawingml/table/tablecellcontext.cxx
index dde824fb72f9..e02889514119 100644
--- a/oox/source/drawingml/table/tablecellcontext.cxx
+++ b/oox/source/drawingml/table/tablecellcontext.cxx
@@ -36,9 +36,9 @@ TableCellContext::TableCellContext( ContextHandler2Helper 
const & rParent, const
 , mrTableCell( rTableCell )
 {
     if ( rAttribs.hasAttribute( XML_rowSpan ) )
-        mrTableCell.setRowSpan( rAttribs.getString( XML_rowSpan 
).get().toInt32() );
+        mrTableCell.setRowSpan( rAttribs.getString( XML_rowSpan 
).value().toInt32() );
     if ( rAttribs.hasAttribute( XML_gridSpan ) )
-        mrTableCell.setGridSpan( rAttribs.getString( XML_gridSpan 
).get().toInt32() );
+        mrTableCell.setGridSpan( rAttribs.getString( XML_gridSpan 
).value().toInt32() );
 
     mrTableCell.sethMerge( rAttribs.getBool( XML_hMerge, false ) );
     mrTableCell.setvMerge( rAttribs.getBool( XML_vMerge, false ) );
diff --git a/oox/source/drawingml/table/tablecontext.cxx 
b/oox/source/drawingml/table/tablecontext.cxx
index 84e80dccd710..b57af6d24f55 100644
--- a/oox/source/drawingml/table/tablecontext.cxx
+++ b/oox/source/drawingml/table/tablecontext.cxx
@@ -79,7 +79,7 @@ TableContext::onCreateContext( ::sal_Int32 aElementToken, 
const AttributeList& r
     case A_TOKEN( gridCol ):            // CT_TableCol
         {
             std::vector< sal_Int32 >& rvTableGrid( 
mrTableProperties.getTableGrid() );
-            rvTableGrid.push_back( rAttribs.getString( XML_w ).get().toInt32() 
);
+            rvTableGrid.push_back( rAttribs.getString( XML_w 
).value().toInt32() );
         }
         break;
     case A_TOKEN( tr ):                 // CT_TableRow
diff --git a/oox/source/drawingml/table/tablerowcontext.cxx 
b/oox/source/drawingml/table/tablerowcontext.cxx
index aafd13b00e20..c6a34afa54b4 100644
--- a/oox/source/drawingml/table/tablerowcontext.cxx
+++ b/oox/source/drawingml/table/tablerowcontext.cxx
@@ -34,7 +34,7 @@ TableRowContext::TableRowContext( ContextHandler2Helper const 
& rParent, const A
 : ContextHandler2( rParent )
 , mrTableRow( rTableRow )
 {
-    rTableRow.setHeight( rAttribs.getString( XML_h ).get().toInt32() );
+    rTableRow.setHeight( rAttribs.getString( XML_h ).value().toInt32() );
 }
 
 TableRowContext::~TableRowContext()
diff --git a/oox/source/drawingml/table/tablestylecontext.cxx 
b/oox/source/drawingml/table/tablestylecontext.cxx
index 7dc4ce468cf1..242889b63d80 100644
--- a/oox/source/drawingml/table/tablestylecontext.cxx
+++ b/oox/source/drawingml/table/tablestylecontext.cxx
@@ -37,8 +37,8 @@ TableStyleContext::TableStyleContext( ContextHandler2Helper 
const & rParent,
 : ContextHandler2( rParent )
 , mrTableStyle( rTableStyle )
 {
-    mrTableStyle.getStyleId() = rAttribs.getString( XML_styleId ).get();
-    mrTableStyle.getStyleName() = rAttribs.getString( XML_styleName ).get();
+    mrTableStyle.getStyleId() = rAttribs.getString( XML_styleId ).value();
+    mrTableStyle.getStyleName() = rAttribs.getString( XML_styleName ).value();
 }
 
 TableStyleContext::~TableStyleContext()
diff --git a/oox/source/drawingml/table/tablestylelistfragmenthandler.cxx 
b/oox/source/drawingml/table/tablestylelistfragmenthandler.cxx
index 77dc4267757d..20ff3e0dc105 100644
--- a/oox/source/drawingml/table/tablestylelistfragmenthandler.cxx
+++ b/oox/source/drawingml/table/tablestylelistfragmenthandler.cxx
@@ -48,7 +48,7 @@ ContextHandlerRef 
TableStyleListFragmentHandler::onCreateContext(
     switch( aElementToken )
     {
         case A_TOKEN( tblStyleLst ):    // CT_TableStyleList
-            mrTableStyleList.getDefaultStyleId() = rAttribs.getString( XML_def 
).get();
+            mrTableStyleList.getDefaultStyleId() = rAttribs.getString( XML_def 
).value();
             break;
         case A_TOKEN( tblStyle ):       // CT_TableStyle
             std::vector< TableStyle >& rTableStyles = 
mrTableStyleList.getTableStyles();
diff --git a/oox/source/drawingml/textbodypropertiescontext.cxx 
b/oox/source/drawingml/textbodypropertiescontext.cxx
index a2a0d365b0c2..7a0a0212ef51 100644
--- a/oox/source/drawingml/textbodypropertiescontext.cxx
+++ b/oox/source/drawingml/textbodypropertiescontext.cxx
@@ -68,7 +68,7 @@ TextBodyPropertiesContext::TextBodyPropertiesContext( 
ContextHandler2Helper cons
     sal_Int32 aIns[] = { XML_lIns, XML_tIns, XML_rIns, XML_bIns };
     for( sal_Int32 i = 0; i < sal_Int32(SAL_N_ELEMENTS( aIns )); i++)
     {
-        sValue = rAttribs.getString( aIns[i] ).get();
+        sValue = rAttribs.getString( aIns[i] ).value();
         if( !sValue.isEmpty() )
             mrTextBodyProp.moInsets[i] = GetCoordinate( sValue );
     }
@@ -151,7 +151,7 @@ ContextHandlerRef 
TextBodyPropertiesContext::onCreateContext( sal_Int32 aElement
                     const OptValue<OUString> sPrst = rAttribs.getString( 
XML_prst );
                     if( sPrst.has_value() )
                     {
-                        mrTextBodyProp.msPrst = sPrst.get();
+                        mrTextBodyProp.msPrst = sPrst.value();
                         if( mrTextBodyProp.msPrst != "textNoShape" )
                             return new PresetTextShapeContext( *this, rAttribs,
                                                            *( 
mpShapePtr->getCustomShapeProperties() ) );
diff --git a/oox/source/drawingml/textcharacterproperties.cxx 
b/oox/source/drawingml/textcharacterproperties.cxx
index b99723e6cd01..c38906308ddd 100644
--- a/oox/source/drawingml/textcharacterproperties.cxx
+++ b/oox/source/drawingml/textcharacterproperties.cxx
@@ -114,16 +114,16 @@ void TextCharacterProperties::pushToPropMap( PropertyMap& 
rPropMap, const XmlFil
         bool bContoured = false;
 
         // noFill doesn't exist for characters. Map noFill to 99% transparency
-        if (maFillProperties.moFillType.get() == XML_noFill)
+        if (maFillProperties.moFillType.value() == XML_noFill)
             aColor.addTransformation(XML_alpha, 1000);
 
         // tdf#137438 Emulate text outline color/transparency.
         // If the outline color dominates, then use it as the text color.
         if (moTextOutlineProperties.has_value()
-            && moTextOutlineProperties.get().maLineFill.moFillType.has_value()
-            && moTextOutlineProperties.get().maLineFill.moFillType.get() != 
XML_noFill)
+            && 
moTextOutlineProperties.value().maLineFill.moFillType.has_value()
+            && moTextOutlineProperties.value().maLineFill.moFillType.value() 
!= XML_noFill)
         {
-            Color aLineColor = 
moTextOutlineProperties.get().maLineFill.getBestSolidColor();
+            Color aLineColor = 
moTextOutlineProperties.value().maLineFill.getBestSolidColor();
             sal_Int16 nLineTransparency = aLineColor.getTransparency();
 
             // tdf#127696 If the text color is white (and the outline color 
doesn't dominate),
@@ -146,9 +146,9 @@ void TextCharacterProperties::pushToPropMap( PropertyMap& 
rPropMap, const XmlFil
         }
     }
 
-    if( moLang.has_value() && !moLang.get().isEmpty() )
+    if( moLang.has_value() && !moLang.value().isEmpty() )
     {
-        LanguageTag aTag(moLang.get());
+        LanguageTag aTag(moLang.value());
         lang::Locale aLocale(aTag.getLocale());
         switch(MsLangId::getScriptType(aTag.getLanguageType()))
         {
@@ -163,9 +163,9 @@ void TextCharacterProperties::pushToPropMap( PropertyMap& 
rPropMap, const XmlFil
 
     if( moHeight.has_value() )
     {
-        float fHeight = GetFontHeight( moHeight.get() );
+        float fHeight = GetFontHeight( moHeight.value() );
         if (moFontScale.has_value())
-            fHeight *= (moFontScale.get() / 100000);
+            fHeight *= (moFontScale.value() / 100000);
         rPropMap.setProperty( PROP_CharHeight, fHeight);
         rPropMap.setProperty( PROP_CharHeightAsian, fHeight);
         rPropMap.setProperty( PROP_CharHeightComplex, fHeight);
@@ -234,7 +234,7 @@ void TextCharacterProperties::pushToPropSet( PropertySet& 
rPropSet, const XmlFil
 
 float TextCharacterProperties::getCharHeightPoints( float fDefault ) const
 {
-    return moHeight.has_value() ? GetFontHeight( moHeight.get() ) : fDefault;
+    return moHeight.has_value() ? GetFontHeight( moHeight.value() ) : fDefault;
 }
 
 } // namespace oox::drawingml
diff --git a/oox/source/drawingml/textcharacterpropertiescontext.cxx 
b/oox/source/drawingml/textcharacterpropertiescontext.cxx
index f6aef3c92987..a13803e5319d 100644
--- a/oox/source/drawingml/textcharacterpropertiescontext.cxx
+++ b/oox/source/drawingml/textcharacterpropertiescontext.cxx
@@ -67,7 +67,7 @@ 
TextCharacterPropertiesContext::TextCharacterPropertiesContext(
         mrTextCharacterProperties.moUnderline = rAttribs.getToken( XML_u );
     if ( rAttribs.hasAttribute( XML_strike ) )
         mrTextCharacterProperties.moStrikeout = rAttribs.getToken( XML_strike 
);
-    if ( rAttribs.hasAttribute( XML_baseline ) && rAttribs.getInteger( 
XML_baseline ).get() != 0 )
+    if ( rAttribs.hasAttribute( XML_baseline ) && rAttribs.getInteger( 
XML_baseline ).value() != 0 )
         mrTextCharacterProperties.moBaseline = rAttribs.getInteger( 
XML_baseline );
 
     if ( rAttribs.hasAttribute( XML_b ) )
@@ -233,7 +233,7 @@ ContextHandlerRef 
TextCharacterPropertiesContext::onCreateContext( sal_Int32 aEl
             if (colorAttrib.has_value())
             {
                 oox::drawingml::Color theColor;
-                theColor.setSrgbClr(colorAttrib.get());
+                theColor.setSrgbClr(colorAttrib.value());
                 mrTextCharacterProperties.maUnderlineColor = theColor;
             }
             break;
@@ -263,14 +263,14 @@ ContextHandlerRef 
TextCharacterPropertiesContext::onCreateContext( sal_Int32 aEl
         case W_TOKEN( color ):
             if (rAttribs.getInteger(W_TOKEN(val)).has_value())
             {
-                
mrTextCharacterProperties.maFillProperties.maFillColor.setSrgbClr(rAttribs.getIntegerHex(W_TOKEN(val)).get());
+                
mrTextCharacterProperties.maFillProperties.maFillColor.setSrgbClr(rAttribs.getIntegerHex(W_TOKEN(val)).value());
                 mrTextCharacterProperties.maFillProperties.moFillType = 
XML_solidFill;
             }
             break;
         case W_TOKEN(  sz ):
             if (rAttribs.getInteger(W_TOKEN(val)).has_value())
             {
-                sal_Int32 nVal = rAttribs.getInteger(W_TOKEN(val)).get();
+                sal_Int32 nVal = rAttribs.getInteger(W_TOKEN(val)).value();
                 // wml has half points, dml has hundred points
                 mrTextCharacterProperties.moHeight = nVal * 50;
             }
diff --git a/oox/source/drawingml/texteffectscontext.cxx 
b/oox/source/drawingml/texteffectscontext.cxx
index e380b6425413..61529a2c9b9c 100644
--- a/oox/source/drawingml/texteffectscontext.cxx
+++ b/oox/source/drawingml/texteffectscontext.cxx
@@ -119,7 +119,7 @@ void TextEffectsContext::pushAttributeToGrabBag (sal_Int32 
aAttributeId, const O
 {
     if (!rAttribs.hasAttribute(aAttributeId))
         return;
-    OUString aString = rAttribs.getString(aAttributeId).get();
+    OUString aString = rAttribs.getString(aAttributeId).value();
     mpGrabBagStack->addString(rElementName, aString);
 }
 
diff --git a/oox/source/drawingml/textfield.cxx 
b/oox/source/drawingml/textfield.cxx
index 76615409901f..ee2f9313b2d8 100644
--- a/oox/source/drawingml/textfield.cxx
+++ b/oox/source/drawingml/textfield.cxx
@@ -170,7 +170,7 @@ sal_Int32 TextField::insertAt(
         aTextCharacterProps.assignUsed( 
maTextParagraphProperties.getTextCharacterProperties() );
         aTextCharacterProps.assignUsed( getTextCharacterProperties() );
         if ( aTextCharacterProps.moHeight.has_value() )
-            nCharHeight = aTextCharacterProps.moHeight.get();
+            nCharHeight = aTextCharacterProps.moHeight.value();
         aTextCharacterProps.pushToPropSet( aPropSet, rFilterBase );
 
         std::vector< Reference< XTextField > > fields;
diff --git a/oox/source/drawingml/textfieldcontext.cxx 
b/oox/source/drawingml/textfieldcontext.cxx
index 8cfb17c08b18..2bd53a745528 100644
--- a/oox/source/drawingml/textfieldcontext.cxx
+++ b/oox/source/drawingml/textfieldcontext.cxx
@@ -38,8 +38,8 @@ TextFieldContext::TextFieldContext( ContextHandler2Helper 
const & rParent,
         , mrTextField( rTextField )
         , mbIsInText( false )
 {
-    mrTextField.setUuid( rAttributes.getString( XML_id ).get() );
-    mrTextField.setType( rAttributes.getString( XML_type ).get() );
+    mrTextField.setUuid( rAttributes.getString( XML_id ).value() );
+    mrTextField.setType( rAttributes.getString( XML_type ).value() );
 }
 
 void TextFieldContext::onEndElement( )
diff --git a/oox/source/drawingml/textparagraph.cxx 
b/oox/source/drawingml/textparagraph.cxx
index 8b333bc4545c..23f051cdd5df 100644
--- a/oox/source/drawingml/textparagraph.cxx
+++ b/oox/source/drawingml/textparagraph.cxx
@@ -109,7 +109,7 @@ void TextParagraph::insertAt(
             TextCharacterProperties aTextCharacterProps( aTextCharacterStyle );
             aTextCharacterProps.assignUsed( maEndProperties );
             if ( aTextCharacterProps.moHeight.has_value() )
-                nCharHeight = nCharHeightFirst = 
aTextCharacterProps.moHeight.get();
+                nCharHeight = nCharHeightFirst = 
aTextCharacterProps.moHeight.value();
             aTextCharacterProps.pushToPropSet( aPropSet, rFilterBase );
         }
         else
diff --git a/oox/source/drawingml/textparagraphpropertiescontext.cxx 
b/oox/source/drawingml/textparagraphpropertiescontext.cxx
index a6b8f2c275cc..076361d121df 100644
--- a/oox/source/drawingml/textparagraphpropertiescontext.cxx
+++ b/oox/source/drawingml/textparagraphpropertiescontext.cxx
@@ -119,7 +119,7 @@ 
TextParagraphPropertiesContext::TextParagraphPropertiesContext( ContextHandler2H
   // ST_Coordinate
     if ( rAttribs.hasAttribute( XML_indent ) )
     {
-        sValue = rAttribs.getString( XML_indent ).get();
+        sValue = rAttribs.getString( XML_indent ).value();
         mrTextParagraphProperties.getFirstLineIndentation() = std::optional< 
sal_Int32 >( sValue.isEmpty() ? 0 : GetCoordinate( sValue ) );
     }
 
@@ -142,14 +142,14 @@ 
TextParagraphPropertiesContext::TextParagraphPropertiesContext( ContextHandler2H
     // ParaLeftMargin
     if ( rAttribs.hasAttribute( XML_marL ) )
     {
-        sValue = rAttribs.getString( XML_marL ).get();
+        sValue = rAttribs.getString( XML_marL ).value();
         mrTextParagraphProperties.getParaLeftMargin() = std::optional< 
sal_Int32 >( sValue.isEmpty() ? 0 : GetCoordinate( sValue ) );
     }
 
     // ParaRightMargin
     if ( rAttribs.hasAttribute( XML_marR ) )
     {
-        sValue = rAttribs.getString( XML_marR ).get();
+        sValue = rAttribs.getString( XML_marR ).value();
         sal_Int32 nMarR  = sValue.isEmpty() ? 0 : GetCoordinate( sValue ) ;
         rPropertyMap.setProperty( PROP_ParaRightMargin, nMarR);
     }
@@ -216,11 +216,11 @@ ContextHandlerRef 
TextParagraphPropertiesContext::onCreateContext( sal_Int32 aEl
             mrBulletList.mbBulletSizeFollowText <<= true;
             break;
         case A_TOKEN( buSzPct ):        // CT_TextBulletSizePercent
-            mrBulletList.setBulletSize( std::lround( GetPercent( 
rAttribs.getString( XML_val ).get() ) / 1000.f ) );
+            mrBulletList.setBulletSize( std::lround( GetPercent( 
rAttribs.getString( XML_val ).value() ) / 1000.f ) );
             break;
         case A_TOKEN( buSzPts ):        // CT_TextBulletSizePoint
             mrBulletList.setBulletSize(0);
-            mrBulletList.setFontSize( static_cast<sal_Int16>(GetTextSize( 
rAttribs.getString( XML_val ).get() ) ) );
+            mrBulletList.setFontSize( static_cast<sal_Int16>(GetTextSize( 
rAttribs.getString( XML_val ).value() ) ) );
             break;
 
         // EG_TextBulletTypeface
@@ -260,7 +260,7 @@ ContextHandlerRef 
TextParagraphPropertiesContext::onCreateContext( sal_Int32 aEl
         case A_TOKEN( buChar ):         // CT_TextCharBullet
             try {
 
-                mrBulletList.setBulletChar( rAttribs.getString( XML_char 
).get() );
+                mrBulletList.setBulletChar( rAttribs.getString( XML_char 
).value() );
                 mrBulletList.setSuffixNone();
             }
             catch(SAXException& /* e */)
@@ -280,9 +280,9 @@ ContextHandlerRef 
TextParagraphPropertiesContext::onCreateContext( sal_Int32 aEl
         case W_TOKEN( jc ):
             {
                 OptValue< OUString > oParaAdjust = rAttribs.getString( 
W_TOKEN(val) );
-                if( oParaAdjust.has_value() && !oParaAdjust.get().isEmpty() )
+                if( oParaAdjust.has_value() && !oParaAdjust.value().isEmpty() )
                 {
-                    const OUString& sParaAdjust = oParaAdjust.get();
+                    const OUString& sParaAdjust = oParaAdjust.value();
                     if( sParaAdjust == "left" )
                         
mrTextParagraphProperties.setParaAdjust(ParagraphAdjust_LEFT);
                     else if ( sParaAdjust == "right" )
@@ -304,7 +304,7 @@ ContextHandlerRef 
TextParagraphPropertiesContext::onCreateContext( sal_Int32 aEl
                     {
                         TextSpacing& rSpacing = 
mrTextParagraphProperties.getParaTopMargin();
                         rSpacing.nUnit = TextSpacing::Unit::Points;
-                        rSpacing.nValue = convertTwipToMm100(oBefore.get());
+                        rSpacing.nValue = convertTwipToMm100(oBefore.value());
                         rSpacing.bHasValue = true;
                     }
                     else
@@ -314,7 +314,7 @@ ContextHandlerRef 
TextParagraphPropertiesContext::onCreateContext( sal_Int32 aEl
                         {
                             TextSpacing& rSpacing = 
mrTextParagraphProperties.getParaTopMargin();
                             rSpacing.nUnit = TextSpacing::Unit::Percent;
-                            rSpacing.nValue = oBeforeLines.get() * MAX_PERCENT 
/ 100;
+                            rSpacing.nValue = oBeforeLines.value() * 
MAX_PERCENT / 100;
                             rSpacing.bHasValue = true;
                         }
                     }
@@ -328,7 +328,7 @@ ContextHandlerRef 
TextParagraphPropertiesContext::onCreateContext( sal_Int32 aEl
                     {
                         TextSpacing& rSpacing = 
mrTextParagraphProperties.getParaBottomMargin();
                         rSpacing.nUnit = TextSpacing::Unit::Points;
-                        rSpacing.nValue = convertTwipToMm100(oAfter.get());
+                        rSpacing.nValue = convertTwipToMm100(oAfter.value());
                         rSpacing.bHasValue = true;
                     }
                     else
@@ -338,7 +338,7 @@ ContextHandlerRef 
TextParagraphPropertiesContext::onCreateContext( sal_Int32 aEl
                         {
                             TextSpacing& rSpacing = 
mrTextParagraphProperties.getParaBottomMargin();
                             rSpacing.nUnit = TextSpacing::Unit::Percent;
-                            rSpacing.nValue = oAfterLines.get() * MAX_PERCENT 
/ 100;
+                            rSpacing.nValue = oAfterLines.value() * 
MAX_PERCENT / 100;
                             rSpacing.bHasValue = true;
                         }
                     }
@@ -350,15 +350,15 @@ ContextHandlerRef 
TextParagraphPropertiesContext::onCreateContext( sal_Int32 aEl
                 if (oLineSpacing.has_value())
                 {
                     TextSpacing& rLineSpacing = 
mrTextParagraphProperties.getLineSpacing();
-                    if( !oLineRule.has_value() || oLineRule.get() == "auto" )
+                    if( !oLineRule.has_value() || oLineRule.value() == "auto" )
                     {
                         rLineSpacing.nUnit = TextSpacing::Unit::Percent;
-                        rLineSpacing.nValue = oLineSpacing.get() * MAX_PERCENT 
/ 240;
+                        rLineSpacing.nValue = oLineSpacing.value() * 
MAX_PERCENT / 240;
                     }
                     else
                     {
                         rLineSpacing.nUnit = TextSpacing::Unit::Points;
-                        rLineSpacing.nValue = 
convertTwipToMm100(oLineSpacing.get());
+                        rLineSpacing.nValue = 
convertTwipToMm100(oLineSpacing.value());
                     }
                     rLineSpacing.bHasValue = true;
                 }
diff --git a/oox/source/drawingml/textrun.cxx b/oox/source/drawingml/textrun.cxx
index 2482e9da0461..24b94b6e4db2 100644
--- a/oox/source/drawingml/textrun.cxx
+++ b/oox/source/drawingml/textrun.cxx
@@ -77,7 +77,7 @@ sal_Int32 TextRun::insertAt(
 
         aTextCharacterProps.assignUsed(maTextCharacterProperties);
         if ( aTextCharacterProps.moHeight.has_value() )
-            nCharHeight = aTextCharacterProps.moHeight.get();
+            nCharHeight = aTextCharacterProps.moHeight.value();
         else
             // UNO API has the character height as float, DML has it as int, 
but in hundreds.
             aTextCharacterProps.moHeight = 
static_cast<sal_Int32>(nDefaultCharHeight * 100);
diff --git a/oox/source/drawingml/textspacingcontext.cxx 
b/oox/source/drawingml/textspacingcontext.cxx
index f96f1a20caac..83adaa806cf9 100644
--- a/oox/source/drawingml/textspacingcontext.cxx
+++ b/oox/source/drawingml/textspacingcontext.cxx
@@ -44,11 +44,11 @@ namespace oox::drawingml {
         {
         case A_TOKEN( spcPct ):
             maSpacing.nUnit = TextSpacing::Unit::Percent;
-            maSpacing.nValue = GetPercent( rAttribs.getString( XML_val ).get() 
);
+            maSpacing.nValue = GetPercent( rAttribs.getString( XML_val 
).value() );
             break;
         case A_TOKEN( spcPts ):
             maSpacing.nUnit = TextSpacing::Unit::Points;
-            maSpacing.nValue = GetTextSpacingPoint( rAttribs.getString( 
XML_val ).get() );
+            maSpacing.nValue = GetTextSpacingPoint( rAttribs.getString( 
XML_val ).value() );
             maSpacing.bExactValue = true;
             break;
         default:
diff --git a/oox/source/drawingml/texttabstoplistcontext.cxx 
b/oox/source/drawingml/texttabstoplistcontext.cxx
index 09809a9d891a..5387d3b5365e 100644
--- a/oox/source/drawingml/texttabstoplistcontext.cxx
+++ b/oox/source/drawingml/texttabstoplistcontext.cxx
@@ -50,7 +50,7 @@ namespace oox::drawingml {
             {
                 OUString sValue;
                 TabStop aTabStop;
-                sValue = rAttribs.getString( XML_pos ).get();
+                sValue = rAttribs.getString( XML_pos ).value();
                 if( !sValue.isEmpty() )
                 {
                     aTabStop.Position = GetCoordinate( sValue );
diff --git a/oox/source/drawingml/themeelementscontext.cxx 
b/oox/source/drawingml/themeelementscontext.cxx
index 0922c4b61939..57ce15a4c0c5 100644
--- a/oox/source/drawingml/themeelementscontext.cxx
+++ b/oox/source/drawingml/themeelementscontext.cxx
@@ -216,14 +216,14 @@ ContextHandlerRef ThemeElementsContext::onCreateContext( 
sal_Int32 nElement, con
         case A_TOKEN( clrScheme ):  // CT_ColorScheme
             if (rAttribs.hasAttribute(XML_name))
             {
-                
mrTheme.getClrScheme().SetName(rAttribs.getString(XML_name).get());
+                
mrTheme.getClrScheme().SetName(rAttribs.getString(XML_name).value());
             }
             return new clrSchemeContext( *this, mrTheme.getClrScheme() );
         case A_TOKEN( fontScheme ): // CT_FontScheme
             return new FontSchemeContext( *this, mrTheme.getFontScheme() );
 
         case A_TOKEN( fmtScheme ):  // CT_StyleMatrix
-            mrTheme.setStyleName( rAttribs.getString( XML_name ).get() );
+            mrTheme.setStyleName( rAttribs.getString( XML_name ).value() );
             return this;
 
         case A_TOKEN( fillStyleLst ):   // CT_FillStyleList
diff --git a/oox/source/drawingml/themefragmenthandler.cxx 
b/oox/source/drawingml/themefragmenthandler.cxx
index 5ab0ee1e6b21..5616dd75a08b 100644
--- a/oox/source/drawingml/themefragmenthandler.cxx
+++ b/oox/source/drawingml/themefragmenthandler.cxx
@@ -48,7 +48,7 @@ ContextHandlerRef ThemeFragmentHandler::onCreateContext( 
sal_Int32 nElement, con
             {
                 case A_TOKEN( theme ):
                 {
-                    mrTheme.setThemeName(rAttribs.getString(XML_name).get());
+                    mrTheme.setThemeName(rAttribs.getString(XML_name).value());
                     return this;
                 }
             }
diff --git a/oox/source/drawingml/transform2dcontext.cxx 
b/oox/source/drawingml/transform2dcontext.cxx
index 19945d23f7af..20c90731ed8d 100644
--- a/oox/source/drawingml/transform2dcontext.cxx
+++ b/oox/source/drawingml/transform2dcontext.cxx
@@ -43,7 +43,7 @@ Transform2DContext::Transform2DContext( ContextHandler2Helper 
const & rParent, c
     else
     {
         if( rAttribs.hasAttribute( XML_rot ) )
-            mrShape.getTextBody()->getTextProperties().moRotation = 
rAttribs.getInteger( XML_rot ).get();
+            mrShape.getTextBody()->getTextProperties().moRotation = 
rAttribs.getInteger( XML_rot ).value();
     }
 }
 
@@ -59,8 +59,8 @@ ContextHandlerRef Transform2DContext::onCreateContext( 
sal_Int32 aElementToken,
             {
                 case A_TOKEN( off ):
                     {
-                        const OUString sXValue = rAttribs.getString( XML_x 
).get();
-                        const OUString sYValue = rAttribs.getString( XML_y 
).get();
+                        const OUString sXValue = rAttribs.getString( XML_x 
).value();
+                        const OUString sYValue = rAttribs.getString( XML_y 
).value();
 
                         if( !sXValue.isEmpty() && nType != XML_ellipse )
                             
mrShape.getTextBody()->getTextProperties().moTextOffLeft = GetCoordinate( 
sXValue.toInt32() - mrShape.getPosition().X );
@@ -70,8 +70,8 @@ ContextHandlerRef Transform2DContext::onCreateContext( 
sal_Int32 aElementToken,
                     break;
                 case A_TOKEN( ext ):
                     {
-                        const OUString sXValue = rAttribs.getString( XML_cx 
).get();
-                        const OUString sYValue = rAttribs.getString( XML_cy 
).get();
+                        const OUString sXValue = rAttribs.getString( XML_cx 
).value();
+                        const OUString sYValue = rAttribs.getString( XML_cy 
).value();
 
                         if( !sXValue.isEmpty() && nType == XML_rect )
                         {
@@ -96,22 +96,22 @@ ContextHandlerRef Transform2DContext::onCreateContext( 
sal_Int32 aElementToken,
     switch( aElementToken )
     {
     case A_TOKEN( off ):        // horz/vert translation
-        mrShape.setPosition( awt::Point( rAttribs.getString( XML_x 
).get().toInt32(), rAttribs.getString( XML_y ).get().toInt32() ) );
+        mrShape.setPosition( awt::Point( rAttribs.getString( XML_x 
).value().toInt32(), rAttribs.getString( XML_y ).value().toInt32() ) );
         break;
     case A_TOKEN( ext ):        // horz/vert size
-        mrShape.setSize( awt::Size( rAttribs.getString( XML_cx 
).get().toInt32(), rAttribs.getString( XML_cy ).get().toInt32() ) );
+        mrShape.setSize( awt::Size( rAttribs.getString( XML_cx 
).value().toInt32(), rAttribs.getString( XML_cy ).value().toInt32() ) );
         break;
     case A_TOKEN( chOff ):  // horz/vert translation of children
-        mrShape.setChildPosition( awt::Point( rAttribs.getString( XML_x 
).get().toInt32(), rAttribs.getString( XML_y ).get().toInt32() ) );
+        mrShape.setChildPosition( awt::Point( rAttribs.getString( XML_x 
).value().toInt32(), rAttribs.getString( XML_y ).value().toInt32() ) );
         break;
     case A_TOKEN( chExt ):  // horz/vert size of children
         {
-            sal_Int32 nChExtCx = rAttribs.getString(XML_cx).get().toInt32();
+            sal_Int32 nChExtCx = rAttribs.getString(XML_cx).value().toInt32();
 
             if(nChExtCx == 0)
                 nChExtCx = mrShape.getSize().Width;
 
-            sal_Int32 nChExtCy = rAttribs.getString(XML_cy).get().toInt32();
+            sal_Int32 nChExtCy = rAttribs.getString(XML_cy).value().toInt32();
 
             if(nChExtCy == 0)
                 nChExtCy = mrShape.getSize().Height;
diff --git a/oox/source/helper/attributelist.cxx 
b/oox/source/helper/attributelist.cxx
index 5752af105c72..7e17d5b77171 100644
--- a/oox/source/helper/attributelist.cxx
+++ b/oox/source/helper/attributelist.cxx
@@ -243,7 +243,7 @@ OptValue< bool > AttributeList::getBool( sal_Int32 
nAttrToken ) const
         case XML_off:   return OptValue< bool >( false );
     }
     OptValue< sal_Int32 > onValue = getInteger( nAttrToken );
-    return onValue.has_value() ? OptValue< bool >( onValue.get() != 0 ) : 
OptValue< bool >();
+    return onValue.has_value() ? OptValue< bool >( onValue.value() != 0 ) : 
OptValue< bool >();
 }
 
 OptValue< util::DateTime > AttributeList::getDateTime( sal_Int32 nAttrToken ) 
const
diff --git a/oox/source/ppt/layoutfragmenthandler.cxx 
b/oox/source/ppt/layoutfragmenthandler.cxx
index 3dcc7df15ef8..4b6c19a53145 100644
--- a/oox/source/ppt/layoutfragmenthandler.cxx
+++ b/oox/source/ppt/layoutfragmenthandler.cxx
@@ -54,7 +54,7 @@ ContextHandlerRef LayoutFragmentHandler::onCreateContext( 
sal_Int32 aElementToke
             mpSlidePersistPtr->setLayoutValueToken( rAttribs.getToken( 
XML_type, 0 ) ); // CT_SlideLayoutType
 
             OptValue< bool > aShowMasterShapes = rAttribs.getBool( 
XML_showMasterSp );
-            if( aShowMasterShapes.has_value() && !aShowMasterShapes.get() ) {
+            if( aShowMasterShapes.has_value() && !aShowMasterShapes.value() ) {
                 mpSlidePersistPtr->hideShapesAsMasterShapes();
             }
             break;
diff --git a/oox/source/ppt/pptgraphicshapecontext.cxx 
b/oox/source/ppt/pptgraphicshapecontext.cxx
index 254f44809c85..a98d38624b5c 100644
--- a/oox/source/ppt/pptgraphicshapecontext.cxx
+++ b/oox/source/ppt/pptgraphicshapecontext.cxx
@@ -51,14 +51,14 @@ ContextHandlerRef PPTGraphicShapeContext::onCreateContext( 
sal_Int32 aElementTok
 //  case NMSP_PPT|XML_drElemPr:
 //      break;
     case PPT_TOKEN(cNvPr):
-        mpShapePtr->setId( rAttribs.getString( XML_id ).get() );
-        mpShapePtr->setName( rAttribs.getString( XML_name ).get() );
+        mpShapePtr->setId( rAttribs.getString( XML_id ).value() );
+        mpShapePtr->setName( rAttribs.getString( XML_name ).value() );
         break;
     case PPT_TOKEN(ph):
     {
         sal_Int32 nSubType( rAttribs.getToken( XML_type, XML_obj ) );
         mpShapePtr->setSubType( nSubType );
-        OUString sIdx( rAttribs.getString( XML_idx ).get() );
+        OUString sIdx( rAttribs.getString( XML_idx ).value() );
         bool bHasIdx = !sIdx.isEmpty();
         sal_Int32 nIdx = sIdx.toInt32();
         if( rAttribs.hasAttribute( XML_idx ) )
diff --git a/oox/source/ppt/pptshape.cxx b/oox/source/ppt/pptshape.cxx
index e730f4f26c6c..cecd2796b734 100644
--- a/oox/source/ppt/pptshape.cxx
+++ b/oox/source/ppt/pptshape.cxx
@@ -345,7 +345,7 @@ void PPTShape::addShape(
 
         if (mnSubType && getSubTypeIndex().has_value() && meShapeLocation == 
Layout)
         {
-            oox::drawingml::ShapePtr pPlaceholder = 
PPTShape::findPlaceholderByIndex( getSubTypeIndex().get(), 
rSlidePersist.getShapes()->getChildren(), true );
+            oox::drawingml::ShapePtr pPlaceholder = 
PPTShape::findPlaceholderByIndex( getSubTypeIndex().value(), 
rSlidePersist.getShapes()->getChildren(), true );
             if (!pPlaceholder)
                 pPlaceholder = PPTShape::findPlaceholder( mnSubType, 0, 
getSubTypeIndex(), rSlidePersist.getShapes()->getChildren(), true );
 
@@ -372,13 +372,13 @@ void PPTShape::addShape(
         // use placeholder index if possible
         if (mnSubType && getSubTypeIndex().has_value() && 
rSlidePersist.getMasterPersist())
         {
-            oox::drawingml::ShapePtr pPlaceholder = 
PPTShape::findPlaceholderByIndex(getSubTypeIndex().get(), 
rSlidePersist.getMasterPersist()->getShapes()->getChildren());
+            oox::drawingml::ShapePtr pPlaceholder = 
PPTShape::findPlaceholderByIndex(getSubTypeIndex().value(), 
rSlidePersist.getMasterPersist()->getShapes()->getChildren());
             // TODO: Check if this is required for non-notes slides as well...
             if (rSlidePersist.isNotesPage() && pPlaceholder && 
pPlaceholder->getSubType() != getSubType())
                 pPlaceholder.reset();
 
             if (pPlaceholder) {
-                SAL_INFO("oox.ppt","found placeholder with index: " << 
getSubTypeIndex().get() << " and type: " << lclDebugSubType( mnSubType ));
+                SAL_INFO("oox.ppt","found placeholder with index: " << 
getSubTypeIndex().value() << " and type: " << lclDebugSubType( mnSubType ));
                 PPTShape* pPPTPlaceholder = dynamic_cast< PPTShape* >( 
pPlaceholder.get() );
                 TextListStylePtr pNewTextListStyle = 
std::make_shared<TextListStyle>();
 
@@ -398,7 +398,7 @@ void PPTShape::addShape(
                     // aMasterTextListStyle->dump();
                 }
                 if (pPPTPlaceholder && pPPTPlaceholder->mpPlaceholder) {
-                    SAL_INFO("oox.ppt","placeholder has parent placeholder: " 
<< pPPTPlaceholder->mpPlaceholder->getId() << " type: " << lclDebugSubType( 
pPPTPlaceholder->mpPlaceholder->getSubType() ) << " index: " << 
pPPTPlaceholder->mpPlaceholder->getSubTypeIndex().get() );
+                    SAL_INFO("oox.ppt","placeholder has parent placeholder: " 
<< pPPTPlaceholder->mpPlaceholder->getId() << " type: " << lclDebugSubType( 
pPPTPlaceholder->mpPlaceholder->getSubType() ) << " index: " << 
pPPTPlaceholder->mpPlaceholder->getSubTypeIndex().value() );
                     SAL_INFO("oox.ppt","has textbody " << 
(pPPTPlaceholder->mpPlaceholder->getTextBody() != nullptr) );
                     TextListStylePtr pPlaceholderStyle = 
getSubTypeTextListStyle( rSlidePersist, 
pPPTPlaceholder->mpPlaceholder->getSubType() );
                     if (pPPTPlaceholder->mpPlaceholder->getTextBody())
@@ -789,7 +789,7 @@ oox::drawingml::ShapePtr PPTShape::findPlaceholderByIndex( 
const sal_Int32 nIdx,
     std::vector< oox::drawingml::ShapePtr >::reverse_iterator aRevIter( 
rShapes.rbegin() );
     while( aRevIter != rShapes.rend() )
     {
-        if ( (*aRevIter)->getSubTypeIndex().has_value() && 
(*aRevIter)->getSubTypeIndex().get() == nIdx &&
+        if ( (*aRevIter)->getSubTypeIndex().has_value() && 
(*aRevIter)->getSubTypeIndex().value() == nIdx &&
              ( !bMasterOnly || ShapeLocationIsMaster((*aRevIter).get()) ) )
         {
             aShapePtr = *aRevIter;
diff --git a/oox/source/ppt/pptshapecontext.cxx 
b/oox/source/ppt/pptshapecontext.cxx
index 8aa8ea9dc0e6..1bf7e1f30622 100644
--- a/oox/source/ppt/pptshapecontext.cxx
+++ b/oox/source/ppt/pptshapecontext.cxx
@@ -59,8 +59,8 @@ ContextHandlerRef PPTShapeContext::onCreateContext( sal_Int32 
aElementToken, con
         case PPT_TOKEN( cNvPr ):
         {
             mpShapePtr->setHidden( rAttribs.getBool( XML_hidden, false ) );
-            mpShapePtr->setId( rAttribs.getString( XML_id ).get() );
-            mpShapePtr->setName( rAttribs.getString( XML_name ).get() );
+            mpShapePtr->setId( rAttribs.getString( XML_id ).value() );
+            mpShapePtr->setName( rAttribs.getString( XML_name ).value() );
             break;
         }
         case PPT_TOKEN( ph ):
@@ -74,7 +74,7 @@ ContextHandlerRef PPTShapeContext::onCreateContext( sal_Int32 
aElementToken, con
 
             if( rAttribs.hasAttribute( XML_idx ) )
             {
-                sal_Int32 nSubTypeIndex = rAttribs.getString( XML_idx 
).get().toInt32();
+                sal_Int32 nSubTypeIndex = rAttribs.getString( XML_idx 
).value().toInt32();
                 mpShapePtr->setSubTypeIndex( nSubTypeIndex );
 
                 if(!oSubType.has_value() && pMasterPersist)
diff --git a/oox/source/ppt/pptshapegroupcontext.cxx 
b/oox/source/ppt/pptshapegroupcontext.cxx
index 17d3e6c5815e..6d7d33dbda16 100644
--- a/oox/source/ppt/pptshapegroupcontext.cxx
+++ b/oox/source/ppt/pptshapegroupcontext.cxx
@@ -69,15 +69,15 @@ ContextHandlerRef PPTShapeGroupContext::onCreateContext( 
sal_Int32 aElementToken
         // don't override SmartArt properties for embedded drawing's spTree
         mpGroupShapePtr->setHidden( rAttribs.getBool( XML_hidden, false ) );
         if (mpGroupShapePtr->getId().isEmpty())
-            mpGroupShapePtr->setId(rAttribs.getString(XML_id).get());
+            mpGroupShapePtr->setId(rAttribs.getString(XML_id).value());
         if (mpGroupShapePtr->getName().isEmpty())
-            mpGroupShapePtr->setName( rAttribs.getString( XML_name ).get() );
+            mpGroupShapePtr->setName( rAttribs.getString( XML_name ).value() );
         break;
     }
     case PPT_TOKEN( ph ):
         mpGroupShapePtr->setSubType( rAttribs.getToken( XML_type, 
FastToken::DONTKNOW ) );
         if( rAttribs.hasAttribute( XML_idx ) )
-            mpGroupShapePtr->setSubTypeIndex( rAttribs.getString( XML_idx 
).get().toInt32() );
+            mpGroupShapePtr->setSubTypeIndex( rAttribs.getString( XML_idx 
).value().toInt32() );
         break;
     // nvSpPr CT_ShapeNonVisual end
 
@@ -106,7 +106,7 @@ ContextHandlerRef PPTShapeGroupContext::onCreateContext( 
sal_Int32 aElementToken
                 pShape->getFillProperties().moFillType = XML_noFill;
                 pShape->getFillProperties().moUseBgFill = true;
             }
-            pShape->setModelId(rAttribs.getString( XML_modelId ).get());
+            pShape->setModelId(rAttribs.getString( XML_modelId ).value());
             return new PPTShapeContext( *this, mpSlidePersistPtr, 
mpGroupShapePtr, pShape );
         }

... etc. - the rest is truncated

Reply via email to