Changeset 8627:6e0745ed5c22 in orange


Ignore:
Timestamp:
08/07/11 12:40:27 (3 years ago)
Author:
Noughmad <Noughmad@…>
Branch:
default
Convert:
e1b93a5c5de9a7281f70510c6261314b980868fd
Message:

Fix some issues reported by static code checker Krazy

Location:
source/orangeplot
Files:
8 edited

Legend:

Unmodified
Added
Removed
  • source/orangeplot/CMakeLists.txt

    r8521 r8627  
    2121    networkcurve.cpp 
    2222    plot.cpp  
    23     sceneeventfilter.cpp 
    2423) 
    2524qt4_automoc(${orangeplot_SRCS}) 
  • source/orangeplot/curve.cpp

    r8620 r8627  
    336336void Curve::cancelAllUpdates() 
    337337{ 
    338     foreach (QFuture<void> f, m_currentUpdate) 
    339     { 
    340         if (f.isRunning()) 
     338    QMap<UpdateFlag, QFuture< void > >::iterator it = m_currentUpdate.begin(); 
     339    QMap<UpdateFlag, QFuture< void > >::iterator end = m_currentUpdate.end(); 
     340    for (it; it != end; ++it) 
     341    { 
     342        if (it.value().isRunning()) 
    341343        { 
    342             f.cancel(); 
     344            it.value().cancel(); 
    343345        } 
    344346    } 
    345     foreach (QFuture<void> f, m_currentUpdate) 
    346     { 
    347         if (f.isRunning()) 
     347    for (it = m_currentUpdate.begin(); it != end; ++it) 
     348    { 
     349        if (it.value().isRunning()) 
    348350        { 
    349             f.waitForFinished(); 
     351            it.value().waitForFinished(); 
    350352        } 
    351353    } 
  • source/orangeplot/curve.h

    r8613 r8627  
    100100   **/ 
    101101  Curve(const QList< double >& x_data, const QList< double >& y_data, QGraphicsItem* parent = 0, QGraphicsScene* scene = 0); 
    102   Curve(QGraphicsItem* parent = 0, QGraphicsScene* scene = 0); 
     102  explicit Curve(QGraphicsItem* parent = 0, QGraphicsScene* scene = 0); 
    103103  /** 
    104104   * Default destructor 
  • source/orangeplot/networkcurve.cpp

    r8624 r8627  
    451451 
    452452        // update neighbours 
    453         for (i = 0; i < vertex->neighbours.size(); i++) 
     453        for (i = 0; i < vertex->neighbours.size(); ++i) 
    454454        { 
    455455            int ndx = vertex->neighbours[i]; 
     
    465465            std::vector<int> lCrossings; 
    466466            std::vector<int> rCrossings; 
    467             for (i = 0; i < positions.size(); i++) 
     467            for (i = 0; i < positions.size(); ++i) 
    468468            { 
    469469                int ndx = positions[i]; 
     
    482482            int rightCrossings = 0; 
    483483 
    484             for (i = 0; i < lCrossings.size(); i++) 
     484            for (i = 0; i < lCrossings.size(); --i) 
    485485                leftCrossings += lCrossings[i]; 
    486486 
    487487            rCrossings.push_back(left); 
    488             for (i = rCrossings.size() - 1; i > 0 ; i--) 
     488            for (i = rCrossings.size() - 1; i > 0 ; --i) 
    489489                rightCrossings += rCrossings[i] - rCrossings[i - 1]; 
    490490            //cout << "left: " << leftCrossings << " right: " <<rightCrossings << endl; 
     
    502502 
    503503    // Circular sifting 
    504     for (i = 0; i < positions.size(); i++) 
     504    for (i = 0; i < positions.size(); ++i) 
    505505        original[positions[i]]->position = i; 
    506506 
    507507    int step; 
    508     for (step = 0; step < 5; step++) 
    509     { 
    510         for (i = 0; i < m_nodes.size(); i++) 
     508    for (step = 0; step < 5; ++step) 
     509    { 
     510        for (i = 0; i < m_nodes.size(); ++i) 
    511511        { 
    512512            bool stop = false; 
     
    522522                int crossings = 0; 
    523523                int j,k; 
    524                 for (j = 0; j < u->neighbours.size(); j++) 
    525                     for (k = 0; k < v->neighbours.size(); k++) 
     524                for (j = 0; j < u->neighbours.size(); ++j) 
     525                    for (k = 0; k < v->neighbours.size(); ++k) 
    526526                        if ((original[u->neighbours[j]]->position == v->position) || (original[v->neighbours[k]]->position == u->position)) 
    527527                            midCrossings = (u->neighbours.size() - 1) * (v->neighbours.size() - 1) / 2; 
    528528                        else if ((original[u->neighbours[j]]->position + m_nodes.size() - u->position) % m_nodes.size() < (original[v->neighbours[k]]->position + m_nodes.size() - u->position) % m_nodes.size()) 
    529                             crossings++; 
     529                            ++crossings; 
    530530 
    531531                //cout << "v: " <<  v->ndx << " crossings: " << crossings << " u.n.size: " << u->neighbours.size() << " v.n.size: " << v->neighbours.size() << " mid: " << midCrossings << endl; 
     
    544544                positions.insert(positions.begin() + switchNdx, u->ndx); 
    545545 
    546                 for (j = i; j <= switchNdx; j++) 
     546                for (j = i; j <= switchNdx; ++j) 
    547547                    original[positions[j]]->position = j; 
    548548            } 
     
    559559    double fiStep = 2 * PI / m_nodes.size(); 
    560560 
    561     for (i = 0; i < m_nodes.size(); i++) 
     561    for (i = 0; i < m_nodes.size(); ++i) 
    562562    { 
    563563        m_nodes[positions[i]]->set_x(r * cos(fi) + xCenter); 
     
    566566    } 
    567567 
    568     for (std::vector<QueueVertex*>::iterator i = original.begin(); i != original.end(); ++i) 
    569         delete *i; 
     568        qDeleteAll(original); 
    570569 
    571570    original.clear(); 
     
    644643 
    645644    // iterations 
    646     for (i = 0; i < steps; i++) 
     645    for (i = 0; i < steps; ++i) 
    647646    { 
    648647        foreach (const NodeItem* node, m_nodes) 
  • source/orangeplot/networkcurve.h

    r8624 r8627  
    208208    typedef QMap<int, NodeItem*> Nodes; 
    209209 
    210     NetworkCurve(QGraphicsItem* parent = 0, QGraphicsScene* scene = 0); 
     210    explicit NetworkCurve(QGraphicsItem* parent = 0, QGraphicsScene* scene = 0); 
    211211    virtual ~NetworkCurve(); 
    212212 
  • source/orangeplot/plot.cpp

    r8626 r8627  
    66#include <QtCore/qmath.h> 
    77#include <limits> 
    8 #include "sceneeventfilter.h" 
    98 
    109inline uint qHash(const DataPoint& pos) 
  • source/orangeplot/plotitem.h

    r8613 r8627  
    1010 
    1111public: 
    12     PlotItem(QGraphicsItem* parent = 0, QGraphicsScene* scene = 0); 
     12    explicit PlotItem(QGraphicsItem* parent = 0, QGraphicsScene* scene = 0); 
    1313    virtual ~PlotItem(); 
    1414     
  • source/orangeplot/point.h

    r8613 r8627  
    7878    } 
    7979     
    80     Point(QGraphicsItem* parent = 0, QGraphicsScene* scene = 0); 
     80    explicit Point(QGraphicsItem* parent = 0, QGraphicsScene* scene = 0); 
    8181    Point(int symbol, QColor color, int size, QGraphicsItem* parent = 0); 
    8282    virtual ~Point(); 
Note: See TracChangeset for help on using the changeset viewer.