Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-25 05:41:12 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1708422: part 21) Reduce scope of erv variable in mozInlineSpellChecker::SpellCheckerSlice::Execute. r=smaug

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D115687
This commit is contained in:
Mirko Brodesser 2021-05-26 10:04:36 +00:00
parent 0694578bb0
commit 59984954da
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
extensions/spellcheck/src/mozInlineSpellChecker.cpp
View File

@ -1397,7 +1397,7 @@ nsresult mozInlineSpellChecker::SpellCheckerSlice::Execute() {
// see if the selection has any ranges, if not, then we can optimize checking
// range inclusion later (we have no ranges when we are initially checking or
// when there are no misspelled words yet).
int32_t originalRangeCount = mSpellCheckSelection.RangeCount();
const int32_t originalRangeCount = mSpellCheckSelection.RangeCount();
// set the starting DOM position to be the beginning of our range
if (nsresult rv = mWordUtil.SetPositionAndEnd(
@ -1457,11 +1457,11 @@ nsresult mozInlineSpellChecker::SpellCheckerSlice::Execute() {
NS_ConvertUTF16toUTF8(word.mText).get(),
word.mSkipChecking ? " (not checking)" : ""));
ErrorResult erv;
// see if there is a spellcheck range that already intersects the word
// and remove it. We only need to remove old ranges, so don't bother if
// there were no ranges when we started out.
if (originalRangeCount > 0) {
ErrorResult erv;
// likewise, if this word is inside new text, we won't bother testing
if (!mStatus->GetCreatedRange() ||
!mStatus->GetCreatedRange()->IsPointInRange(*beginNode, beginOffset,