diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-02-23 09:34:00 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-02-24 09:48:16 +0100 |
commit | 48f2d4e58004796884ececcccd46bd83f0752c3e (patch) | |
tree | 903c8936d10ffbc30b5bac548e87a7a7495dcc1c /idlc | |
parent | 11c13789f97dff642d52418815c0727abce5f152 (diff) |
loplugin:unusedfields in helpcompiler..jvmfwk
Change-Id: Ic10c521de310e0f0ac1f79a1ae169252c20075b2
Reviewed-on: https://gerrit.libreoffice.org/68226
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'idlc')
-rw-r--r-- | idlc/inc/astexpression.hxx | 4 | ||||
-rw-r--r-- | idlc/source/astexpression.cxx | 18 |
2 files changed, 0 insertions, 22 deletions
diff --git a/idlc/inc/astexpression.hxx b/idlc/inc/astexpression.hxx index 03e47004c232..527746245bff 100644 --- a/idlc/inc/astexpression.hxx +++ b/idlc/inc/astexpression.hxx @@ -117,16 +117,12 @@ public: OString toString(); private: - // Fill out the lineno, filename and definition scope details - void fillDefinitionDetails(); // Evaluate different sets of operators std::unique_ptr<AstExprValue> eval_bin_op(); std::unique_ptr<AstExprValue> eval_bit_op(); std::unique_ptr<AstExprValue> eval_un_op(); std::unique_ptr<AstExprValue> eval_symbol(); - OString m_fileName; // fileName defined in - ExprComb m_combOperator; std::unique_ptr<AstExpression> m_subExpr1; diff --git a/idlc/source/astexpression.cxx b/idlc/source/astexpression.cxx index 486c74977864..48963d43a0ac 100644 --- a/idlc/source/astexpression.cxx +++ b/idlc/source/astexpression.cxx @@ -34,15 +34,11 @@ AstExpression::AstExpression(ExprComb c, AstExpression *pExpr1, AstExpression *p , m_subExpr1(pExpr1) , m_subExpr2(pExpr2) { - fillDefinitionDetails(); - } AstExpression::AstExpression(sal_Int32 l) : m_combOperator(ExprComb::NONE) { - fillDefinitionDetails(); - m_exprValue.reset( new AstExprValue ); m_exprValue->et = ET_long; m_exprValue->u.lval = l; @@ -51,8 +47,6 @@ AstExpression::AstExpression(sal_Int32 l) AstExpression::AstExpression(sal_Int32 l, ExprType et) : m_combOperator(ExprComb::NONE) { - fillDefinitionDetails(); - m_exprValue.reset( new AstExprValue ); m_exprValue->et = et; m_exprValue->u.lval = l; @@ -61,8 +55,6 @@ AstExpression::AstExpression(sal_Int32 l, ExprType et) AstExpression::AstExpression(sal_Int64 h) : m_combOperator(ExprComb::NONE) { - fillDefinitionDetails(); - m_exprValue.reset( new AstExprValue ); m_exprValue->et = ET_hyper; m_exprValue->u.hval = h; @@ -71,8 +63,6 @@ AstExpression::AstExpression(sal_Int64 h) AstExpression::AstExpression(sal_uInt64 uh) : m_combOperator(ExprComb::NONE) { - fillDefinitionDetails(); - m_exprValue.reset( new AstExprValue ); m_exprValue->et = ET_uhyper; m_exprValue->u.uhval = uh; @@ -81,8 +71,6 @@ AstExpression::AstExpression(sal_uInt64 uh) AstExpression::AstExpression(double d) : m_combOperator(ExprComb::NONE) { - fillDefinitionDetails(); - m_exprValue.reset( new AstExprValue ); m_exprValue->et = ET_double; m_exprValue->u.dval = d; @@ -93,7 +81,6 @@ AstExpression::AstExpression(OString* scopedName) { if (scopedName) m_xSymbolicName = *scopedName; - fillDefinitionDetails(); } AstExpression::~AstExpression() @@ -704,11 +691,6 @@ bool AstExpression::compareLong(AstExpression *pExpr) return bRet; } -void AstExpression::fillDefinitionDetails() -{ - m_fileName = idlc()->getFileName(); -} - void AstExpression::evaluate() { /* |