diff options
author | Andrzej Hunt <andrzej@ahunt.org> | 2015-06-01 22:12:54 +0100 |
---|---|---|
committer | Andrzej Hunt <andrzej@ahunt.org> | 2015-10-20 18:18:40 +0200 |
commit | 6f761896405732b86f2a77b839a563bbbecbd3e2 (patch) | |
tree | dec1afb2cbb79d6ac6825a73a605ac0364627269 | |
parent | fe731c3be3cfd230fc2086f66a04110804de6e3a (diff) |
Rename VERIFIED->VALID for formula status.
Change-Id: I93a07e11546ebecf912449c72e57404731c346e9
-rw-r--r-- | sc/inc/units.hxx | 2 | ||||
-rw-r--r-- | sc/qa/unit/units.cxx | 24 | ||||
-rw-r--r-- | sc/source/core/units/unitsimpl.cxx | 10 | ||||
-rw-r--r-- | sc/source/ui/view/viewfunc.cxx | 2 |
4 files changed, 19 insertions, 19 deletions
diff --git a/sc/inc/units.hxx b/sc/inc/units.hxx index 411fb843c95c..1e0c5ea8ae95 100644 --- a/sc/inc/units.hxx +++ b/sc/inc/units.hxx @@ -45,7 +45,7 @@ struct RangeUnits { * is used). */ enum class FormulaStatus { - VERIFIED, + VALID, UNKNOWN, ERROR_INPUT_SCALING, ERROR_INPUT_INCOMPATIBLE, diff --git a/sc/qa/unit/units.cxx b/sc/qa/unit/units.cxx index e2ad7cc50b7d..1dbf74f755c0 100644 --- a/sc/qa/unit/units.cxx +++ b/sc/qa/unit/units.cxx @@ -221,7 +221,7 @@ void UnitsTest::testUnitVerification() { mpDoc->SetFormula(address, "=A1+A2"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); // Test that addition of different units fails - incompatible types address = ScAddress(0, 6, 0); @@ -235,14 +235,14 @@ void UnitsTest::testUnitVerification() { mpDoc->SetFormula(address, "=A1*B1+A2*B2"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); // Test another combination (i.e. cm/s+'cm/s') address = ScAddress(0, 8, 0); mpDoc->SetFormula(address, "=A1/C1+D1"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); // Test that another combination fails (cm*kg/s+'cm/s') address = ScAddress(0, 9, 0); @@ -256,7 +256,7 @@ void UnitsTest::testUnitVerification() { mpDoc->SetFormula(address, "=A1+100*E1"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); // 10cm + 100*1m = 110cm CPPUNIT_ASSERT_EQUAL(mpDoc->GetValue(address), 110.0); @@ -287,7 +287,7 @@ void UnitsTest::testUnitVerification() { mpDoc->SetFormula(address, "=SUM(B1:B3)"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); // SUM("cm"&"kg") address.IncRow(); @@ -315,14 +315,14 @@ void UnitsTest::testUnitVerification() { mpDoc->SetFormula(address, "=SUM(A1:A3)/SUM(C1:C3)+SUM(D1:D3)"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); // PRODUCT("cm/","s")+"cm" address.IncRow(); mpDoc->SetFormula(address, "=PRODUCT(C1:D1)+A1"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); // PRODUCT("cm/","s")+"kg" address.IncRow(); @@ -337,7 +337,7 @@ void UnitsTest::testUnitVerification() { mpDoc->SetFormula(address, "=SUM(A1,A2:A3)"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); // But mixing the columns fails because of mixed units // (This test is primarily to ensure that we can handle arbitrary numbers @@ -358,7 +358,7 @@ void UnitsTest::testUnitVerification() { mpDoc->SetFormula(address, "=" + aFunc + "(A1:A2)+A3"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); // FOO(cm) + kg address.IncRow(); @@ -376,7 +376,7 @@ void UnitsTest::testUnitVerification() { mpDoc->SetFormula(address, "=A1"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); // header in [cm], cell with 100*cm @@ -405,14 +405,14 @@ void UnitsTest::testUnitVerification() { mpDoc->SetFormula(address, "=100*E1"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); // header in (cm/s), formula resulting in cm/s address = ScAddress(6, 1, 0); mpDoc->SetFormula(address, "=A1/C1"); pCell = mpDoc->GetFormulaCell(address); pTokens = pCell->GetCode(); - CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VERIFIED); + CPPUNIT_ASSERT(mpUnitsImpl->verifyFormula(pTokens, address, mpDoc) == FormulaStatus::VALID); } void UnitsTest::testUnitFromFormatStringExtraction() { diff --git a/sc/source/core/units/unitsimpl.cxx b/sc/source/core/units/unitsimpl.cxx index d160e1727388..63327478570b 100644 --- a/sc/source/core/units/unitsimpl.cxx +++ b/sc/source/core/units/unitsimpl.cxx @@ -246,7 +246,7 @@ UnitsResult UnitsImpl::getOutputUnitsForOpCode(stack< RAUSItem >& rStack, const } - return { FormulaStatus::VERIFIED, aFirstUnit }; + return { FormulaStatus::VALID, aFirstUnit }; } case ocProduct: { @@ -269,7 +269,7 @@ UnitsResult UnitsImpl::getOutputUnitsForOpCode(stack< RAUSItem >& rStack, const } while (aIt.next()); } - return { FormulaStatus::VERIFIED, aUnit }; + return { FormulaStatus::VALID, aUnit }; } default: return { FormulaStatus::UNKNOWN, boost::none }; @@ -278,7 +278,7 @@ UnitsResult UnitsImpl::getOutputUnitsForOpCode(stack< RAUSItem >& rStack, const SAL_INFO("sc.units", "unit verification not supported for opcode: " << nOpCode); return { FormulaStatus::UNKNOWN, boost::none }; } - return { FormulaStatus::VERIFIED, pOut }; + return { FormulaStatus::VALID, pOut }; } OUString UnitsImpl::extractUnitStringFromFormat(const OUString& rFormatString) { @@ -566,7 +566,7 @@ FormulaStatus UnitsImpl::verifyFormula(ScTokenArray* pArray, const ScAddress& rF case FormulaStatus::ERROR_INPUT_INCOMPATIBLE: case FormulaStatus::UNKNOWN: return aResult.status; - case FormulaStatus::VERIFIED: + case FormulaStatus::VALID: assert(aResult.units); // ensure that we have the optional unit assert(aResult.units->isValid()); aStack.push( { RAUSItemType::UNITS, aResult.units.get() } ); @@ -619,7 +619,7 @@ FormulaStatus UnitsImpl::verifyFormula(ScTokenArray* pArray, const ScAddress& rF } } - return FormulaStatus::VERIFIED; + return FormulaStatus::VALID; } bool IsDigit(sal_Unicode c) { diff --git a/sc/source/ui/view/viewfunc.cxx b/sc/source/ui/view/viewfunc.cxx index dbbff8eadfd9..402c5740082b 100644 --- a/sc/source/ui/view/viewfunc.cxx +++ b/sc/source/ui/view/viewfunc.cxx @@ -473,7 +473,7 @@ void ScViewFunc::EnterData( SCCOL nCol, SCROW nRow, SCTAB nTab, #ifdef ENABLE_CALC_UNITVERIFICATION boost::shared_ptr< Units > pUnits = Units::GetUnits(); FormulaStatus aStatus = pUnits->verifyFormula( pArr, aPos, pDoc ); - if ( aStatus == FormulaStatus::VERIFIED || aStatus == FormulaStatus::UNKNOWN ) + if ( aStatus == FormulaStatus::VALID || aStatus == FormulaStatus::UNKNOWN ) { SAL_INFO( "sc.units", "verification successful" ); |