[vlc-devel] [PATCH] Cppcheck: (performance) Prefer prefix operators for non-primitive types
Daniel Winzen
d at winzen4.de
Fri May 1 09:41:12 CEST 2015
---
modules/codec/omxil/iomx.cpp | 20 ++++++++++----------
.../demux/adaptative/http/HTTPConnectionManager.cpp | 4 ++--
.../adaptative/logic/Representationselectors.cpp | 8 ++++----
.../demux/adaptative/playlist/BaseAdaptationSet.cpp | 2 +-
modules/demux/adaptative/playlist/BasePeriod.cpp | 6 +++---
modules/demux/adaptative/playlist/Segment.cpp | 6 +++---
.../demux/adaptative/playlist/SegmentInformation.cpp | 8 ++++----
modules/demux/adaptative/playlist/SegmentList.cpp | 2 +-
.../demux/adaptative/playlist/SegmentTimeline.cpp | 8 ++++----
modules/demux/adaptative/playlist/Url.cpp | 2 +-
modules/demux/dash/mpd/IsoffMainParser.cpp | 8 ++++----
modules/demux/dash/mpd/MPD.cpp | 4 ++--
modules/demux/dash/xml/Node.cpp | 2 +-
modules/gui/qt4/components/interface_widgets.cpp | 2 +-
modules/gui/qt4/components/preferences_widgets.cpp | 2 +-
modules/gui/skins2/utils/var_list.cpp | 4 ++--
16 files changed, 44 insertions(+), 44 deletions(-)
diff --git a/modules/codec/omxil/iomx.cpp b/modules/codec/omxil/iomx.cpp
index 2de5df4..894b09a 100644
--- a/modules/codec/omxil/iomx.cpp
+++ b/modules/codec/omxil/iomx.cpp
@@ -98,7 +98,7 @@ void OMXCodecObserver::onMessage(const omx_message &msg)
node->callbacks.EventHandler(node->handle, node->app_data, msg.u.event_data.event, msg.u.event_data.data1, msg.u.event_data.data2, NULL);
break;
case omx_message::EMPTY_BUFFER_DONE:
- for( List<OMX_BUFFERHEADERTYPE*>::iterator it = node->buffers.begin(); it != node->buffers.end(); it++ ) {
+ for( List<OMX_BUFFERHEADERTYPE*>::iterator it = node->buffers.begin(); it != node->buffers.end(); ++it ) {
OMXBuffer* info = (OMXBuffer*) (*it)->pPlatformPrivate;
if (msg.u.buffer_data.buffer == info->id) {
node->callbacks.EmptyBufferDone(node->handle, node->app_data, *it);
@@ -107,7 +107,7 @@ void OMXCodecObserver::onMessage(const omx_message &msg)
}
break;
case omx_message::FILL_BUFFER_DONE:
- for( List<OMX_BUFFERHEADERTYPE*>::iterator it = node->buffers.begin(); it != node->buffers.end(); it++ ) {
+ for( List<OMX_BUFFERHEADERTYPE*>::iterator it = node->buffers.begin(); it != node->buffers.end(); ++it ) {
OMXBuffer* info = (OMXBuffer*) (*it)->pPlatformPrivate;
if (msg.u.extended_buffer_data.buffer == info->id) {
OMX_BUFFERHEADERTYPE *buffer = *it;
@@ -217,7 +217,7 @@ static OMX_ERRORTYPE iomx_free_buffer(OMX_HANDLETYPE component, OMX_U32 port, OM
OMXNode* node = (OMXNode*) ((OMX_COMPONENTTYPE*)component)->pComponentPrivate;
OMXBuffer* info = (OMXBuffer*) buffer->pPlatformPrivate;
status_t ret = ctx->iomx->freeBuffer(node->node, port, info->id);
- for( List<OMX_BUFFERHEADERTYPE*>::iterator it = node->buffers.begin(); it != node->buffers.end(); it++ ) {
+ for( List<OMX_BUFFERHEADERTYPE*>::iterator it = node->buffers.begin(); it != node->buffers.end(); ++it ) {
if (buffer == *it) {
node->buffers.erase(it);
break;
@@ -245,12 +245,12 @@ static OMX_ERRORTYPE iomx_fill_this_buffer(OMX_HANDLETYPE component, OMX_BUFFERH
static OMX_ERRORTYPE iomx_component_role_enum(OMX_HANDLETYPE component, OMX_U8 *role, OMX_U32 index)
{
OMXNode* node = (OMXNode*) ((OMX_COMPONENTTYPE*)component)->pComponentPrivate;
- for( List<IOMX::ComponentInfo>::iterator it = ctx->components.begin(); it != ctx->components.end(); it++ ) {
+ for( List<IOMX::ComponentInfo>::iterator it = ctx->components.begin(); it != ctx->components.end(); ++it ) {
if (node->component_name == it->mName) {
if (index >= it->mRoles.size())
return OMX_ErrorNoMore;
List<String8>::iterator it2 = it->mRoles.begin();
- for( OMX_U32 i = 0; it2 != it->mRoles.end() && i < index; i++, it2++ ) ;
+ for( OMX_U32 i = 0; it2 != it->mRoles.end() && i < index; i++, ++it2 ) ;
strncpy((char*)role, it2->string(), OMX_MAX_STRINGNAME_SIZE);
if (it2->length() >= OMX_MAX_STRINGNAME_SIZE)
role[OMX_MAX_STRINGNAME_SIZE - 1] = '\0';
@@ -360,7 +360,7 @@ OMX_ERRORTYPE PREFIX(OMX_ComponentNameEnum)(OMX_STRING component_name, OMX_U32 n
return OMX_ErrorNoMore;
List<IOMX::ComponentInfo>::iterator it = ctx->components.begin();
for( OMX_U32 i = 0; i < index; i++ )
- it++;
+ ++it;
strncpy(component_name, it->mName.string(), name_length);
component_name[name_length - 1] = '\0';
return OMX_ErrorNone;
@@ -368,7 +368,7 @@ OMX_ERRORTYPE PREFIX(OMX_ComponentNameEnum)(OMX_STRING component_name, OMX_U32 n
OMX_ERRORTYPE PREFIX(OMX_GetRolesOfComponent)(OMX_STRING component_name, OMX_U32 *num_roles, OMX_U8 **roles)
{
- for( List<IOMX::ComponentInfo>::iterator it = ctx->components.begin(); it != ctx->components.end(); it++ ) {
+ for( List<IOMX::ComponentInfo>::iterator it = ctx->components.begin(); it != ctx->components.end(); ++it ) {
if (!strcmp(component_name, it->mName.string())) {
if (!roles) {
*num_roles = it->mRoles.size();
@@ -378,7 +378,7 @@ OMX_ERRORTYPE PREFIX(OMX_GetRolesOfComponent)(OMX_STRING component_name, OMX_U32
return OMX_ErrorInsufficientResources;
*num_roles = it->mRoles.size();
OMX_U32 i = 0;
- for( List<String8>::iterator it2 = it->mRoles.begin(); it2 != it->mRoles.end(); i++, it2++ ) {
+ for( List<String8>::iterator it2 = it->mRoles.begin(); it2 != it->mRoles.end(); i++, ++it2 ) {
strncpy((char*)roles[i], it2->string(), OMX_MAX_STRINGNAME_SIZE);
roles[i][OMX_MAX_STRINGNAME_SIZE - 1] = '\0';
}
@@ -391,8 +391,8 @@ OMX_ERRORTYPE PREFIX(OMX_GetRolesOfComponent)(OMX_STRING component_name, OMX_U32
OMX_ERRORTYPE PREFIX(OMX_GetComponentsOfRole)(OMX_STRING role, OMX_U32 *num_comps, OMX_U8 **comp_names)
{
OMX_U32 i = 0;
- for( List<IOMX::ComponentInfo>::iterator it = ctx->components.begin(); it != ctx->components.end(); it++ ) {
- for( List<String8>::iterator it2 = it->mRoles.begin(); it2 != it->mRoles.end(); it2++ ) {
+ for( List<IOMX::ComponentInfo>::iterator it = ctx->components.begin(); it != ctx->components.end(); ++it ) {
+ for( List<String8>::iterator it2 = it->mRoles.begin(); it2 != it->mRoles.end(); ++it2 ) {
if (!strcmp(it2->string(), role)) {
if (comp_names) {
if (*num_comps < i)
diff --git a/modules/demux/adaptative/http/HTTPConnectionManager.cpp b/modules/demux/adaptative/http/HTTPConnectionManager.cpp
index f704dde..f77e1e2 100644
--- a/modules/demux/adaptative/http/HTTPConnectionManager.cpp
+++ b/modules/demux/adaptative/http/HTTPConnectionManager.cpp
@@ -53,14 +53,14 @@ void HTTPConnectionManager::closeAllConnections ()
void HTTPConnectionManager::releaseAllConnections()
{
std::vector<PersistentConnection *>::iterator it;
- for(it = connectionPool.begin(); it != connectionPool.end(); it++)
+ for(it = connectionPool.begin(); it != connectionPool.end(); ++it)
(*it)->releaseChunk();
}
PersistentConnection * HTTPConnectionManager::getConnectionForHost(const std::string &hostname)
{
std::vector<PersistentConnection *>::const_iterator it;
- for(it = connectionPool.begin(); it != connectionPool.end(); it++)
+ for(it = connectionPool.begin(); it != connectionPool.end(); ++it)
{
if(!(*it)->getHostname().compare(hostname) && (*it)->isAvailable())
return *it;
diff --git a/modules/demux/adaptative/logic/Representationselectors.cpp b/modules/demux/adaptative/logic/Representationselectors.cpp
index 058fde0..67e417f 100644
--- a/modules/demux/adaptative/logic/Representationselectors.cpp
+++ b/modules/demux/adaptative/logic/Representationselectors.cpp
@@ -42,7 +42,7 @@ BaseRepresentation * RepresentationSelector::select(BasePeriod *period, Streams:
BaseRepresentation *best = NULL;
std::vector<BaseAdaptationSet *>::const_iterator adaptIt;
- for(adaptIt=adaptSets.begin(); adaptIt!=adaptSets.end(); adaptIt++)
+ for(adaptIt=adaptSets.begin(); adaptIt!=adaptSets.end(); ++adaptIt)
{
std::vector<BaseRepresentation *> reps = (*adaptIt)->getRepresentations();
BaseRepresentation *candidate = select(reps, (best)?best->getBandwidth():0, bitrate);
@@ -67,11 +67,11 @@ BaseRepresentation * RepresentationSelector::select(BasePeriod *period, Streams:
/* subset matching WxH */
std::vector<BaseAdaptationSet *> adaptSets = period->getAdaptationSets(type);
std::vector<BaseAdaptationSet *>::const_iterator adaptIt;
- for(adaptIt=adaptSets.begin(); adaptIt!=adaptSets.end(); adaptIt++)
+ for(adaptIt=adaptSets.begin(); adaptIt!=adaptSets.end(); ++adaptIt)
{
std::vector<BaseRepresentation *> reps = (*adaptIt)->getRepresentations();
std::vector<BaseRepresentation *>::const_iterator repIt;
- for(repIt=reps.begin(); repIt!=reps.end(); repIt++)
+ for(repIt=reps.begin(); repIt!=reps.end(); ++repIt)
{
if((*repIt)->getWidth() == width && (*repIt)->getHeight() == height)
resMatchReps.push_back(*repIt);
@@ -89,7 +89,7 @@ BaseRepresentation * RepresentationSelector::select(std::vector<BaseRepresentati
{
BaseRepresentation *candidate = NULL, *lowest = NULL;
std::vector<BaseRepresentation *>::const_iterator repIt;
- for(repIt=reps.begin(); repIt!=reps.end(); repIt++)
+ for(repIt=reps.begin(); repIt!=reps.end(); ++repIt)
{
if ( !lowest || (*repIt)->getBandwidth() < lowest->getBandwidth())
lowest = *repIt;
diff --git a/modules/demux/adaptative/playlist/BaseAdaptationSet.cpp b/modules/demux/adaptative/playlist/BaseAdaptationSet.cpp
index 4174424..9715677 100644
--- a/modules/demux/adaptative/playlist/BaseAdaptationSet.cpp
+++ b/modules/demux/adaptative/playlist/BaseAdaptationSet.cpp
@@ -86,7 +86,7 @@ std::vector<std::string> BaseAdaptationSet::toString(int indent) const
text.append("BaseAdaptationSet");
ret.push_back(text);
std::vector<BaseRepresentation *>::const_iterator k;
- for(k = representations.begin(); k != representations.end(); k++)
+ for(k = representations.begin(); k != representations.end(); ++k)
{
std::vector<std::string> debug = (*k)->toString(indent + 1);
ret.insert(ret.end(), debug.begin(), debug.end());
diff --git a/modules/demux/adaptative/playlist/BasePeriod.cpp b/modules/demux/adaptative/playlist/BasePeriod.cpp
index fbc0f1a..2ceca2b 100644
--- a/modules/demux/adaptative/playlist/BasePeriod.cpp
+++ b/modules/demux/adaptative/playlist/BasePeriod.cpp
@@ -59,7 +59,7 @@ const std::vector<BaseAdaptationSet*> BasePeriod::getAdaptationSets(Streams::T
{
std::vector<BaseAdaptationSet*> list;
std::vector<BaseAdaptationSet*>::const_iterator it;
- for(it = adaptationSets.begin(); it!= adaptationSets.end(); it++)
+ for(it = adaptationSets.begin(); it!= adaptationSets.end(); ++it)
{
if( Streams::Stream::mimeToType((*it)->getMimeType()) == type )
list.push_back(*it);
@@ -79,7 +79,7 @@ void BasePeriod::addAdaptationSet(BaseAdaptationSet *adaptationSet)
BaseAdaptationSet * BasePeriod::getAdaptationSet(Streams::Type type) const
{
std::vector<BaseAdaptationSet *>::const_iterator it;
- for(it = adaptationSets.begin(); it != adaptationSets.end(); it++)
+ for(it = adaptationSets.begin(); it != adaptationSets.end(); ++it)
{
if ( Streams::Stream::mimeToType((*it)->getMimeType()) == type )
return *it;
@@ -94,7 +94,7 @@ std::vector<std::string> BasePeriod::toString(int indent) const
text.append("Period");
ret.push_back(text);
std::vector<BaseAdaptationSet *>::const_iterator k;
- for(k = adaptationSets.begin(); k != adaptationSets.end(); k++)
+ for(k = adaptationSets.begin(); k != adaptationSets.end(); ++k)
{
std::vector<std::string> debug = (*k)->toString(indent + 1);
ret.insert(ret.end(), debug.begin(), debug.end());
diff --git a/modules/demux/adaptative/playlist/Segment.cpp b/modules/demux/adaptative/playlist/Segment.cpp
index 5954033..e6acd87 100644
--- a/modules/demux/adaptative/playlist/Segment.cpp
+++ b/modules/demux/adaptative/playlist/Segment.cpp
@@ -141,7 +141,7 @@ void Segment::addSubSegment(SubSegment *subsegment)
Segment::~Segment()
{
std::vector<SubSegment*>::iterator it;
- for(it=subsegments.begin();it!=subsegments.end();it++)
+ for(it=subsegments.begin();it!=subsegments.end();++it)
delete *it;
}
@@ -161,7 +161,7 @@ std::string Segment::toString(int indent) const
{
std::string ret;
std::vector<SubSegment *>::const_iterator l;
- for(l = subsegments.begin(); l != subsegments.end(); l++)
+ for(l = subsegments.begin(); l != subsegments.end(); ++l)
{
ret.append( (*l)->toString(indent + 1) );
}
@@ -191,7 +191,7 @@ std::vector<ISegment*> Segment::subSegments()
if(!subsegments.empty())
{
std::vector<SubSegment*>::iterator it;
- for(it=subsegments.begin();it!=subsegments.end();it++)
+ for(it=subsegments.begin();it!=subsegments.end();++it)
list.push_back(*it);
}
else
diff --git a/modules/demux/adaptative/playlist/SegmentInformation.cpp b/modules/demux/adaptative/playlist/SegmentInformation.cpp
index 4f4ec3a..ed1bfb5 100644
--- a/modules/demux/adaptative/playlist/SegmentInformation.cpp
+++ b/modules/demux/adaptative/playlist/SegmentInformation.cpp
@@ -94,7 +94,7 @@ vector<ISegment *> SegmentInformation::getSegments(SegmentInfoType type) const
{
std::vector<Segment *>::const_iterator it;
for(it=segmentList->getSegments().begin();
- it!=segmentList->getSegments().end(); it++)
+ it!=segmentList->getSegments().end(); ++it)
{
std::vector<ISegment *> list = (*it)->subSegments();
retSegments.insert( retSegments.end(), list.begin(), list.end() );
@@ -227,7 +227,7 @@ void SegmentInformation::collectTimelines(std::vector<SegmentTimeline *> *timeli
timelines->push_back(mediaSegmentTemplate->segmentTimeline.Get());
std::vector<SegmentInformation *>::const_iterator it;
- for(it = childs.begin(); it != childs.end(); it++)
+ for(it = childs.begin(); it != childs.end(); ++it)
(*it)->collectTimelines(timelines);
}
@@ -266,7 +266,7 @@ static void insertIntoSegment(std::vector<Segment *> &seglist, size_t start,
size_t end, mtime_t time)
{
std::vector<Segment *>::iterator segIt;
- for(segIt = seglist.begin(); segIt < seglist.end(); segIt++)
+ for(segIt = seglist.begin(); segIt < seglist.end(); ++segIt)
{
Segment *segment = *segIt;
if(segment->getClassId() == Segment::CLASSID_SEGMENT &&
@@ -289,7 +289,7 @@ void SegmentInformation::SplitUsingIndex(std::vector<SplitPoint> &splitlist)
size_t start = 0, end = 0;
mtime_t time = 0;
- for(splitIt = splitlist.begin(); splitIt < splitlist.end(); splitIt++)
+ for(splitIt = splitlist.begin(); splitIt < splitlist.end(); ++splitIt)
{
start = end;
SplitPoint split = *splitIt;
diff --git a/modules/demux/adaptative/playlist/SegmentList.cpp b/modules/demux/adaptative/playlist/SegmentList.cpp
index 399441d..ab366da 100644
--- a/modules/demux/adaptative/playlist/SegmentList.cpp
+++ b/modules/demux/adaptative/playlist/SegmentList.cpp
@@ -35,7 +35,7 @@ SegmentList::SegmentList( SegmentInformation *parent ):
SegmentList::~SegmentList()
{
std::vector<Segment *>::iterator it;
- for(it = segments.begin(); it != segments.end(); it++)
+ for(it = segments.begin(); it != segments.end(); ++it)
delete(*it);
}
diff --git a/modules/demux/adaptative/playlist/SegmentTimeline.cpp b/modules/demux/adaptative/playlist/SegmentTimeline.cpp
index 5923fbf..9101a62 100644
--- a/modules/demux/adaptative/playlist/SegmentTimeline.cpp
+++ b/modules/demux/adaptative/playlist/SegmentTimeline.cpp
@@ -37,7 +37,7 @@ SegmentTimeline::SegmentTimeline(TimescaleAble *parent)
SegmentTimeline::~SegmentTimeline()
{
std::list<Element *>::iterator it;
- for(it = elements.begin(); it != elements.end(); it++)
+ for(it = elements.begin(); it != elements.end(); ++it)
delete *it;
}
@@ -59,7 +59,7 @@ uint64_t SegmentTimeline::getElementNumberByScaledPlaybackTime(time_t scaled) co
{
uint64_t count = 0;
std::list<Element *>::const_iterator it;
- for(it = elements.begin(); it != elements.end(); it++)
+ for(it = elements.begin(); it != elements.end(); ++it)
{
const Element *el = *it;
for(uint64_t repeat = 1 + el->r; repeat; repeat--)
@@ -85,7 +85,7 @@ mtime_t SegmentTimeline::getScaledPlaybackTimeByElementNumber(uint64_t number) c
number -= pruned;
std::list<Element *>::const_iterator it;
- for(it = elements.begin(); it != elements.end(); it++)
+ for(it = elements.begin(); it != elements.end(); ++it)
{
const Element *el = *it;
@@ -113,7 +113,7 @@ size_t SegmentTimeline::maxElementNumber() const
size_t count = 0;
std::list<Element *>::const_iterator it;
- for(it = elements.begin(); it != elements.end(); it++)
+ for(it = elements.begin(); it != elements.end(); ++it)
count += (*it)->r + 1;
return pruned + count - 1;
diff --git a/modules/demux/adaptative/playlist/Url.cpp b/modules/demux/adaptative/playlist/Url.cpp
index 354a811..9d4fb4d 100644
--- a/modules/demux/adaptative/playlist/Url.cpp
+++ b/modules/demux/adaptative/playlist/Url.cpp
@@ -83,7 +83,7 @@ std::string Url::toString(size_t index, const BaseRepresentation *rep) const
{
std::string ret;
std::vector<Component>::const_iterator it;
- for(it = components.begin(); it != components.end(); it++)
+ for(it = components.begin(); it != components.end(); +it)
{
const Component *comp = & (*it);
if(rep)
diff --git a/modules/demux/dash/mpd/IsoffMainParser.cpp b/modules/demux/dash/mpd/IsoffMainParser.cpp
index ba04015..d024bad 100644
--- a/modules/demux/dash/mpd/IsoffMainParser.cpp
+++ b/modules/demux/dash/mpd/IsoffMainParser.cpp
@@ -125,7 +125,7 @@ void IsoffMainParser::parsePeriods(Node *root)
std::vector<Node *> periods = DOMHelper::getElementByTagName(root, "Period", false);
std::vector<Node *>::const_iterator it;
- for(it = periods.begin(); it != periods.end(); it++)
+ for(it = periods.begin(); it != periods.end(); ++it)
{
Period *period = new (std::nothrow) Period(mpd);
if (!period)
@@ -204,7 +204,7 @@ void IsoffMainParser::setAdaptationSets (Node *periodNode, Period *period)
std::vector<Node *> adaptationSets = DOMHelper::getElementByTagName(periodNode, "AdaptationSet", false);
std::vector<Node *>::const_iterator it;
- for(it = adaptationSets.begin(); it != adaptationSets.end(); it++)
+ for(it = adaptationSets.begin(); it != adaptationSets.end(); ++it)
{
AdaptationSet *adaptationSet = new AdaptationSet(period);
if(!adaptationSet)
@@ -323,7 +323,7 @@ size_t IsoffMainParser::parseSegmentList(Node * segListNode, SegmentInformation
list->timescale.Set(Integer<uint64_t>(segListNode->getAttributeValue("timescale")));
std::vector<Node *>::const_iterator it;
- for(it = segments.begin(); it != segments.end(); it++)
+ for(it = segments.begin(); it != segments.end(); ++it)
{
Node *segmentURL = *it;
std::string mediaUrl = segmentURL->getAttributeValue("media");
@@ -387,7 +387,7 @@ void IsoffMainParser::parseTimeline(Node *node, MediaSegmentTemplate *templ)
{
std::vector<Node *> elements = DOMHelper::getElementByTagName(node, "S", false);
std::vector<Node *>::const_iterator it;
- for(it = elements.begin(); it != elements.end(); it++)
+ for(it = elements.begin(); it != elements.end(); ++it)
{
const Node *s = *it;
if(!s->hasAttribute("d")) /* Mandatory */
diff --git a/modules/demux/dash/mpd/MPD.cpp b/modules/demux/dash/mpd/MPD.cpp
index da41f83..7b23bf4 100644
--- a/modules/demux/dash/mpd/MPD.cpp
+++ b/modules/demux/dash/mpd/MPD.cpp
@@ -71,11 +71,11 @@ void MPD::debug()
msg_Dbg(stream, "BaseUrl=%s", getUrlSegment().toString().c_str());
std::vector<BasePeriod *>::const_iterator i;
- for(i = getPeriods().begin(); i != getPeriods().end(); i++)
+ for(i = getPeriods().begin(); i != getPeriods().end(); ++i)
{
std::vector<std::string> debug = (*i)->toString();
std::vector<std::string>::const_iterator l;
- for(l = debug.begin(); l < debug.end(); l++)
+ for(l = debug.begin(); l < debug.end(); ++l)
{
msg_Dbg(stream, "%s", (*l).c_str());
}
diff --git a/modules/demux/dash/xml/Node.cpp b/modules/demux/dash/xml/Node.cpp
index d7e0431..6d3704f 100644
--- a/modules/demux/dash/xml/Node.cpp
+++ b/modules/demux/dash/xml/Node.cpp
@@ -126,7 +126,7 @@ std::vector<std::string> Node::toString(int indent) const
text.append(getName());
ret.push_back(text);
std::vector<Node *>::const_iterator l;
- for(l = subNodes.begin(); l < subNodes.end(); l++)
+ for(l = subNodes.begin(); l < subNodes.end(); ++l)
{
std::vector<std::string> sub = (*l)->toString(indent + 1);
ret.insert(ret.end(), sub.begin(), sub.end());
diff --git a/modules/gui/qt4/components/interface_widgets.cpp b/modules/gui/qt4/components/interface_widgets.cpp
index 8bef5bd..0ae578a 100644
--- a/modules/gui/qt4/components/interface_widgets.cpp
+++ b/modules/gui/qt4/components/interface_widgets.cpp
@@ -375,7 +375,7 @@ void EasterEggBackgroundWidget::spawnFlakes()
it = flakes->erase( it );
}
else
- it++;
+ ++it;
}
if ( flakes->size() < MAX_FLAKES )
diff --git a/modules/gui/qt4/components/preferences_widgets.cpp b/modules/gui/qt4/components/preferences_widgets.cpp
index da724dc..39465f2 100644
--- a/modules/gui/qt4/components/preferences_widgets.cpp
+++ b/modules/gui/qt4/components/preferences_widgets.cpp
@@ -1262,7 +1262,7 @@ void KeySelectorControl::finish()
if( list.count() >= 2 )
msg_Dbg( p_this, "This is probably wrong, %s", qtu(i.key()) );
- i++;
+ ++i;
}
module_config_free (p_config);
diff --git a/modules/gui/skins2/utils/var_list.cpp b/modules/gui/skins2/utils/var_list.cpp
index 37770df..08b63df 100644
--- a/modules/gui/skins2/utils/var_list.cpp
+++ b/modules/gui/skins2/utils/var_list.cpp
@@ -56,12 +56,12 @@ void VarList::delSelected()
if( (*it).m_selected )
{
Iterator oldIt = it;
- it++;
+ ++it;
m_list.erase( oldIt );
}
else
{
- it++;
+ ++it;
}
}
notify();
--
2.1.4
More information about the vlc-devel
mailing list