diff options
| author | Laurent Godard <lgodard.libre@laposte.net> | 2012-01-04 15:28:35 +0100 |
|---|---|---|
| committer | Kohei Yoshida <kohei.yoshida@suse.com> | 2012-01-04 10:01:06 -0500 |
| commit | 26d174c1a31a74c39820aa621118f02cff4d43a6 (patch) | |
| tree | 27307041cd0f072d2f0d6d448a991d6d28a4129d /sc/source | |
| parent | 5ecc8a0d2524d841f5edd21495602f09819e574d (diff) | |
move static casts outside loops
some little perf. improvements on saving spreadsheets
Diffstat (limited to 'sc/source')
| -rw-r--r-- | sc/source/core/data/table5.cxx | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/sc/source/core/data/table5.cxx b/sc/source/core/data/table5.cxx index f5a2abe35f92..fffd42fa36e5 100644 --- a/sc/source/core/data/table5.cxx +++ b/sc/source/core/data/table5.cxx @@ -966,9 +966,11 @@ void lcl_syncFlags(ScFlatBoolColSegments& rColSegments, ScFlatBoolRowSegments& r { using ::sal::static_int_cast; - pRowFlags->AndValue(0, MAXROW, static_int_cast<sal_uInt8>(~nFlagMask)); + sal_uInt8 nFlagMaskComplement = static_int_cast<sal_uInt8>(~nFlagMask); + + pRowFlags->AndValue(0, MAXROW, nFlagMaskComplement); for (SCCOL i = 0; i <= MAXCOL; ++i) - pColFlags[i] &= static_int_cast<sal_uInt8>(~nFlagMask); + pColFlags[i] &= nFlagMaskComplement; { // row hidden flags. @@ -981,7 +983,7 @@ void lcl_syncFlags(ScFlatBoolColSegments& rColSegments, ScFlatBoolRowSegments& r break; if (aData.mbValue) - pRowFlags->OrValue(nRow, aData.mnRow2, static_int_cast<sal_uInt8>(nFlagMask)); + pRowFlags->OrValue(nRow, aData.mnRow2, nFlagMask); nRow = aData.mnRow2 + 1; } @@ -1014,16 +1016,18 @@ void ScTable::SyncColRowFlags() { using ::sal::static_int_cast; + sal_uInt8 nManualBreakComplement = static_int_cast<sal_uInt8>(~CR_MANUALBREAK); + // Manual breaks. - pRowFlags->AndValue(0, MAXROW, static_int_cast<sal_uInt8>(~CR_MANUALBREAK)); + pRowFlags->AndValue(0, MAXROW, nManualBreakComplement); for (SCCOL i = 0; i <= MAXCOL; ++i) - pColFlags[i] &= static_int_cast<sal_uInt8>(~CR_MANUALBREAK); + pColFlags[i] &= nManualBreakComplement; if (!maRowManualBreaks.empty()) { for (set<SCROW>::const_iterator itr = maRowManualBreaks.begin(), itrEnd = maRowManualBreaks.end(); itr != itrEnd; ++itr) - pRowFlags->OrValue(*itr, static_int_cast<sal_uInt8>(CR_MANUALBREAK)); + pRowFlags->OrValue(*itr, CR_MANUALBREAK); } if (!maColManualBreaks.empty()) |