aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
-rw-r--r--lexers/LexCPP.cxx18
-rw-r--r--src/CaseConvert.cxx6
-rw-r--r--src/Catalogue.cxx14
-rw-r--r--src/Document.cxx48
-rw-r--r--src/EditView.cxx10
-rw-r--r--src/Editor.cxx5
-rw-r--r--src/PerLine.cxx6
-rw-r--r--src/PositionCache.cxx20
-rw-r--r--src/Selection.cxx32
-rw-r--r--src/XPM.cxx12
10 files changed, 84 insertions, 87 deletions
diff --git a/lexers/LexCPP.cxx b/lexers/LexCPP.cxx
index bd9977b81..238856b1c 100644
--- a/lexers/LexCPP.cxx
+++ b/lexers/LexCPP.cxx
@@ -91,8 +91,8 @@ bool IsSpaceOrTab(int ch) {
}
bool OnlySpaceOrTab(const std::string &s) {
- for (std::string::const_iterator it = s.begin(); it != s.end(); ++it) {
- if (!IsSpaceOrTab(*it))
+ for (const char ch : s) {
+ if (!IsSpaceOrTab(ch))
return false;
}
return true;
@@ -100,11 +100,11 @@ bool OnlySpaceOrTab(const std::string &s) {
std::vector<std::string> StringSplit(const std::string &text, int separator) {
std::vector<std::string> vs(text.empty() ? 0 : 1);
- for (std::string::const_iterator it = text.begin(); it != text.end(); ++it) {
- if (*it == separator) {
+ for (const char ch : text) {
+ if (ch == separator) {
vs.push_back(std::string());
} else {
- vs.back() += *it;
+ vs.back() += ch;
}
}
return vs;
@@ -709,11 +709,11 @@ void SCI_METHOD LexerCPP::Lex(Sci_PositionU startPos, Sci_Position length, int i
}
SymbolTable preprocessorDefinitions = preprocessorDefinitionsStart;
- for (std::vector<PPDefinition>::iterator itDef = ppDefineHistory.begin(); itDef != ppDefineHistory.end(); ++itDef) {
- if (itDef->isUndef)
- preprocessorDefinitions.erase(itDef->key);
+ for (const PPDefinition &ppDef : ppDefineHistory) {
+ if (ppDef.isUndef)
+ preprocessorDefinitions.erase(ppDef.key);
else
- preprocessorDefinitions[itDef->key] = SymbolValue(itDef->value, itDef->arguments);
+ preprocessorDefinitions[ppDef.key] = SymbolValue(ppDef.value, ppDef.arguments);
}
std::string rawStringTerminator = rawStringTerminators.ValueAt(lineCurrent-1);
diff --git a/src/CaseConvert.cxx b/src/CaseConvert.cxx
index 4d0e5e271..24205dd81 100644
--- a/src/CaseConvert.cxx
+++ b/src/CaseConvert.cxx
@@ -653,9 +653,9 @@ public:
std::sort(characterToConversion.begin(), characterToConversion.end());
characters.reserve(characterToConversion.size());
conversions.reserve(characterToConversion.size());
- for (CharacterToConversion::iterator it = characterToConversion.begin(); it != characterToConversion.end(); ++it) {
- characters.push_back(it->character);
- conversions.push_back(it->conversion);
+ for (const CharacterConversion &chConv : characterToConversion) {
+ characters.push_back(chConv.character);
+ conversions.push_back(chConv.conversion);
}
// Empty the original calculated data completely
CharacterToConversion().swap(characterToConversion);
diff --git a/src/Catalogue.cxx b/src/Catalogue.cxx
index 8dbe1dde0..c4f7a7eda 100644
--- a/src/Catalogue.cxx
+++ b/src/Catalogue.cxx
@@ -28,10 +28,9 @@ static int nextLanguage = SCLEX_AUTOMATIC+1;
const LexerModule *Catalogue::Find(int language) {
Scintilla_LinkLexers();
- for (std::vector<LexerModule *>::iterator it=lexerCatalogue.begin();
- it != lexerCatalogue.end(); ++it) {
- if ((*it)->GetLanguage() == language) {
- return *it;
+ for (const LexerModule *lm : lexerCatalogue) {
+ if (lm->GetLanguage() == language) {
+ return lm;
}
}
return 0;
@@ -40,10 +39,9 @@ const LexerModule *Catalogue::Find(int language) {
const LexerModule *Catalogue::Find(const char *languageName) {
Scintilla_LinkLexers();
if (languageName) {
- for (std::vector<LexerModule *>::iterator it=lexerCatalogue.begin();
- it != lexerCatalogue.end(); ++it) {
- if ((*it)->languageName && (0 == strcmp((*it)->languageName, languageName))) {
- return *it;
+ for (const LexerModule *lm : lexerCatalogue) {
+ if (lm->languageName && (0 == strcmp(lm->languageName, languageName))) {
+ return lm;
}
}
}
diff --git a/src/Document.cxx b/src/Document.cxx
index d8fdc1153..d7e3bf62b 100644
--- a/src/Document.cxx
+++ b/src/Document.cxx
@@ -134,12 +134,12 @@ Document::Document() {
}
Document::~Document() {
- for (std::vector<WatcherWithUserData>::iterator it = watchers.begin(); it != watchers.end(); ++it) {
- it->watcher->NotifyDeleted(this, it->userData);
+ for (const WatcherWithUserData &watcher : watchers) {
+ watcher.watcher->NotifyDeleted(this, watcher.userData);
}
- for (int j=0; j<ldSize; j++) {
- delete perLineData[j];
- perLineData[j] = 0;
+ for (PerLine *&pl : perLineData) {
+ delete pl;
+ pl = nullptr;
}
regex.release();
delete pli;
@@ -149,9 +149,9 @@ Document::~Document() {
}
void Document::Init() {
- for (int j=0; j<ldSize; j++) {
- if (perLineData[j])
- perLineData[j]->Init();
+ for (PerLine *pl : perLineData) {
+ if (pl)
+ pl->Init();
}
}
@@ -190,16 +190,16 @@ bool Document::SetLineEndTypesAllowed(int lineEndBitSet_) {
}
void Document::InsertLine(Sci::Line line) {
- for (int j=0; j<ldSize; j++) {
- if (perLineData[j])
- perLineData[j]->InsertLine(line);
+ for (PerLine *pl : perLineData) {
+ if (pl)
+ pl->InsertLine(line);
}
}
void Document::RemoveLine(Sci::Line line) {
- for (int j=0; j<ldSize; j++) {
- if (perLineData[j])
- perLineData[j]->RemoveLine(line);
+ for (PerLine *pl : perLineData) {
+ if (pl)
+ pl->RemoveLine(line);
}
}
@@ -383,8 +383,8 @@ Sci_Position SCI_METHOD Document::LineEnd(Sci_Position line) const {
void SCI_METHOD Document::SetErrorStatus(int status) {
// Tell the watchers an error has occurred.
- for (std::vector<WatcherWithUserData>::iterator it = watchers.begin(); it != watchers.end(); ++it) {
- it->watcher->NotifyErrorOccurred(this, it->userData, status);
+ for (const WatcherWithUserData &watcher : watchers) {
+ watcher.watcher->NotifyErrorOccurred(this, watcher.userData, status);
}
}
@@ -2119,8 +2119,8 @@ void Document::StyleToAdjustingLineDuration(Sci::Position pos) {
void Document::LexerChanged() {
// Tell the watchers the lexer has changed.
- for (std::vector<WatcherWithUserData>::iterator it = watchers.begin(); it != watchers.end(); ++it) {
- it->watcher->NotifyLexerChanged(this, it->userData);
+ for (const WatcherWithUserData &watcher : watchers) {
+ watcher.watcher->NotifyLexerChanged(this, watcher.userData);
}
}
@@ -2254,14 +2254,14 @@ bool Document::RemoveWatcher(DocWatcher *watcher, void *userData) {
}
void Document::NotifyModifyAttempt() {
- for (std::vector<WatcherWithUserData>::iterator it = watchers.begin(); it != watchers.end(); ++it) {
- it->watcher->NotifyModifyAttempt(this, it->userData);
+ for (const WatcherWithUserData &watcher : watchers) {
+ watcher.watcher->NotifyModifyAttempt(this, watcher.userData);
}
}
void Document::NotifySavePoint(bool atSavePoint) {
- for (std::vector<WatcherWithUserData>::iterator it = watchers.begin(); it != watchers.end(); ++it) {
- it->watcher->NotifySavePoint(this, it->userData, atSavePoint);
+ for (const WatcherWithUserData &watcher : watchers) {
+ watcher.watcher->NotifySavePoint(this, watcher.userData, atSavePoint);
}
}
@@ -2271,8 +2271,8 @@ void Document::NotifyModified(DocModification mh) {
} else if (mh.modificationType & SC_MOD_DELETETEXT) {
decorations.DeleteRange(mh.position, mh.length);
}
- for (std::vector<WatcherWithUserData>::iterator it = watchers.begin(); it != watchers.end(); ++it) {
- it->watcher->NotifyModified(this, mh, it->userData);
+ for (const WatcherWithUserData &watcher : watchers) {
+ watcher.watcher->NotifyModified(this, mh, watcher.userData);
}
}
diff --git a/src/EditView.cxx b/src/EditView.cxx
index d3ebeeda5..d40ac51d8 100644
--- a/src/EditView.cxx
+++ b/src/EditView.cxx
@@ -2036,7 +2036,7 @@ void EditView::PaintText(Surface *surfaceWindow, const EditModel &model, PRectan
} else {
phases.push_back(drawAll);
}
- for (std::vector<DrawPhase>::iterator it = phases.begin(); it != phases.end(); ++it) {
+ for (const DrawPhase &phase : phases) {
int ypos = 0;
if (!bufferedDraw)
ypos += screenLinePaintFirst * vsDraw.lineHeight;
@@ -2075,7 +2075,7 @@ void EditView::PaintText(Surface *surfaceWindow, const EditModel &model, PRectan
ll->SetBracesHighlight(rangeLine, model.braces, static_cast<char>(model.bracesMatchStyle),
static_cast<int>(model.highlightGuideColumn * vsDraw.spaceWidth), bracesIgnoreStyle);
- if (leftTextOverlap && (bufferedDraw || ((phasesDraw < phasesMultiple) && (*it & drawBack)))) {
+ if (leftTextOverlap && (bufferedDraw || ((phasesDraw < phasesMultiple) && (phase & drawBack)))) {
// Clear the left margin
PRectangle rcSpacer = rcLine;
rcSpacer.right = rcSpacer.left;
@@ -2083,17 +2083,17 @@ void EditView::PaintText(Surface *surfaceWindow, const EditModel &model, PRectan
surface->FillRectangle(rcSpacer, vsDraw.styles[STYLE_DEFAULT].back);
}
- DrawLine(surface, model, vsDraw, ll, lineDoc, visibleLine, xStart, rcLine, subLine, *it);
+ DrawLine(surface, model, vsDraw, ll, lineDoc, visibleLine, xStart, rcLine, subLine, phase);
//durPaint += et.Duration(true);
// Restore the previous styles for the brace highlights in case layout is in cache.
ll->RestoreBracesHighlight(rangeLine, model.braces, bracesIgnoreStyle);
- if (*it & drawFoldLines) {
+ if (phase & drawFoldLines) {
DrawFoldLines(surface, model, vsDraw, lineDoc, rcLine);
}
- if (*it & drawCarets) {
+ if (phase & drawCarets) {
DrawCarets(surface, model, vsDraw, ll, lineDoc, xStart, rcLine, subLine);
}
diff --git a/src/Editor.cxx b/src/Editor.cxx
index 78b1845db..2c7c2e5a2 100644
--- a/src/Editor.cxx
+++ b/src/Editor.cxx
@@ -4147,9 +4147,8 @@ void Editor::CopySelectionRange(SelectionText *ss, bool allowLineCopy) {
std::vector<SelectionRange> rangesInOrder = sel.RangesCopy();
if (sel.selType == Selection::selRectangle)
std::sort(rangesInOrder.begin(), rangesInOrder.end());
- for (size_t r=0; r<rangesInOrder.size(); r++) {
- const SelectionRange current = rangesInOrder[r];
- text.append(RangeText(current.Start().Position(), current.End().Position()));
+ for (const SelectionRange &current : rangesInOrder) {
+ text.append(RangeText(current.Start().Position(), current.End().Position()));
if (sel.selType == Selection::selRectangle) {
if (pdoc->eolMode != SC_EOL_LF)
text.push_back('\r');
diff --git a/src/PerLine.cxx b/src/PerLine.cxx
index abe2230e6..f26b48b77 100644
--- a/src/PerLine.cxx
+++ b/src/PerLine.cxx
@@ -548,9 +548,9 @@ int LineTabstops::GetNextTabstop(Sci::Line line, int x) const {
if (line < tabstops.Length()) {
TabstopList *tl = tabstops[line];
if (tl) {
- for (size_t i = 0; i < tl->size(); i++) {
- if ((*tl)[i] > x) {
- return (*tl)[i];
+ for (const int i : *tl) {
+ if (i > x) {
+ return i;
}
}
}
diff --git a/src/PositionCache.cxx b/src/PositionCache.cxx
index 1b69fccdb..6df915b8b 100644
--- a/src/PositionCache.cxx
+++ b/src/PositionCache.cxx
@@ -293,16 +293,16 @@ void LineLayoutCache::AllocateForLevel(Sci::Line linesOnScreen, Sci::Line linesI
void LineLayoutCache::Deallocate() {
PLATFORM_ASSERT(useCount == 0);
- for (size_t i = 0; i < cache.size(); i++)
- delete cache[i];
+ for (LineLayout *ll : cache)
+ delete ll;
cache.clear();
}
void LineLayoutCache::Invalidate(LineLayout::validLevel validity_) {
if (!cache.empty() && !allInvalidated) {
- for (size_t i = 0; i < cache.size(); i++) {
- if (cache[i]) {
- cache[i]->Invalidate(validity_);
+ for (LineLayout *ll : cache) {
+ if (ll) {
+ ll->Invalidate(validity_);
}
}
if (validity_ == LineLayout::llInvalid) {
@@ -482,7 +482,7 @@ BreakFinder::BreakFinder(const LineLayout *ll_, const Selection *psel, Range lin
}
}
}
- if (pvsDraw && pvsDraw->indicatorsSetFore > 0) {
+ if (pvsDraw && pvsDraw->indicatorsSetFore) {
for (Decoration *deco = pdoc->decorations.Root(); deco; deco = deco->Next()) {
if (pvsDraw->indicators[deco->Indicator()].OverridesTextFore()) {
Sci::Position startPos = deco->rs.EndRun(posLineStart);
@@ -639,8 +639,8 @@ PositionCache::~PositionCache() {
void PositionCache::Clear() {
if (!allClear) {
- for (size_t i=0; i<pces.size(); i++) {
- pces[i].Clear();
+ for (PositionCacheEntry &pce : pces) {
+ pce.Clear();
}
}
clock = 1;
@@ -700,8 +700,8 @@ void PositionCache::MeasureWidths(Surface *surface, const ViewStyle &vstyle, uns
if (clock > 60000) {
// Since there are only 16 bits for the clock, wrap it round and
// reset all cache entries so none get stuck with a high clock.
- for (size_t i=0; i<pces.size(); i++) {
- pces[i].ResetClock();
+ for (PositionCacheEntry &pce : pces) {
+ pce.ResetClock();
}
clock = 2;
}
diff --git a/src/Selection.cxx b/src/Selection.cxx
index cc7065e3a..fe81cdf49 100644
--- a/src/Selection.cxx
+++ b/src/Selection.cxx
@@ -266,8 +266,8 @@ void Selection::SetMoveExtends(bool moveExtends_) {
}
bool Selection::Empty() const {
- for (size_t i=0; i<ranges.size(); i++) {
- if (!ranges[i].Empty())
+ for (const SelectionRange &range : ranges) {
+ if (!range.Empty())
return false;
}
return true;
@@ -275,26 +275,26 @@ bool Selection::Empty() const {
SelectionPosition Selection::Last() const {
SelectionPosition lastPosition;
- for (size_t i=0; i<ranges.size(); i++) {
- if (lastPosition < ranges[i].caret)
- lastPosition = ranges[i].caret;
- if (lastPosition < ranges[i].anchor)
- lastPosition = ranges[i].anchor;
+ for (const SelectionRange &range : ranges) {
+ if (lastPosition < range.caret)
+ lastPosition = range.caret;
+ if (lastPosition < range.anchor)
+ lastPosition = range.anchor;
}
return lastPosition;
}
Sci::Position Selection::Length() const {
Sci::Position len = 0;
- for (size_t i=0; i<ranges.size(); i++) {
- len += ranges[i].Length();
+ for (const SelectionRange &range : ranges) {
+ len += range.Length();
}
return len;
}
void Selection::MovePositions(bool insertion, Sci::Position startChange, Sci::Position length) {
- for (size_t i=0; i<ranges.size(); i++) {
- ranges[i].MoveForInsertDelete(insertion, startChange, length);
+ for (SelectionRange &range : ranges) {
+ range.MoveForInsertDelete(insertion, startChange, length);
}
if (selType == selRectangle) {
rangeRectangular.MoveForInsertDelete(insertion, startChange, length);
@@ -394,11 +394,11 @@ int Selection::InSelectionForEOL(Sci::Position pos) const {
Sci::Position Selection::VirtualSpaceFor(Sci::Position pos) const {
Sci::Position virtualSpace = 0;
- for (size_t i=0; i<ranges.size(); i++) {
- if ((ranges[i].caret.Position() == pos) && (virtualSpace < ranges[i].caret.VirtualSpace()))
- virtualSpace = ranges[i].caret.VirtualSpace();
- if ((ranges[i].anchor.Position() == pos) && (virtualSpace < ranges[i].anchor.VirtualSpace()))
- virtualSpace = ranges[i].anchor.VirtualSpace();
+ for (const SelectionRange &range : ranges) {
+ if ((range.caret.Position() == pos) && (virtualSpace < range.caret.VirtualSpace()))
+ virtualSpace = range.caret.VirtualSpace();
+ if ((range.anchor.Position() == pos) && (virtualSpace < range.anchor.VirtualSpace()))
+ virtualSpace = range.anchor.VirtualSpace();
}
return virtualSpace;
}
diff --git a/src/XPM.cxx b/src/XPM.cxx
index 0e5795cc5..c3b36c7ff 100644
--- a/src/XPM.cxx
+++ b/src/XPM.cxx
@@ -282,9 +282,9 @@ RGBAImage *RGBAImageSet::Get(int ident) {
/// Give the largest height of the set.
int RGBAImageSet::GetHeight() const {
if (height < 0) {
- for (ImageMap::const_iterator it=images.begin(); it != images.end(); ++it) {
- if (height < it->second->GetHeight()) {
- height = it->second->GetHeight();
+ for (const std::pair<int, RGBAImage*> &image : images) {
+ if (height < image.second->GetHeight()) {
+ height = image.second->GetHeight();
}
}
}
@@ -294,9 +294,9 @@ int RGBAImageSet::GetHeight() const {
/// Give the largest width of the set.
int RGBAImageSet::GetWidth() const {
if (width < 0) {
- for (ImageMap::const_iterator it=images.begin(); it != images.end(); ++it) {
- if (width < it->second->GetWidth()) {
- width = it->second->GetWidth();
+ for (const std::pair<int, RGBAImage*> &image : images) {
+ if (width < image.second->GetWidth()) {
+ width = image.second->GetWidth();
}
}
}