Changeset 16427 for trunk/Mars/mcore


Ignore:
Timestamp:
05/29/13 14:03:15 (11 years ago)
Author:
tbretz
Message:
Replaced push_back by emplace_back and size() by empty() where reasonable; fixed a possible problem with invalidated iterators in SetStr
File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/Mars/mcore/ofits.h

    r16331 r16427  
    6363        {
    6464            key = Trim(key);
    65             if (key.size()==0)
     65            if (key.empty())
    6666            {
    6767#ifdef __EXCEPTIONS
     
    195195
    196196            sout << "= ";
    197             sout << (value[0]=='\''?std::left:std::right);
     197            sout << (!value.empty() && value[0]=='\''?std::left:std::right);
    198198            sout << setw(20) << value << std::left;
    199199
    200             if (comment.size()>0)
     200            if (!comment.empty())
    201201                sout << " / " << comment;
    202202
     
    428428    bool SetStr(const string &key, string s, const string &comment="")
    429429    {
    430         for (string::iterator c=s.begin(); c<s.end(); c++)
    431             if (*c=='\'')
    432                 s.insert(c++, '\'');
     430        for (uint i=0; i<s.length(); i++)
     431            if (s[i]=='\'')
     432                s.insert(i++, "\'");
    433433
    434434        return Set(key, true, "'"+s+"'", comment);
     
    492492        Set("END");
    493493        while (fKeys.size()%36!=0)
    494             fKeys.push_back(Key());
     494            fKeys.emplace_back();
    495495    }
    496496
     
    596596
    597597        Table::Column col;
     598
    598599        col.name   = name;
    599600        col.type   = typechar;
Note: See TracChangeset for help on using the changeset viewer.